tomtongue commented on PR #12017: URL: https://github.com/apache/iceberg/pull/12017#issuecomment-3345281687
@flyrain Thanks for checking my comment. Regarding `metadata` that is added when customizing the view location using the `location` property, I don't think there's a big concern about it. It seems that it’s necessary to complicate the current code if you attempt to prevent the addition of `metadata` to the custom location when the view location is configured. This is because the `location` value is set around [here](https://github.com/apache/iceberg/blob/apache-iceberg-1.10.0/core/src/main/java/org/apache/iceberg/view/BaseMetastoreViewCatalog.java#L209). So, based on your suggestion and my investigation, please let me deprecate `write.metadata.path` parameter and add a test to validate customizing the view location with the `location` -- 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]
