-
Notifications
You must be signed in to change notification settings - Fork 89
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.
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
Issue 6316 - lmdb reindex is broken if index type is specified #6318
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a typo and some questions.
Also is it related to the failure dogtagpki/pki#4839 ?
@tbordaz It is the issue with discussed in chat. |
f71e934
to
acee615
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple of minor polishing moments; otherwise - Looks Good!
assert keys2p > 0 | ||
assert keys2s > 0 | ||
|
||
# Reindex 'cn' without types |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
minor: this is "attr" not " 'cn'"
While reindexing using task or offline reindex, if the attribute name contains the index type (for example :eq,pres) Then the attribute is not reindexed. Problem occurs when lmdb is used, things are working fine with bdb. Solution: strip the index type in reindex as it is done in bdb case. Anyway the reindex design requires that for a given attribute all the configured index types must be rebuild. Issue: #6316 Reviewed by: @tbordaz, @droideck (Thanks!)
While reindexing using task or offline reindex, if the attribute name contains the index type (for example :eq,pres) Then the attribute is not reindexed. Problem occurs when lmdb is used, things are working fine with bdb. Solution: strip the index type in reindex as it is done in bdb case. Anyway the reindex design requires that for a given attribute all the configured index types must be rebuild. Issue: #6316 Reviewed by: @tbordaz, @droideck (Thanks!)
While reindexing using task or offline reindex, if the attribute name contains the index type (for example :eq,pres)
Then the attribute is not reindexed. Problem occurs when lmdb is used, things are working fine with bdb.
Solution: strip the index type in reindex as it is done in bdb case.
Anyway the reindex design requires that for a given attribute all the configured index types must be rebuild.
Issue: #6316
Reviewed by: @tbordaz, @droideck (Thanks!)