[GitHub] [spark] SparkQA commented on issue #26668: [SPARK-30031][BUILD][SQL] Remove `hive-2.3` profile from `sql/hive/pom.xml`

2019-11-25 Thread GitBox
SparkQA commented on issue #26668: [SPARK-30031][BUILD][SQL] Remove `hive-2.3` profile from `sql/hive/pom.xml` URL: https://github.com/apache/spark/pull/26668#issuecomment-558322672 **[Test build #114428 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core`

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core` URL: https://github.com/apache/spark/pull/26658#issuecomment-558320539 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core`

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core` URL: https://github.com/apache/spark/pull/26658#issuecomment-558320551 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins commented on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core`

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core` URL: https://github.com/apache/spark/pull/26658#issuecomment-558320551 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core`

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core` URL: https://github.com/apache/spark/pull/26658#issuecomment-558320539 Merged build finished. Test PASSed.

[GitHub] [spark] dongjoon-hyun opened a new pull request #26668: [SPARK-30031][BUILD][SQL] Remove `hive-2.3` profile from `sql/hive/pom.xml`

2019-11-25 Thread GitBox
dongjoon-hyun opened a new pull request #26668: [SPARK-30031][BUILD][SQL] Remove `hive-2.3` profile from `sql/hive/pom.xml` URL: https://github.com/apache/spark/pull/26668 … ### What changes were proposed in this pull request? ### Why are the changes needed?

[GitHub] [spark] SparkQA removed a comment on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core`

2019-11-25 Thread GitBox
SparkQA removed a comment on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core` URL: https://github.com/apache/spark/pull/26658#issuecomment-558260901 **[Test build #114416 has

[GitHub] [spark] SparkQA commented on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core`

2019-11-25 Thread GitBox
SparkQA commented on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core` URL: https://github.com/apache/spark/pull/26658#issuecomment-558319876 **[Test build #114416 has

[GitHub] [spark] LucaCanali commented on issue #24132: [SPARK-27189][CORE] Add Executor metrics and memory usage instrumentation to the metrics system

2019-11-25 Thread GitBox
LucaCanali commented on issue #24132: [SPARK-27189][CORE] Add Executor metrics and memory usage instrumentation to the metrics system URL: https://github.com/apache/spark/pull/24132#issuecomment-558318439 I think it should be OK to have ExecutorMetrics on by default. It is the case for

[GitHub] [spark] dongjoon-hyun closed pull request #26666: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `org.apache.commons.lang.`

2019-11-25 Thread GitBox
dongjoon-hyun closed pull request #2: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `org.apache.commons.lang.` URL: https://github.com/apache/spark/pull/2 This is an automated message from

[GitHub] [spark] AmplabJenkins removed a comment on issue #26649: [SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26649: [SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26649#issuecomment-558317751 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] AmplabJenkins removed a comment on issue #26649: [SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26649: [SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26649#issuecomment-558317742 Merged build finished. Test PASSed.

[GitHub] [spark] dongjoon-hyun commented on issue #26666: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex`

2019-11-25 Thread GitBox
dongjoon-hyun commented on issue #2: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex` URL: https://github.com/apache/spark/pull/2#issuecomment-558317675 Since the GitHub Action Linter passed with the new rule and the changed code, I'll merge this PR.

[GitHub] [spark] AmplabJenkins commented on issue #26649: [SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26649: [SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26649#issuecomment-558317742 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26649: [SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26649: [SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26649#issuecomment-558317751 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] SparkQA commented on issue #26649: [SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands

2019-11-25 Thread GitBox
SparkQA commented on issue #26649: [SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26649#issuecomment-558317104 **[Test build #114427 has

[GitHub] [spark] huaxingao commented on a change in pull request #26649: [SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands

2019-11-25 Thread GitBox
huaxingao commented on a change in pull request #26649: [SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26649#discussion_r350391937 ## File path:

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #26666: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex`

