Fix compound unique index for hierarchies_table #415
+10
−4
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
related issue: #410
I feel that the unique compound index of
ancestor_id
anddescendant_id
, is needed andgenerations
is redundant.Because there should be only one kind of
generations
for a set ofancestor_id
anddescendant_id
.The current table definition allows for the creation of multiple depth parent-child relationships.
example
items
item_hierarchies
Before the refurbishment, this record could also be added to the above.
However, If there is a search by
ancestor_id
,descendant_id
andgenerations
columns, considering performance point of view, the disappearance of theancestor_id
,descendant_id
andgenerations
composite indexes is not good.Therefore, I think it needs to be redefined.