[GitHub] [iceberg] RussellSpitzer commented on pull request #1228: Fix Handling of SparkPartitions with Whitepsace in Location

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

RussellSpitzer commented on pull request #1228:
URL: https://github.com/apache/iceberg/pull/1228#issuecomment-662543772


   I think I mentioned this before, but we can pass the URI around as a string if Neccessary, we just need to reconvert it back into a URI before passing it into the Hadoop Path constructor. If we pass it as a String literal it assumes it has been un-escaped. This is just a little weirder solution because we end up keeping it in a different format that we actually use it.
   
   If we want to go that route, we can keep everything as a string, and then just at the last moment instead of passing the string directly do
   
   partitionUri = new URI(partitionUriString)
   path = new Path(partitionUri)


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