2019-11-25 Thread GitBox
dongjoon-hyun commented on a change in pull request #2: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex` URL: https://github.com/apache/spark/pull/2#discussion_r350388967 ## File path: scalastyle-config.xml ## @@ -258,7 +258,7 @@ This

[GitHub] [spark] AmplabJenkins removed a comment on issue #26666: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex`

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #2: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex` URL: https://github.com/apache/spark/pull/2#issuecomment-558314357 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26666: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex`

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #2: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex` URL: https://github.com/apache/spark/pull/2#issuecomment-558314354 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26666: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex`

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #2: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex` URL: https://github.com/apache/spark/pull/2#issuecomment-558314357 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26666: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex`

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #2: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex` URL: https://github.com/apache/spark/pull/2#issuecomment-558314354 Merged build finished. Test PASSed.

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #26666: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex`

2019-11-25 Thread GitBox
dongjoon-hyun commented on a change in pull request #2: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex` URL: https://github.com/apache/spark/pull/2#discussion_r350388967 ## File path: scalastyle-config.xml ## @@ -258,7 +258,7 @@ This

[GitHub] [spark] SparkQA commented on issue #26666: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex`

2019-11-25 Thread GitBox
SparkQA commented on issue #2: [SPARK-30030][INFRA] Use RegexChecker instead of TokenChecker to check `regex` URL: https://github.com/apache/spark/pull/2#issuecomment-558313912 **[Test build #114425 has

[GitHub] [spark] SparkQA commented on issue #26082: [SPARK-29431][WebUI] Improve Web UI / Sql tab visualization with cached dataframes.

2019-11-25 Thread GitBox
SparkQA commented on issue #26082: [SPARK-29431][WebUI] Improve Web UI / Sql tab visualization with cached dataframes. URL: https://github.com/apache/spark/pull/26082#issuecomment-558313910 **[Test build #114426 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #26651: [SPARK-30008][SQL] The dataType of collect_list/collect_set aggs should be ArrayType(_, false)

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26651: [SPARK-30008][SQL] The dataType of collect_list/collect_set aggs should be ArrayType(_, false) URL: https://github.com/apache/spark/pull/26651#issuecomment-558313160 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] AmplabJenkins removed a comment on issue #26651: [SPARK-30008][SQL] The dataType of collect_list/collect_set aggs should be ArrayType(_, false)

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26651: [SPARK-30008][SQL] The dataType of collect_list/collect_set aggs should be ArrayType(_, false) URL: https://github.com/apache/spark/pull/26651#issuecomment-558313150 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26651: [SPARK-30008][SQL] The dataType of collect_list/collect_set aggs should be ArrayType(_, false)

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26651: [SPARK-30008][SQL] The dataType of collect_list/collect_set aggs should be ArrayType(_, false) URL: https://github.com/apache/spark/pull/26651#issuecomment-558313160 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins commented on issue #26651: [SPARK-30008][SQL] The dataType of collect_list/collect_set aggs should be ArrayType(_, false)

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26651: [SPARK-30008][SQL] The dataType of collect_list/collect_set aggs should be ArrayType(_, false) URL: https://github.com/apache/spark/pull/26651#issuecomment-558313150 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA removed a comment on issue #26651: [SPARK-30008][SQL] The dataType of collect_list/collect_set aggs should be ArrayType(_, false)

2019-11-25 Thread GitBox
SparkQA removed a comment on issue #26651: [SPARK-30008][SQL] The dataType of collect_list/collect_set aggs should be ArrayType(_, false) URL: https://github.com/apache/spark/pull/26651#issuecomment-558208448 **[Test build #114411 has

[GitHub] [spark] SparkQA commented on issue #26651: [SPARK-30008][SQL] The dataType of collect_list/collect_set aggs should be ArrayType(_, false)

