[GitHub] [iceberg] gaborkaszab commented on pull request #7738: Data: TableMigrationUtil shouldn't use path to get partition values

["gaborkaszab (via GitHub)" <gi...@apache.org>]

gaborkaszab commented on PR #7738:
URL: https://github.com/apache/iceberg/pull/7738#issuecomment-1570162761

   Hey @aokolnychyi, @szehon-ho, @amogh-jahagirdar,
   I found an issue wrt TableMigrationUtil.listPartition(). For details see https://github.com/apache/iceberg/issues/7612
   I'm tagging you because apparently Spark also has an issue with table migration when there are '/' chars in the partition columns. Could you please take a look at this fix? I added new functions for the fix while deprecated the old ones. Note, there is an extra 'schema' parameter needed for this solution, so I couldn't do the fix in-place.
   Once this fix is in, I can make a change on Spark side to use the new functions instead.


-- 
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: issues-unsubscribe@iceberg.apache.org

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