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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