2019-11-25 Thread GitBox
SparkQA commented on issue #26651: [SPARK-30008][SQL] The dataType of collect_list/collect_set aggs should be ArrayType(_, false) URL: https://github.com/apache/spark/pull/26651#issuecomment-558312435 **[Test build #114411 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #26082: [SPARK-29431][WebUI] Improve Web UI / Sql tab visualization with cached dataframes.

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26082: [SPARK-29431][WebUI] Improve Web UI / Sql tab visualization with cached dataframes. URL: https://github.com/apache/spark/pull/26082#issuecomment-558311307 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26633: [SPARK-29994][CORE] Add WILDCARD task location

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26633: [SPARK-29994][CORE] Add WILDCARD task location URL: https://github.com/apache/spark/pull/26633#issuecomment-558311338 Merged build finished. Test PASSed. This is an automated message

[GitHub] [spark] AmplabJenkins removed a comment on issue #26633: [SPARK-29994][CORE] Add WILDCARD task location

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26633: [SPARK-29994][CORE] Add WILDCARD task location URL: https://github.com/apache/spark/pull/26633#issuecomment-558311347 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26082: [SPARK-29431][WebUI] Improve Web UI / Sql tab visualization with cached dataframes.

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26082: [SPARK-29431][WebUI] Improve Web UI / Sql tab visualization with cached dataframes. URL: https://github.com/apache/spark/pull/26082#issuecomment-558311320 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins commented on issue #26082: [SPARK-29431][WebUI] Improve Web UI / Sql tab visualization with cached dataframes.

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26082: [SPARK-29431][WebUI] Improve Web UI / Sql tab visualization with cached dataframes. URL: https://github.com/apache/spark/pull/26082#issuecomment-558311320 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26633: [SPARK-29994][CORE] Add WILDCARD task location

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26633: [SPARK-29994][CORE] Add WILDCARD task location URL: https://github.com/apache/spark/pull/26633#issuecomment-558311347 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26082: [SPARK-29431][WebUI] Improve Web UI / Sql tab visualization with cached dataframes.

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26082: [SPARK-29431][WebUI] Improve Web UI / Sql tab visualization with cached dataframes. URL: https://github.com/apache/spark/pull/26082#issuecomment-558311307 Merged build finished. Test PASSed.

[GitHub] [spark] planga82 commented on issue #26082: [SPARK-29431][WebUI] Improve Web UI / Sql tab visualization with cached dataframes.

2019-11-25 Thread GitBox
planga82 commented on issue #26082: [SPARK-29431][WebUI] Improve Web UI / Sql tab visualization with cached dataframes. URL: https://github.com/apache/spark/pull/26082#issuecomment-558311529 @dongjoon-hyun What is your opinion about this changes? Thank you

[GitHub] [spark] AmplabJenkins removed a comment on issue #26633: [SPARK-29994][CORE] Add WILDCARD task location

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26633: [SPARK-29994][CORE] Add WILDCARD task location URL: https://github.com/apache/spark/pull/26633#issuecomment-558311338 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] SparkQA removed a comment on issue #26633: [SPARK-29994][CORE] Add WILDCARD task location

2019-11-25 Thread GitBox
SparkQA removed a comment on issue #26633: [SPARK-29994][CORE] Add WILDCARD task location URL: https://github.com/apache/spark/pull/26633#issuecomment-558244366 **[Test build #114415 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114415/testReport)** for

[GitHub] [spark] AmplabJenkins removed a comment on issue #26667: [SPARK-29922][SQL] SHOW FUNCTIONS should do multi-catalog resolution

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26667: [SPARK-29922][SQL] SHOW FUNCTIONS should do multi-catalog resolution URL: https://github.com/apache/spark/pull/26667#issuecomment-558307932 Can one of the admins verify this patch?

[GitHub] [spark] AmplabJenkins commented on issue #26667: [SPARK-29922][SQL] SHOW FUNCTIONS should do multi-catalog resolution

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26667: [SPARK-29922][SQL] SHOW FUNCTIONS should do multi-catalog resolution URL: https://github.com/apache/spark/pull/26667#issuecomment-558310499 Can one of the admins verify this patch?

[GitHub] [spark] SparkQA commented on issue #26633: [SPARK-29994][CORE] Add WILDCARD task location

