eric-maynard commented on code in PR #1966: URL: https://github.com/apache/polaris/pull/1966#discussion_r2191021689
########## LICENSE: ########## @@ -222,6 +222,7 @@ This product includes code from Apache Iceberg. * service/common/src/main/java/org/apache/polaris/service/catalog/iceberg/CatalogHandlerUtils.java * plugins/spark/v3.5/spark/src/main/java/org/apache/polaris/spark/PolarisRESTCatalog.java * plugins/spark/v3.5/spark/src/main/java/org/apache/polaris/spark/SparkCatalog.java +* service/common/src/main/java/org/apache/polaris/service/catalog/common/LocationUtils.java Review Comment: I'm a big fan of always adding things to the end of lists like this to make potential merge conflicts easier to resolve. If there's some style guide for this file, we should maintain it though. -- 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...@polaris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
