Skip to content

Remove unnecessary default index on tenantId when using default prima…#3817

Merged
jeremydmiller merged 9 commits into
JasperFx:masterfrom
schadowfax:removeTenantIdIndex
Jun 3, 2025
Merged

Remove unnecessary default index on tenantId when using default prima…#3817
jeremydmiller merged 9 commits into
JasperFx:masterfrom
schadowfax:removeTenantIdIndex

Conversation

@schadowfax

Copy link
Copy Markdown
Contributor

Remove unnecessary default index on tenantId when using default primary key tenancy ordering id then tenantId

connects #3670

@jeremydmiller

Copy link
Copy Markdown
Member

@schadowfax It's a breaking change. You'll have to make this opt out. If you'd gotten this in 48 hours ago, it's maybe fine, but not after the 8.0 release.

@jeremydmiller

Copy link
Copy Markdown
Member

Meh, I'll allow it I guess since it's just day 1, but we should have gotten this before.

@schadowfax

schadowfax commented Jun 2, 2025

Copy link
Copy Markdown
Contributor Author

@jeremydmiller i'm sorry i tryed when i read the possible release date but i failed to get to it in the weekend.

Glad your looking into it and considering it.

Only not sure about the documentation of the index. there is namely a =>

@jeremydmiller jeremydmiller merged commit ff8a777 into JasperFx:master Jun 3, 2025
3 checks passed
@schadowfax schadowfax deleted the removeTenantIdIndex branch February 3, 2026 09:24
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.

2 participants