eric-maynard commented on code in PR #1378:
URL: https://github.com/apache/polaris/pull/1378#discussion_r2047909097
##########
polaris-core/src/main/java/org/apache/polaris/core/config/BehaviorChangeConfiguration.java:
##########
@@ -59,4 +59,13 @@ protected BehaviorChangeConfiguration(
.description("Whether or not to use soft values in the entity cache")
.defaultValue(false)
.buildBehaviorChangeConfiguration();
+
+ public static final BehaviorChangeConfiguration<Boolean>
TABLE_OPERATIONS_COMMIT_UPDATE_METADATA =
+ PolarisConfiguration.<Boolean>builder()
+ .key("TABLE_OPERATIONS_COMMIT_UPDATE_METADATA")
+ .description(
+ "If true, BasePolarisTableOperations should cache metadata that
has been committed"
Review Comment:
Good callout, there is a mismatch between the description and the name of
this flag after my rebase. I will adjust the description for now but let me
know if you think we need further changes
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
