[GitHub] [spark] maropu commented on pull request #31598: [SPARK-34478][SQL] When build SparkSession, we should check config keys

2021-03-18 Thread GitBox
maropu commented on pull request #31598: URL: https://github.com/apache/spark/pull/31598#issuecomment-802461832 You've already opened a new PR to address the @cloud-fan comment? https://github.com/apache/spark/pull/31598#discussion_r586155367 The direction looks reasonable to me, too.

[GitHub] [spark] maropu commented on pull request #31598: [SPARK-34478][SQL] When build SparkSession, we should check config keys

2021-02-28 Thread GitBox
maropu commented on pull request #31598: URL: https://github.com/apache/spark/pull/31598#issuecomment-787562300 For reviews, could you add the screenshot of the updated doc part? This is an automated message from the Apache