2019-11-25 Thread GitBox
SparkQA commented on issue #26633: [SPARK-29994][CORE] Add WILDCARD task location URL: https://github.com/apache/spark/pull/26633#issuecomment-558310516 **[Test build #114415 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/114415/testReport)** for PR

[GitHub] [spark] SparkQA commented on issue #26082: [SPARK-29431][WebUI] Improve Web UI / Sql tab visualization with cached dataframes.

2019-11-25 Thread GitBox
SparkQA commented on issue #26082: [SPARK-29431][WebUI] Improve Web UI / Sql tab visualization with cached dataframes. URL: https://github.com/apache/spark/pull/26082#issuecomment-558310694 **[Test build #114424 has

[GitHub] [spark] viirya commented on a change in pull request #26649: [SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands

2019-11-25 Thread GitBox
viirya commented on a change in pull request #26649: [SPARK-29862][SQL] CREATE (OR REPLACE) ... VIEW should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26649#discussion_r350384426 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #26667: [SPARK-29922][SQL] SHOW FUNCTIONS should do multi-catalog resolution

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26667: [SPARK-29922][SQL] SHOW FUNCTIONS should do multi-catalog resolution URL: https://github.com/apache/spark/pull/26667#issuecomment-558307932 Can one of the admins verify this patch?

[GitHub] [spark] planga82 commented on issue #26667: [SPARK-29922][SQL] SHOW FUNCTIONS should do multi-catalog resolution

2019-11-25 Thread GitBox
planga82 commented on issue #26667: [SPARK-29922][SQL] SHOW FUNCTIONS should do multi-catalog resolution URL: https://github.com/apache/spark/pull/26667#issuecomment-558307837 cc: @cloud-fan @rdblue @viirya @imback82 This is

[GitHub] [spark] planga82 opened a new pull request #26667: [SPARK-29922][SQL] SHOW FUNCTIONS should do multi-catalog resolution

2019-11-25 Thread GitBox
planga82 opened a new pull request #26667: [SPARK-29922][SQL] SHOW FUNCTIONS should do multi-catalog resolution URL: https://github.com/apache/spark/pull/26667 ### What changes were proposed in this pull request? Add ShowFunctionsStatement and make SHOW FUNCTIONS go through

[GitHub] [spark] dongjoon-hyun commented on issue #26666: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2

2019-11-25 Thread GitBox
dongjoon-hyun commented on issue #2: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2 URL: https://github.com/apache/spark/pull/2#issuecomment-558306997 Thank you for review, @databricks-david-lewis . Yes. I'm also suspecting that and checking now (if we

[GitHub] [spark] databricks-david-lewis commented on issue #26666: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2

2019-11-25 Thread GitBox
databricks-david-lewis commented on issue #2: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2 URL: https://github.com/apache/spark/pull/2#issuecomment-558305738 LGTM, sorry for letting it sneak back in! Maybe benchmark code is excluded from that check?

[GitHub] [spark] SparkQA commented on issue #26664: [SPARK-9853][Doc][Follow-up] Update the description for all the SQLConf affected by spark.sql.adaptive.enabled

2019-11-25 Thread GitBox
SparkQA commented on issue #26664: [SPARK-9853][Doc][Follow-up] Update the description for all the SQLConf affected by spark.sql.adaptive.enabled URL: https://github.com/apache/spark/pull/26664#issuecomment-558304425 **[Test build #114423 has

[GitHub] [spark] dongjoon-hyun commented on issue #26666: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2

2019-11-25 Thread GitBox
dongjoon-hyun commented on issue #2: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2 URL: https://github.com/apache/spark/pull/2#issuecomment-558303060 Oh, interesting. We have a rule for that. Let me dig the reason how this escape from the checker. ```

