Skip to content

Ensure MSSQL sever deployments have the IDX_CLIENT_ATT_BY_NAME_VALUE …#48737

Open
sguilhen wants to merge 1 commit intokeycloak:mainfrom
sguilhen:48735-missing-mssql-client-idx
Open

Ensure MSSQL sever deployments have the IDX_CLIENT_ATT_BY_NAME_VALUE …#48737
sguilhen wants to merge 1 commit intokeycloak:mainfrom
sguilhen:48735-missing-mssql-client-idx

Conversation

@sguilhen
Copy link
Copy Markdown
Contributor

@sguilhen sguilhen commented May 6, 2026

…index created

Closes #48735

@sguilhen sguilhen requested a review from a team as a code owner May 6, 2026 02:57
@sguilhen sguilhen requested a review from ahus1 May 6, 2026 02:57
@ahus1 ahus1 added the status/hold PR should not be merged. On hold for later. label May 7, 2026
@ahus1
Copy link
Copy Markdown
Member

ahus1 commented May 7, 2026

Adding the hold label as the discussion continues in the parent issue.

@sguilhen sguilhen force-pushed the 48735-missing-mssql-client-idx branch 2 times, most recently from 3852f78 to 17df4a8 Compare May 7, 2026 21:05
@sschu
Copy link
Copy Markdown
Contributor

sschu commented May 8, 2026

@sguilhen Will the non-mssql index work with Oracle? Because you said there was also an issue there and the index was name-only?

@sguilhen
Copy link
Copy Markdown
Contributor Author

sguilhen commented May 8, 2026

@sschu I haven't touched the Oracle index in this PR, for now it is still a name-only index. Maybe handle that as a follow up?

@sguilhen sguilhen force-pushed the 48735-missing-mssql-client-idx branch from 17df4a8 to 38673d4 Compare May 8, 2026 10:06
@sschu
Copy link
Copy Markdown
Contributor

sschu commented May 8, 2026

@sguilhen Ah, sorry, I misread the index creation. It only affects MS SQL so should be fine.

…created

Closes keycloak#48735

Signed-off-by: Stefan Guilhen <sguilhen@redhat.com>
@sguilhen sguilhen force-pushed the 48735-missing-mssql-client-idx branch from 9f50f37 to 5f24d85 Compare May 8, 2026 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

flaky-test status/hold PR should not be merged. On hold for later.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Re-introduce index idx_client_att_by_name_value for MSSQL

3 participants