Skip to content

Conversation

@stevenzwu
Copy link
Contributor

SQL config default change is only applied to 1.20.

See the dev ML discussion here.

@stevenzwu stevenzwu requested a review from pvary October 17, 2024 22:02
@stevenzwu stevenzwu added this to the Iceberg 1.7.0 milestone Oct 17, 2024
ConfigOptions.key("table.exec.iceberg.use-flip27-source")
.booleanType()
.defaultValue(false)
.defaultValue(true)
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this is the default config change. only applied to 1.20

import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

@Experimental
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 want to say that this is PublicEvolving instead?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I feel it is fine to be just public.

@stevenzwu stevenzwu merged commit 2ac5c43 into apache:main Oct 18, 2024
@stevenzwu
Copy link
Contributor Author

thanks @pvary for the review

@stevenzwu stevenzwu deleted the flink-flip27-source-default branch October 18, 2024 16:07
zachdisc pushed a commit to zachdisc/iceberg that referenced this pull request Dec 23, 2024
czy006 pushed a commit to czy006/iceberg that referenced this pull request Apr 2, 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