Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix remaining search bugs #146

Closed
2 tasks done
cmgrote opened this issue Dec 16, 2019 · 0 comments · Fixed by #148
Closed
2 tasks done

Fix remaining search bugs #146

cmgrote opened this issue Dec 16, 2019 · 0 comments · Fixed by #148
Assignees
@cmgrote
Labels
bug Something isn't working
Milestone
1.3

Comments

@cmgrote
Copy link
Member

cmgrote commented Dec 16, 2019 •
edited
Loading

A number of subtle search bugs were discovered in v1.2:

  • searches against strings that should match qualifiedName are only possible using an exact match semantic, but should be possible as well with contains, startsWith and endsWith (even if there is a performance penalty associated with these)
  • searches against strings that should match qualifiedName were being used to restrict the search for optimization purposes in all cases, even when there is a match criteria of NONE (resulting in fewer results in some cases)
@cmgrote cmgrote added the bug Something isn't working label Dec 16, 2019
@cmgrote cmgrote added this to the 1.3 milestone Dec 16, 2019
@cmgrote cmgrote self-assigned this Dec 16, 2019
cmgrote referenced this issue in cmgrote/egeria-connector-ibm-information-server Dec 16, 2019
@cmgrote
Fixes #146
3197955 
Signed-off-by: Christopher Grote <chris@thegrotes.net>
@cmgrote cmgrote mentioned this issue Dec 16, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cmgrote cmgrote

Labels
bug Something isn't working
Projects
None yet
Milestone
1.3
Development

Successfully merging a pull request may close this issue.

CTS fixes cmgrote/egeria-connector-ibm-information-server
1 participant
@cmgrote