gengliangwang edited a comment on pull request #28383: URL: https://github.com/apache/spark/pull/28383#issuecomment-623875587
> Shall we remove OptimizeMetadataOnlyQuery? IIRC it has a correcness issue and we disable it by default. cc @gengliangwang On second thought: I think we should keep it for two reasons: 1. when users are 100% sure about their data won't contain empty partition, they can still turn it on. 2. the future developers may come up with the same idea and create exactly the same rule and enable it by default... ---------------------------------------------------------------- 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
