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.

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

Update tag_ident index when removing assignable column #25068

Closed
wants to merge 1 commit into from

Conversation

PVince81
Copy link
Member

@PVince81 PVince81 commented Jan 11, 2021

For #24813 (comment) as follow up of #25001

  • what to do with potential duplicate entries ?
  • add extra migration for those who already ran that one ?
  • test

@PVince81 PVince81 added the 2. developing Work in progress label Jan 11, 2021
@PVince81 PVince81 self-assigned this Jan 11, 2021
@PVince81
Copy link
Member Author

In case of duplicate entries, for example a tag called "X" with id=1 which is assignable and another one "X" with id=2 which is not, we'd need to delete the second one and then adjust all entries in "oc_systemtag_object_mapping" with something like UPDATE oc_systemtag_object_mapping SET systemtagid=1 WHERE systemtagid=2, and this for all the duplicate entries.

And do this before we even touch the index.

@PVince81 PVince81 removed their assignment Jan 30, 2023
@PVince81
Copy link
Member Author

no idea how to move forward due to dilemmas and so far it seems this hasn't been an issue for anyone, so I'll close for now

@PVince81 PVince81 closed this Jan 30, 2023
@skjnldsv skjnldsv deleted the bugfix/24813/update-tag-ident-index branch March 14, 2024 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2. developing Work in progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant