ajantha-bhat commented on code in PR #9459:
URL: https://github.com/apache/iceberg/pull/9459#discussion_r1449688656
##########
nessie/src/main/java/org/apache/iceberg/nessie/NessieCatalog.java:
##########
@@ -128,6 +133,26 @@ public void initialize(String name, Map<String, String>
options) {
catalogOptions);
}
+ private static String inferVersionFromURI(String uri) {
+ if (uri == null) {
+ throw new IllegalArgumentException("URI is not specified in the catalog
properties");
+ }
+
+ // match for uri ending with /v1, /v2 etc
+ Pattern pattern = Pattern.compile("/v(\\d+)$");
+ Matcher matcher = pattern.matcher(uri);
+ // Extract and print the version number
+ if (matcher.find()) {
+ return matcher.group(1);
+ } else {
+ throw new IllegalArgumentException(
+ String.format(
+ "URI doesn't end with the version: %s. "
+ + "Please configure `client-api-version` in the catalog
properties explicitly.",
Review Comment:
If the uri is not ending versions, then it can be a custom implementation
which can be based on v1 or v2. So, I didn't want to assume the version there.
--
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: issues-unsubscr...@iceberg.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org