[GitHub] [spark] AmplabJenkins removed a comment on issue #26664: [SPARK-9853][Doc][Follow-up] Update the description for all the SQLConf affected by spark.sql.adaptive.enabled

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26664: [SPARK-9853][Doc][Follow-up] Update the description for all the SQLConf affected by spark.sql.adaptive.enabled URL: https://github.com/apache/spark/pull/26664#issuecomment-558301849 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26655: [SPARK-29293][BUILD] Update scalafmt plugin to 1.0.2 for compatibility with Scala 2.13

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26655: [SPARK-29293][BUILD] Update scalafmt plugin to 1.0.2 for compatibility with Scala 2.13 URL: https://github.com/apache/spark/pull/26655#issuecomment-558301853 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26664: [SPARK-9853][Doc][Follow-up] Update the description for all the SQLConf affected by spark.sql.adaptive.enabled

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26664: [SPARK-9853][Doc][Follow-up] Update the description for all the SQLConf affected by spark.sql.adaptive.enabled URL: https://github.com/apache/spark/pull/26664#issuecomment-558301858 Test PASSed. Refer to this link for build results

[GitHub] [spark] AmplabJenkins removed a comment on issue #26655: [SPARK-29293][BUILD] Update scalafmt plugin to 1.0.2 for compatibility with Scala 2.13

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26655: [SPARK-29293][BUILD] Update scalafmt plugin to 1.0.2 for compatibility with Scala 2.13 URL: https://github.com/apache/spark/pull/26655#issuecomment-558301863 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] xuanyuanking commented on a change in pull request #26664: [SPARK-9853][Doc][Follow-up] Update the description for all the SQLConf affected by spark.sql.adaptive.enabled

2019-11-25 Thread GitBox
xuanyuanking commented on a change in pull request #26664: [SPARK-9853][Doc][Follow-up] Update the description for all the SQLConf affected by spark.sql.adaptive.enabled URL: https://github.com/apache/spark/pull/26664#discussion_r350375087 ## File path:

[GitHub] [spark] AmplabJenkins commented on issue #26664: [SPARK-9853][Doc][Follow-up] Update the description for all the SQLConf affected by spark.sql.adaptive.enabled

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26664: [SPARK-9853][Doc][Follow-up] Update the description for all the SQLConf affected by spark.sql.adaptive.enabled URL: https://github.com/apache/spark/pull/26664#issuecomment-558301858 Test PASSed. Refer to this link for build results (access rights

[GitHub] [spark] AmplabJenkins commented on issue #26655: [SPARK-29293][BUILD] Update scalafmt plugin to 1.0.2 for compatibility with Scala 2.13

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26655: [SPARK-29293][BUILD] Update scalafmt plugin to 1.0.2 for compatibility with Scala 2.13 URL: https://github.com/apache/spark/pull/26655#issuecomment-558301853 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26655: [SPARK-29293][BUILD] Update scalafmt plugin to 1.0.2 for compatibility with Scala 2.13

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26655: [SPARK-29293][BUILD] Update scalafmt plugin to 1.0.2 for compatibility with Scala 2.13 URL: https://github.com/apache/spark/pull/26655#issuecomment-558301863 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26664: [SPARK-9853][Doc][Follow-up] Update the description for all the SQLConf affected by spark.sql.adaptive.enabled

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26664: [SPARK-9853][Doc][Follow-up] Update the description for all the SQLConf affected by spark.sql.adaptive.enabled URL: https://github.com/apache/spark/pull/26664#issuecomment-558301849 Merged build finished. Test PASSed.

[GitHub] [spark] shahidki31 commented on issue #26616: [SPARK-25392][Webui]Prevent error page when accessing pools page from history server

2019-11-25 Thread GitBox
shahidki31 commented on issue #26616: [SPARK-25392][Webui]Prevent error page when accessing pools page from history server URL: https://github.com/apache/spark/pull/26616#issuecomment-558301458 cc @vanzin @srowen Could you please review this patch. seems a minor change to fix avoiding the

[GitHub] [spark] SparkQA commented on issue #26655: [SPARK-29293][BUILD] Update scalafmt plugin to 1.0.2 for compatibility with Scala 2.13

