rdblue commented on pull request #30421: URL: https://github.com/apache/spark/pull/30421#issuecomment-733979208
In general, I support the idea of moving away from tracking partition values using String. I'm not convinced that using a Literal would work much more easily because literal values would need to be cast to the correct type when constructing an `InternalRow` as well. But the idea seems like it's going in the right direction. ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
