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 SQLServer clusterin… #8811

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

Conversation

m3nax
Copy link
Contributor

@m3nax m3nax commented Jan 11, 2024

…g script

Problem:

As written in the issue #8676 and docs, to set up ADO.NET clustering using the SQLServer you need to run 2 scripts in sequence:

However, the clustering script does not include the creation of the CleanupDefunctSiloEntriesKey query required by the ADO.NET implementation (Microsoft.Orleans.Clustering.AdoNet) in both the 7.2.x and the 8.0.x versions of Orleans.

Missing query causes an exception at program startup.

Proposed solution:

Add the missing query to clustering sql script.

Ref:

Microsoft Reviewers: Open in CodeFlow

juan-arias added a commit to juan-arias/orleans that referenced this pull request Feb 1, 2024
This is the same issue described in dotnet#8676 but for MySQL.

This PR will update it in a similar way as dotnet#8811
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.

1 participant