2019-11-25 Thread GitBox
SparkQA commented on issue #26655: [SPARK-29293][BUILD] Update scalafmt plugin to 1.0.2 for compatibility with Scala 2.13 URL: https://github.com/apache/spark/pull/26655#issuecomment-558301344 **[Test build #114422 has

[GitHub] [spark] dongjoon-hyun commented on issue #26666: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2

2019-11-25 Thread GitBox
dongjoon-hyun commented on issue #2: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2 URL: https://github.com/apache/spark/pull/2#issuecomment-558301021 Thanks you, @srowen . Yes. I'll try to clean up the pom and dependency like this at this time.

[GitHub] [spark] srowen commented on issue #26655: [SPARK-29293][BUILD] Update scalafmt plugin to 1.0.2 for compatibility with Scala 2.13

2019-11-25 Thread GitBox
srowen commented on issue #26655: [SPARK-29293][BUILD] Update scalafmt plugin to 1.0.2 for compatibility with Scala 2.13 URL: https://github.com/apache/spark/pull/26655#issuecomment-558300497 I updated to just: - Move scalafmt under the 2.12 profile - Move it to pluginManagement

[GitHub] [spark] dongjoon-hyun commented on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core`

2019-11-25 Thread GitBox
dongjoon-hyun commented on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core` URL: https://github.com/apache/spark/pull/26658#issuecomment-558300522 Thanks for confirming. Yes. Right. `hadoop-3.2` only works with `Hive 2.3`.

[GitHub] [spark] tgravescs commented on a change in pull request #25299: [SPARK-27651][Core] Avoid the network when shuffle blocks are fetched from the same host

2019-11-25 Thread GitBox
tgravescs commented on a change in pull request #25299: [SPARK-27651][Core] Avoid the network when shuffle blocks are fetched from the same host URL: https://github.com/apache/spark/pull/25299#discussion_r350372735 ## File path:

[GitHub] [spark] tgravescs commented on issue #25299: [SPARK-27651][Core] Avoid the network when shuffle blocks are fetched from the same host

2019-11-25 Thread GitBox
tgravescs commented on issue #25299: [SPARK-27651][Core] Avoid the network when shuffle blocks are fetched from the same host URL: https://github.com/apache/spark/pull/25299#issuecomment-558299984 Definitely agree, sounds like gain and been wanting this for a while so thanks @attilapiros

[GitHub] [spark] AmplabJenkins removed a comment on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558298885 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins removed a comment on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558298881 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558298885 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558298881 Merged build finished. Test PASSed.

[GitHub] [spark] gengliangwang commented on issue #26524: [SPARK-29898][SQL] Support Avro Custom Logical Types

2019-11-25 Thread GitBox
gengliangwang commented on issue #26524: [SPARK-29898][SQL] Support Avro Custom Logical Types URL: https://github.com/apache/spark/pull/26524#issuecomment-558298514 > With this extension you can add your custom logical types and map it to the properly SQL type. For example; read a string

