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/25 19:39:04 UTC
[GitHub] [incubator-iceberg] prodeezy commented on issue #411: Ignore
unsupported partition transforms for forward compatibility
prodeezy commented on issue #411: Ignore unsupported partition transforms for forward compatibility
URL: https://github.com/apache/incubator-iceberg/pull/411#issuecomment-535179212
@rdblue any reason for making `PartitionSpec.Builder add(int sourceId, String name, String transform) ` non-public ? This currently is the only way one can apply a transform with custom name for the partition field. The others enforce a specific name e.g. `.day("timstamp")` names the column `timestamp_day` and there's no way to set that name to something custom.
----------------------------------------------------------------
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