rdblue opened a new issue #507: Remove withPartitionPath from the public API URL: https://github.com/apache/incubator-iceberg/issues/507 Several issues (#428, #505) have been opened where people are using `DataFiles.Builder#withPartitionPath` to parse partition key strings. This is an anti-pattern and Iceberg should not support serialization to and from string. Conversion to a partition key is a one-way conversion. Having the `withPartitionPath` method in the public API encourages using this anti-pattern. I think we should remove it.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