[GitHub] [spark] SparkQA commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
SparkQA commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558298329 **[Test build #114421 has

[GitHub] [spark] liancheng commented on issue #26437: [SPARK-29800][SQL] Plan non-correlated Exists 's subquery in PlanSubqueries

2019-11-25 Thread GitBox
liancheng commented on issue #26437: [SPARK-29800][SQL] Plan non-correlated Exists 's subquery in PlanSubqueries URL: https://github.com/apache/spark/pull/26437#issuecomment-558296560 @AngersZh, I think there's probably some thing wrong with your latest master merge...

[GitHub] [spark] AmplabJenkins removed a comment on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558295959 Test FAILed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins removed a comment on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558295950 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA removed a comment on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
SparkQA removed a comment on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558295163 **[Test build #114420 has

[GitHub] [spark] AmplabJenkins commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558295950 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
SparkQA commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558295940 **[Test build #114420 has

[GitHub] [spark] AmplabJenkins commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558295959 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558295781 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins removed a comment on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558295771 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558295771 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558295781 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
SparkQA commented on issue #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#issuecomment-558295163 **[Test build #114420 has

[GitHub] [spark] shahidki31 commented on a change in pull request #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
shahidki31 commented on a change in pull request #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#discussion_r350366443 ## File path:

[GitHub] [spark] shahidki31 commented on a change in pull request #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
shahidki31 commented on a change in pull request #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#discussion_r350366189 ## File path:

[GitHub] [spark] shahidki31 commented on a change in pull request #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
shahidki31 commented on a change in pull request #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#discussion_r350365849 ## File path:

[GitHub] [spark] shahidki31 commented on a change in pull request #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
shahidki31 commented on a change in pull request #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#discussion_r350365752 ## File path:

[GitHub] [spark] shahidki31 commented on a change in pull request #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI

2019-11-25 Thread GitBox
shahidki31 commented on a change in pull request #26378: [SPARK-29724][SPARK-29726][WEBUI][SQL] Support JDBC/ODBC tab for HistoryServer WebUI URL: https://github.com/apache/spark/pull/26378#discussion_r350365671 ## File path:

[GitHub] [spark] dongjoon-hyun commented on issue #26666: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2

2019-11-25 Thread GitBox
dongjoon-hyun commented on issue #2: [SPARK-29081][CORE][FOLLOWUP] Use commons-lang3 instead of commons-lang2 URL: https://github.com/apache/spark/pull/2#issuecomment-558292404 cc @databricks-david-lewis and @srowen

[GitHub] [spark] dongjoon-hyun commented on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core`

2019-11-25 Thread GitBox
dongjoon-hyun commented on issue #26658: [SPARK-30015][BUILD] Move hive-storage-api dependency from `hive-2.3` to `sql/core` URL: https://github.com/apache/spark/pull/26658#issuecomment-558289134 @tgravescs . After merging this, I'd like to change the following. ``` sql/hive/pom.xml:

[GitHub] [spark] AmplabJenkins removed a comment on issue #26662: [SPARK-30026][SQL] Whitespaces can be identified as delimiters in interval string

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26662: [SPARK-30026][SQL] Whitespaces can be identified as delimiters in interval string URL: https://github.com/apache/spark/pull/26662#issuecomment-558288203 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26662: [SPARK-30026][SQL] Whitespaces can be identified as delimiters in interval string

2019-11-25 Thread GitBox
AmplabJenkins removed a comment on issue #26662: [SPARK-30026][SQL] Whitespaces can be identified as delimiters in interval string URL: https://github.com/apache/spark/pull/26662#issuecomment-558288223 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins commented on issue #26662: [SPARK-30026][SQL] Whitespaces can be identified as delimiters in interval string

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26662: [SPARK-30026][SQL] Whitespaces can be identified as delimiters in interval string URL: https://github.com/apache/spark/pull/26662#issuecomment-558288203 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26662: [SPARK-30026][SQL] Whitespaces can be identified as delimiters in interval string

2019-11-25 Thread GitBox
AmplabJenkins commented on issue #26662: [SPARK-30026][SQL] Whitespaces can be identified as delimiters in interval string URL: https://github.com/apache/spark/pull/26662#issuecomment-558288223 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA removed a comment on issue #26662: [SPARK-30026][SQL] Whitespaces can be identified as delimiters in interval string

2019-11-25 Thread GitBox
SparkQA removed a comment on issue #26662: [SPARK-30026][SQL] Whitespaces can be identified as delimiters in interval string URL: https://github.com/apache/spark/pull/26662#issuecomment-558190748 **[Test build #114410 has

[GitHub] [spark] SparkQA commented on issue #26662: [SPARK-30026][SQL] Whitespaces can be identified as delimiters in interval string

2019-11-25 Thread GitBox
SparkQA commented on issue #26662: [SPARK-30026][SQL] Whitespaces can be identified as delimiters in interval string URL: https://github.com/apache/spark/pull/26662#issuecomment-558287437 **[Test build #114410 has

<    1   2   3   4   5   6   7   8   9   10   >