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

Add missing query CleanupDefunctSiloEntriesKey to PostgreSQL-Clustering #9125

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

tomaszstrejczek
Copy link

@tomaszstrejczek tomaszstrejczek commented Aug 25, 2024

Solves #8676 for PostgreSQL scripts by adding missing query CleanupDefunctSiloEntriesKey.
Follows the same pattern as #8896.

Microsoft Reviewers: Open in CodeFlow

@tomaszstrejczek
Copy link
Author

tomaszstrejczek commented Oct 11, 2024 via email

@NSTA2
Copy link

NSTA2 commented Feb 4, 2025

I'm still experiencing this issue with the postgres clustering provider - presumably because it looks like this PR hasn't completed yet.
I've applied the relevant Sql command to my database manually as an interim workaround (Will this complete soon @ReubenBond ?)
Many Thanks

@ReubenBond
Copy link
Member

This change seems to break the MySQL & MS SQL tests

@ReubenBond
Copy link
Member

I reverted the change to RelationalStorageForTesting.cs. It wasn't clear to me how it's a necessary part of this PR

@ReubenBond
Copy link
Member

Ok, it's because it was in the migrations directory. The idea is to run the base script and then the migration scripts in order. In this PR, you're merging a migration script back into the base script. That is OK by me, but we need to decide how to handle the migration script either by making them idempotent or by suppressing them from tests. The previous change to suppress them suppressed them for Postgres and the other providers, which caused other providers to fail.

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

Successfully merging this pull request may close these issues.

3 participants