[GitHub] [iceberg] boroknagyz commented on issue #2043: Partition data to path, and path to data conversions are not consistent with each other

[GitBox <gi...@apache.org>]

boroknagyz commented on issue #2043:
URL: https://github.com/apache/iceberg/issues/2043#issuecomment-757885743


   Thanks for your reply, @rdblue.
   In Impala we also use Builder.withPartitionPath, because we generate the partition path already in our C++ backend, so this was the easiest way to transfer partition data from C++ to Java.
   
   I see that Iceberg  unit test code also uses withPartitionPath to conveniently create partition data, so for Iceberg this conversion could be useful for unit tests at least.
   
   With that said, if you think two-way conversion could be useful for Iceberg, then I'll happily contribute code. If not, that's also OK for me, I can add partition path -> partition data conversion to Impala's code base.


----------------------------------------------------------------
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



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org