gaborkaszab opened a new pull request, #7738: URL: https://github.com/apache/iceberg/pull/7738
The existing implementation of TableMigrationUtil.loadPartition() requires the partition values parameter to be in URL-encoded format. Otherwise the '/' character in the partition value would cause a parsing error. But if we give URL-encoded partition values to this functions then the partition values in the result DataFiles will be wrong. For instance the string value "11/12/13" on the partition path is URL-encoded to '11%2F12%2F13'. If you run loadPartition() then the resulting DataFiles will incorrectly have the URL-encoded partition values. As a solution this patch adds a new implementation of loadPartition() that doesn't require the caller to pass the partition values in a URL-encoded format and creates DataFiles having the correct partition values. Note, I kept the existing implementation as well and marked it as deprecated so that it can be dropped in a later release. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
