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

Remove triggers on label table #2231

Open
lampajr opened this issue Jan 8, 2025 · 1 comment · May be fixed by #2230
Open

Remove triggers on label table #2231

lampajr opened this issue Jan 8, 2025 · 1 comment · May be fixed by #2230
Assignees
Labels

Comments

@lampajr
Copy link
Member

lampajr commented Jan 8, 2025

Feature idea.

There are still some triggers on the label table that are related to the old label values recalculation mechanism that is no longer in use. Get rid of them.

Affected entities:

event_object_table trigger_name event_manipulation action_statement action_timing
label               lv_before_update           UPDATE             EXECUTE FUNCTION lv_before_label_update_func()     BEFORE
label               lv_after_update             UPDATE             EXECUTE FUNCTION lv_after_label_update_func()     AFTER
label               lv_after_update             INSERT             EXECUTE FUNCTION lv_after_label_update_func()     AFTER
label_recalc_queue recalc_labels               INSERT             EXECUTE FUNCTION recalc_label_values()             AFTER

Describe the solution you'd like

Remove the aforementioned triggers and functions.

As part of this effort, ensure the above triggers and functions get removed.

Additional information

n/a

@lampajr lampajr added type/feature A new feature area/backend priority/medium Medium priority labels Jan 8, 2025
@lampajr lampajr self-assigned this Jan 8, 2025
@lampajr lampajr linked a pull request Jan 8, 2025 that will close this issue
5 tasks
@lampajr
Copy link
Member Author

lampajr commented Jan 8, 2025

We could also get rid of label_recalc_queue table as no longer used once all triggers (and functions) mentioned above are removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant