You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@iceberg.apache.org by GitBox <gi...@apache.org> on 2019/09/30 16:17:59 UTC
[GitHub] [incubator-iceberg] rdblue opened a new issue #507: Remove
withPartitionPath from the public API
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:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org