Skip to content

Conversation

@stefankandic
Copy link
Contributor

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

Was this patch authored or co-authored using generative AI tooling?

@github-actions github-actions bot added the SQL label Oct 24, 2024
case Seq(_) => SqlApiConf.get.defaultStringType
case Seq(_) =>
val collationId = SqlApiConf.get.defaultStringType.collationId
StringType(collationId, CollationStrength.Implicit)
Copy link
Contributor

Choose a reason for hiding this comment

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

This probably relies on the change for session level default and DDL operations, but be careful of backwards compatibility.

@github-actions
Copy link

github-actions bot commented Feb 6, 2025

We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag!

@github-actions github-actions bot added the Stale label Feb 6, 2025
@github-actions github-actions bot closed this Feb 7, 2025
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.

2 participants