[GitHub] [spark] dongjoon-hyun commented on pull request #34733: [SPARK-36346][SQL][FOLLOWUP] Rename `withAllOrcReaders` to `withAllNativeOrcReaders`

2021-11-29 Thread GitBox
dongjoon-hyun commented on pull request #34733: URL: https://github.com/apache/spark/pull/34733#issuecomment-981821610 Thank you all! Merged to master. -- 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

[GitHub] [spark] dongjoon-hyun commented on pull request #34733: [SPARK-36346][SQL][FOLLOWUP] Rename `withAllOrcReaders` to `withAllNativeOrcReaders`

2021-11-29 Thread GitBox
dongjoon-hyun commented on pull request #34733: URL: https://github.com/apache/spark/pull/34733#issuecomment-981821101 To @MaxGekk , as @cloud-fan mentioned, it's a big change and not a good approach because `SQL` module had batter be complete by itself. To @cloud-fan , yes, the