sunchao commented on pull request #33348: URL: https://github.com/apache/spark/pull/33348#issuecomment-881092608
Thanks @dongjoon-hyun @HyukjinKwon and @viirya ! I've shared on the rationale why I went with the existing conf but please share your thoughts on this. If we go with a new config, it'll require a migration and I'm not sure how easy that is. > I'd like to recommend to put those two functions together in the same place which means moving PruneHiveTablePartitions.prunePartitions to ExternalCatalogUtils and adding the new function there too instead of here. HiveTableRelation is in catalyst module. WDYT Thanks for the suggestion. This is done. -- 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]
