Skip to content

Conversation

@shawkins
Copy link
Contributor

closes: #43191

@shawkins shawkins requested a review from a team as a code owner October 16, 2025 18:58
ahus1
ahus1 previously approved these changes Oct 16, 2025
@ahus1
Copy link
Contributor

ahus1 commented Oct 16, 2025

@andymunro - please include this changes in the downstream docs for 26.4.x. Thanks!


[cols="1,1,1"]
|===
|Database |Minimum support |Dialect minimum
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we really want to state dialect minimum? It's not consistent with the guide where we don't state it. It might also be confusing to the users what is "dialect minimum". But I understand the motivation – before, we didn't state the supported versions in the upstream, and users are now surprised by the dialect limitations. So probably ok to keep it.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we really want to state dialect minimum?

Unfortunately from a user perspective that is the actual change they may notice, as was the case with #43191 and #43093

I'll go ahead and copy the statement about the dialect allowing for non-supported versions to the db guide.

Copy link
Contributor

@vmuzikar vmuzikar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks.

@vmuzikar vmuzikar merged commit 736d492 into keycloak:main Oct 17, 2025
56 checks passed
shawkins added a commit to shawkins/keycloak that referenced this pull request Oct 20, 2025
closes: keycloak#43191

Signed-off-by: Steve Hawkins <[email protected]>
(cherry picked from commit 736d492)
vmuzikar pushed a commit that referenced this pull request Oct 21, 2025
closes: #43191


(cherry picked from commit 736d492)

Signed-off-by: Steve Hawkins <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade guide for 26.4.0 should mention new minimal PostgreSQL server version 13 requirement

3 participants