[GitHub] [iceberg] aokolnychyi edited a comment on pull request #1228: Fix Handling of SparkPartitions with Whitepsace in Location

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

aokolnychyi edited a comment on pull request #1228:
URL: https://github.com/apache/iceberg/pull/1228#issuecomment-662533530


   Based on the discussion in the issue, this looks like the correct direction. One question, though. How will this work when we construct a dataframe of partitions in `partitionDF`? I ran tests in `TestSparkTableUtil` locally and they failed too.
   
   I believe `SparkPartition` is a bean and Spark requires that all fields of it to be valid beans too. URI isn't a bean, right?


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