nastra commented on code in PR #15057:
URL: https://github.com/apache/iceberg/pull/15057#discussion_r2697397628
##########
core/src/main/java/org/apache/iceberg/rest/RESTCatalogProperties.java:
##########
@@ -24,6 +24,7 @@ private RESTCatalogProperties() {}
public static final String SNAPSHOT_LOADING_MODE = "snapshot-loading-mode";
public static final String SNAPSHOT_LOADING_MODE_DEFAULT =
SnapshotMode.ALL.name();
+ public static final String SNAPSHOTS_QUERY_PARAMETER = "snapshots";
Review Comment:
is it worth exposing the query parameter here in the catalog properties? I
would prefer if we wouldn't add another property for this and just leave things
around "snapshots" as they are
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
