[GitHub] spark pull request #22494: [SPARK-22036][SQL][followup] add a new config for...

2018-09-21 Thread mgaido91
Github user mgaido91 commented on a diff in the pull request: https://github.com/apache/spark/pull/22494#discussion_r219505527 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala --- @@ -1345,6 +1345,15 @@ object SQLConf { .booleanConf

[GitHub] spark pull request #22494: [SPARK-22036][SQL][followup] add a new config for...

2018-09-21 Thread mgaido91
Github user mgaido91 commented on a diff in the pull request: https://github.com/apache/spark/pull/22494#discussion_r219493420 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala --- @@ -2858,6 +2858,13 @@ class SQLQuerySuite extends QueryTest with

[GitHub] spark pull request #22494: [SPARK-22036][SQL][followup] add a new config for...

2018-09-21 Thread mgaido91
Github user mgaido91 commented on a diff in the pull request: https://github.com/apache/spark/pull/22494#discussion_r219492191 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala --- @@ -1345,6 +1345,16 @@ object SQLConf { .booleanConf

[GitHub] spark pull request #22494: [SPARK-22036][SQL][followup] add a new config for...

2018-09-21 Thread mgaido91
Github user mgaido91 commented on a diff in the pull request: https://github.com/apache/spark/pull/22494#discussion_r219493091 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala --- @@ -1345,6 +1345,16 @@ object SQLConf { .booleanConf

[GitHub] spark pull request #22494: [SPARK-22036][SQL][followup] add a new config for...

2018-09-21 Thread mgaido91
Github user mgaido91 commented on a diff in the pull request: https://github.com/apache/spark/pull/22494#discussion_r219492885 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala --- @@ -1345,6 +1345,16 @@ object SQLConf { .booleanConf