-
Notifications
You must be signed in to change notification settings - Fork 1.2k
fix: SQLite online store deletes tables from other projects in shared registry scenarios #5766
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
Conversation
Signed-off-by: Aniket Paluskar <[email protected]>
Test Results: Before vs After FixBefore Fix (Bug Present)Test Case 1Applying real_snake project (shares registry with modern_hermit) Test Case 2Re-applying modern_hermit project Terminal OutputAfter Fix (Bug Resolved)Test Case 1Applying fit_impala project Test Case 2: Applying premium_dory project (shares registry with fit_impala)Applying changes for project premium_dory Terminal Output |
Signed-off-by: Aniket Paluskar <[email protected]>
ntkathole
left a comment
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.
Looks good
What this PR does / why we need it:
When multiple projects share a SQL registry and the same online store database, applying changes to one project incorrectly deletes tables from other projects.
Root cause analysis
Fix
Which issue(s) this PR fixes:
#5616
RHOAIENG-34863
Misc