[GitHub] AmplabJenkins removed a comment on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system.

AmplabJenkins removed a comment on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system. URL: https://github.com/apache/spark/pull/23363#issuecomment-449698051 Merged build finished. Test PASSed.

[GitHub] AmplabJenkins commented on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system.

AmplabJenkins commented on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system. URL: https://github.com/apache/spark/pull/23363#issuecomment-449698052 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:/

[GitHub] AmplabJenkins removed a comment on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system.

AmplabJenkins removed a comment on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system. URL: https://github.com/apache/spark/pull/23363#issuecomment-449698052 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] AmplabJenkins commented on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system.

AmplabJenkins commented on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system. URL: https://github.com/apache/spark/pull/23363#issuecomment-449698051 Merged build finished. Test PASSed.

[GitHub] SparkQA commented on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system.

SparkQA commented on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system. URL: https://github.com/apache/spark/pull/23363#issuecomment-449697895 **[Test build #100415 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/100

[GitHub] SparkQA removed a comment on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system.

SparkQA removed a comment on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system. URL: https://github.com/apache/spark/pull/23363#issuecomment-449682142 **[Test build #100415 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuil

[GitHub] cloud-fan commented on issue #23325: [SPARK-26376][SQL] Skip inputs without tokens by JSON datasource

cloud-fan commented on issue #23325: [SPARK-26376][SQL] Skip inputs without tokens by JSON datasource URL: https://github.com/apache/spark/pull/23325#issuecomment-449696981 It seems also reasonable to accept @HyukjinKwon 's proposal: just revert https://github.com/apache/spark/commit/38628

[GitHub] cloud-fan commented on a change in pull request #23358: [SPARK-26424][SQL] Use java.time API in date/timestamp expressions

cloud-fan commented on a change in pull request #23358: [SPARK-26424][SQL] Use java.time API in date/timestamp expressions URL: https://github.com/apache/spark/pull/23358#discussion_r243794122 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/util/TimestampFormatterSui

[GitHub] cloud-fan commented on a change in pull request #23358: [SPARK-26424][SQL] Use java.time API in date/timestamp expressions

cloud-fan commented on a change in pull request #23358: [SPARK-26424][SQL] Use java.time API in date/timestamp expressions URL: https://github.com/apache/spark/pull/23358#discussion_r243794114 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/util/DateFormatterSuite.sc

[GitHub] cloud-fan commented on a change in pull request #23349: [SPARK-26403][SQL] Support pivoting using array column for `pivot(column)` API

cloud-fan commented on a change in pull request #23349: [SPARK-26403][SQL] Support pivoting using array column for `pivot(column)` API URL: https://github.com/apache/spark/pull/23349#discussion_r243789817 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expre

[GitHub] AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449690201 Merged build finished. Test PASSed. This is an

[GitHub] AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449690201 Merged build finished. Test PASSed. This is an automate

[GitHub] AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449690203 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.b

[GitHub] AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449690203 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.

[GitHub] SparkQA removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

SparkQA removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449686046 **[Test build #100417 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/100417/testRepor

[GitHub] SparkQA commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

SparkQA commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449690140 **[Test build #100417 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/100417/testReport)** f

[GitHub] AmplabJenkins commented on issue #23243: [SPARK-26288][CORE]add initRegisteredExecutorsDB

AmplabJenkins commented on issue #23243: [SPARK-26288][CORE]add initRegisteredExecutorsDB URL: https://github.com/apache/spark/pull/23243#issuecomment-449689956 Merged build finished. Test PASSed. This is an automated message

[GitHub] AmplabJenkins removed a comment on issue #23243: [SPARK-26288][CORE]add initRegisteredExecutorsDB

AmplabJenkins removed a comment on issue #23243: [SPARK-26288][CORE]add initRegisteredExecutorsDB URL: https://github.com/apache/spark/pull/23243#issuecomment-449689959 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.

[GitHub] AmplabJenkins removed a comment on issue #23243: [SPARK-26288][CORE]add initRegisteredExecutorsDB

AmplabJenkins removed a comment on issue #23243: [SPARK-26288][CORE]add initRegisteredExecutorsDB URL: https://github.com/apache/spark/pull/23243#issuecomment-449689956 Merged build finished. Test PASSed. This is an automated

[GitHub] AmplabJenkins commented on issue #23243: [SPARK-26288][CORE]add initRegisteredExecutorsDB

AmplabJenkins commented on issue #23243: [SPARK-26288][CORE]add initRegisteredExecutorsDB URL: https://github.com/apache/spark/pull/23243#issuecomment-449689959 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenk

[GitHub] wangshuo128 commented on issue #23355: [SPARK-26418][SHUFFLE] Only OpenBlocks without any ChunkFetch for one stream will cause memory leak in ExternalShuffleService

wangshuo128 commented on issue #23355: [SPARK-26418][SHUFFLE] Only OpenBlocks without any ChunkFetch for one stream will cause memory leak in ExternalShuffleService URL: https://github.com/apache/spark/pull/23355#issuecomment-449689636 @vanzin Hi, I fixed the code and updated the PR de

[GitHub] SparkQA commented on issue #23243: [SPARK-26288][CORE]add initRegisteredExecutorsDB

SparkQA commented on issue #23243: [SPARK-26288][CORE]add initRegisteredExecutorsDB URL: https://github.com/apache/spark/pull/23243#issuecomment-449689579 **[Test build #100418 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/100418/testReport)** for PR 2324

[GitHub] AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449689444 Merged build finished. Test PASSed. This is an

[GitHub] AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449689446 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.

[GitHub] AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449689446 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.b

[GitHub] SparkQA removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

SparkQA removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449685445 **[Test build #100416 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/100416/testRepor

[GitHub] AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449689444 Merged build finished. Test PASSed. This is an automate

[GitHub] SparkQA commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

SparkQA commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449689382 **[Test build #100416 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/100416/testReport)** f

[GitHub] AmplabJenkins removed a comment on issue #21308: [SPARK-24253][SQL] Add DeleteSupport mix-in for DataSourceV2.

AmplabJenkins removed a comment on issue #21308: [SPARK-24253][SQL] Add DeleteSupport mix-in for DataSourceV2. URL: https://github.com/apache/spark/pull/21308#issuecomment-449687561 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab

[GitHub] AmplabJenkins commented on issue #21308: [SPARK-24253][SQL] Add DeleteSupport mix-in for DataSourceV2.

AmplabJenkins commented on issue #21308: [SPARK-24253][SQL] Add DeleteSupport mix-in for DataSourceV2. URL: https://github.com/apache/spark/pull/21308#issuecomment-449687558 Merged build finished. Test PASSed. This is an auto

[GitHub] AmplabJenkins removed a comment on issue #21308: [SPARK-24253][SQL] Add DeleteSupport mix-in for DataSourceV2.

AmplabJenkins removed a comment on issue #21308: [SPARK-24253][SQL] Add DeleteSupport mix-in for DataSourceV2. URL: https://github.com/apache/spark/pull/21308#issuecomment-449687558 Merged build finished. Test PASSed. This is

[GitHub] AmplabJenkins commented on issue #21308: [SPARK-24253][SQL] Add DeleteSupport mix-in for DataSourceV2.

AmplabJenkins commented on issue #21308: [SPARK-24253][SQL] Add DeleteSupport mix-in for DataSourceV2. URL: https://github.com/apache/spark/pull/21308#issuecomment-449687561 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berk

[GitHub] asfgit closed pull request #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions

asfgit closed pull request #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions URL: https://github.com/apache/spark/pull/23373 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is displayed bel

[GitHub] dongjoon-hyun commented on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions

dongjoon-hyun commented on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions URL: https://github.com/apache/spark/pull/23373#issuecomment-449686910 Merged to master. ---

[GitHub] AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449686084 Merged build finished. Test PASSed. This is an automate

[GitHub] AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449686084 Merged build finished. Test PASSed. This is an

[GitHub] AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449686086 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley

[GitHub] AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449686086 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.

[GitHub] SparkQA commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

SparkQA commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449686046 **[Test build #100417 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/100417/testReport)** fo

[GitHub] HyukjinKwon commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

HyukjinKwon commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449685987 @mengxr, @srowen and @felixcheung, I think I managed to integrate and address all the concerns and comments.

[GitHub] AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449685488 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.

[GitHub] AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449685487 Merged build finished. Test PASSed. This is an automate

[GitHub] AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449685487 Merged build finished. Test PASSed. This is an

[GitHub] AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449685488 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley

[GitHub] SparkQA commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation

SparkQA commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Instrumentation URL: https://github.com/apache/spark/pull/23263#issuecomment-449685445 **[Test build #100416 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/100416/testReport)** fo

[GitHub] AmplabJenkins removed a comment on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions

AmplabJenkins removed a comment on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions URL: https://github.com/apache/spark/pull/23373#issuecomment-449684264 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] AmplabJenkins commented on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions

AmplabJenkins commented on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions URL: https://github.com/apache/spark/pull/23373#issuecomment-449684257 Merged build finished. Test PASSed. -

[GitHub] AmplabJenkins removed a comment on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions

AmplabJenkins removed a comment on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions URL: https://github.com/apache/spark/pull/23373#issuecomment-449684257 Merged build finished. Test PASSed. -

[GitHub] AmplabJenkins commented on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions

AmplabJenkins commented on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions URL: https://github.com/apache/spark/pull/23373#issuecomment-449684264 Test PASSed. Refer to this link for build results (access rights to CI server n

[GitHub] SparkQA commented on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions

SparkQA commented on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions URL: https://github.com/apache/spark/pull/23373#issuecomment-449684171 **[Test build #100413 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPul

[GitHub] SparkQA removed a comment on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions

SparkQA removed a comment on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions URL: https://github.com/apache/spark/pull/23373#issuecomment-449671902 **[Test build #100413 has started](https://amplab.cs.berkeley.edu/jenkins/job/S

[GitHub] AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline

AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline URL: https://github.com/apache/spark/pull/23263#issuecomment-449683721 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley

[GitHub] SparkQA removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline

SparkQA removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline URL: https://github.com/apache/spark/pull/23263#issuecomment-449681524 **[Test build #100414 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/100414/testReport)** f

[GitHub] AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline

AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline URL: https://github.com/apache/spark/pull/23263#issuecomment-449683720 Merged build finished. Test FAILed. This is an automat

[GitHub] SparkQA commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline

SparkQA commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline URL: https://github.com/apache/spark/pull/23263#issuecomment-449683711 **[Test build #100414 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/100414/testReport)** for PR 2

[GitHub] AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline

AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline URL: https://github.com/apache/spark/pull/23263#issuecomment-449683720 Merged build finished. Test FAILed. This is an automated messa

[GitHub] AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline

AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline URL: https://github.com/apache/spark/pull/23263#issuecomment-449683721 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jen

[GitHub] HyukjinKwon commented on a change in pull request #23349: [SPARK-26403][SQL] Support pivoting using array column for `pivot(column)` API

HyukjinKwon commented on a change in pull request #23349: [SPARK-26403][SQL] Support pivoting using array column for `pivot(column)` API URL: https://github.com/apache/spark/pull/23349#discussion_r243784950 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/exp

[GitHub] HyukjinKwon commented on a change in pull request #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system.

HyukjinKwon commented on a change in pull request #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system. URL: https://github.com/apache/spark/pull/23363#discussion_r243784875 ## File path: sql/catalyst/src/main/java/org/apache/spark/sql/catalyst/expre

[GitHub] AmplabJenkins removed a comment on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system.

AmplabJenkins removed a comment on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system. URL: https://github.com/apache/spark/pull/23363#issuecomment-449682171 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] AmplabJenkins commented on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system.

AmplabJenkins commented on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system. URL: https://github.com/apache/spark/pull/23363#issuecomment-449682169 Merged build finished. Test PASSed.

[GitHub] AmplabJenkins removed a comment on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system.

AmplabJenkins removed a comment on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system. URL: https://github.com/apache/spark/pull/23363#issuecomment-449682169 Merged build finished. Test PASSed.

[GitHub] AmplabJenkins commented on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system.

AmplabJenkins commented on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system. URL: https://github.com/apache/spark/pull/23363#issuecomment-449682171 Test PASSed. Refer to this link for build results (access rights to CI server needed): https:

[GitHub] SparkQA commented on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system.

SparkQA commented on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system. URL: https://github.com/apache/spark/pull/23363#issuecomment-449682142 **[Test build #100415 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/1004

[GitHub] AmplabJenkins removed a comment on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system.

AmplabJenkins removed a comment on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system. URL: https://github.com/apache/spark/pull/23363#issuecomment-449437964 Can one of the admins verify this patch? ---

[GitHub] HyukjinKwon commented on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system.

HyukjinKwon commented on issue #23363: [SPARK-26426][SQL]fix ExpresionInfo assert error in windows operation system. URL: https://github.com/apache/spark/pull/23363#issuecomment-449682024 ok to test This is an automated messa

[GitHub] AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline

AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline URL: https://github.com/apache/spark/pull/23263#issuecomment-449681550 Merged build finished. Test PASSed. This is an automat

[GitHub] AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline

AmplabJenkins removed a comment on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline URL: https://github.com/apache/spark/pull/23263#issuecomment-449681551 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkele

[GitHub] SparkQA commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline

SparkQA commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline URL: https://github.com/apache/spark/pull/23263#issuecomment-449681524 **[Test build #100414 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/100414/testReport)** for PR 23

[GitHub] AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline

AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline URL: https://github.com/apache/spark/pull/23263#issuecomment-449681550 Merged build finished. Test PASSed. This is an automated messa

[GitHub] AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline

AmplabJenkins commented on issue #23263: [SPARK-23674][ML] Adds Spark ML Events to Pipeline URL: https://github.com/apache/spark/pull/23263#issuecomment-449681551 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/je

[GitHub] HyukjinKwon commented on issue #23353: [SPARK-26402][SQL] Accessing nested fields with different cases in case insensitive mode

HyukjinKwon commented on issue #23353: [SPARK-26402][SQL] Accessing nested fields with different cases in case insensitive mode URL: https://github.com/apache/spark/pull/23353#issuecomment-449680169 Looks fine to me. This is

[GitHub] HyukjinKwon commented on a change in pull request #23257: [SPARK-26310][SQL] Verify applicability of JSON options

HyukjinKwon commented on a change in pull request #23257: [SPARK-26310][SQL] Verify applicability of JSON options URL: https://github.com/apache/spark/pull/23257#discussion_r243783315 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala ##

[GitHub] HyukjinKwon commented on a change in pull request #23257: [SPARK-26310][SQL] Verify applicability of JSON options

HyukjinKwon commented on a change in pull request #23257: [SPARK-26310][SQL] Verify applicability of JSON options URL: https://github.com/apache/spark/pull/23257#discussion_r243783335 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala ##

[GitHub] HyukjinKwon commented on a change in pull request #23257: [SPARK-26310][SQL] Verify applicability of JSON options

HyukjinKwon commented on a change in pull request #23257: [SPARK-26310][SQL] Verify applicability of JSON options URL: https://github.com/apache/spark/pull/23257#discussion_r243783464 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JSONOptions.scala ##

[GitHub] asfgit closed pull request #23374: [SPARK-26178][SPARK-26243][SQL][FOLLOWUP] Replacing SimpleDateFormat by DateTimeFormatter in comments

asfgit closed pull request #23374: [SPARK-26178][SPARK-26243][SQL][FOLLOWUP] Replacing SimpleDateFormat by DateTimeFormatter in comments URL: https://github.com/apache/spark/pull/23374 This is a PR merged from a forked repository. As GitHub hides the original diff on merge, it is display

[GitHub] HyukjinKwon commented on issue #23374: [SPARK-26178][SPARK-26243][SQL][FOLLOWUP] Replacing SimpleDateFormat by DateTimeFormatter in comments

HyukjinKwon commented on issue #23374: [SPARK-26178][SPARK-26243][SQL][FOLLOWUP] Replacing SimpleDateFormat by DateTimeFormatter in comments URL: https://github.com/apache/spark/pull/23374#issuecomment-449679031 Merged to master. --

[GitHub] AmplabJenkins commented on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions

AmplabJenkins commented on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions URL: https://github.com/apache/spark/pull/23373#issuecomment-449671924 Merged build finished. Test PASSed. -

[GitHub] AmplabJenkins removed a comment on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions

AmplabJenkins removed a comment on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions URL: https://github.com/apache/spark/pull/23373#issuecomment-449671924 Merged build finished. Test PASSed. -

[GitHub] AmplabJenkins removed a comment on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions

AmplabJenkins removed a comment on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions URL: https://github.com/apache/spark/pull/23373#issuecomment-449671927 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] SparkQA commented on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions

SparkQA commented on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions URL: https://github.com/apache/spark/pull/23373#issuecomment-449671902 **[Test build #100413 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPull

[GitHub] AmplabJenkins commented on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions

AmplabJenkins commented on issue #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions URL: https://github.com/apache/spark/pull/23373#issuecomment-449671927 Test PASSed. Refer to this link for build results (access rights to CI server n

[GitHub] srowen commented on a change in pull request #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions

srowen commented on a change in pull request #23373: [SPARK-14023][CORE][SQL] Don't reference 'field' in StructField errors for clarity in exceptions URL: https://github.com/apache/spark/pull/23373#discussion_r243779585 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql

[GitHub] mgaido91 commented on issue #23372: [SPARK-26366][SQL][BACKPORT-2.3] ReplaceExceptWithFilter should consider NULL as False

mgaido91 commented on issue #23372: [SPARK-26366][SQL][BACKPORT-2.3] ReplaceExceptWithFilter should consider NULL as False URL: https://github.com/apache/spark/pull/23372#issuecomment-449669332 thank you @dongjoon-hyun and @gatorsmile --

[GitHub] hvanhovell commented on issue #21055: [SPARK-23693][SQL] Functions to generate UUIDs

hvanhovell commented on issue #21055: [SPARK-23693][SQL] Functions to generate UUIDs URL: https://github.com/apache/spark/pull/21055#issuecomment-449667376 @tashoyan Adding a dependency to remove a 26 LOC (minus header) class is debatable. In this case I wouldn't do it because I don't thin

[GitHub] hvanhovell commented on a change in pull request #21055: [SPARK-23693][SQL] Functions to generate UUIDs

hvanhovell commented on a change in pull request #21055: [SPARK-23693][SQL] Functions to generate UUIDs URL: https://github.com/apache/spark/pull/21055#discussion_r243776731 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/TimeBasedUuid.scala ###

[GitHub] hvanhovell commented on a change in pull request #21055: [SPARK-23693][SQL] Functions to generate UUIDs

hvanhovell commented on a change in pull request #21055: [SPARK-23693][SQL] Functions to generate UUIDs URL: https://github.com/apache/spark/pull/21055#discussion_r243777362 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/RandomBasedUuid.scala #

[GitHub] HeartSaVioR commented on issue #23260: [SPARK-26311][YARN] New feature: custom log URL for stdout/stderr

HeartSaVioR commented on issue #23260: [SPARK-26311][YARN] New feature: custom log URL for stdout/stderr URL: https://github.com/apache/spark/pull/23260#issuecomment-449665436 I'm OK to think about having general solution / different URLs between UI in driver / SHS, except parameters.

[GitHub] hvanhovell commented on a change in pull request #21055: [SPARK-23693][SQL] Functions to generate UUIDs

hvanhovell commented on a change in pull request #21055: [SPARK-23693][SQL] Functions to generate UUIDs URL: https://github.com/apache/spark/pull/21055#discussion_r243776731 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/TimeBasedUuid.scala ###

[GitHub] MaxGekk commented on issue #23257: [SPARK-26310][SQL] Verify applicability of JSON options

MaxGekk commented on issue #23257: [SPARK-26310][SQL] Verify applicability of JSON options URL: https://github.com/apache/spark/pull/23257#issuecomment-449661177 @HyukjinKwon What do you think of this PR? This is an automated

[GitHub] AmplabJenkins removed a comment on issue #23371: [SPARK-26223][SQL] Track metastore operation time in scan node

AmplabJenkins removed a comment on issue #23371: [SPARK-26223][SQL] Track metastore operation time in scan node URL: https://github.com/apache/spark/pull/23371#issuecomment-449644273 Merged build finished. Test PASSed. This i

[GitHub] AmplabJenkins commented on issue #23371: [SPARK-26223][SQL] Track metastore operation time in scan node

AmplabJenkins commented on issue #23371: [SPARK-26223][SQL] Track metastore operation time in scan node URL: https://github.com/apache/spark/pull/23371#issuecomment-449644274 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berk

[GitHub] AmplabJenkins removed a comment on issue #23371: [SPARK-26223][SQL] Track metastore operation time in scan node

AmplabJenkins removed a comment on issue #23371: [SPARK-26223][SQL] Track metastore operation time in scan node URL: https://github.com/apache/spark/pull/23371#issuecomment-449644274 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab

[GitHub] AmplabJenkins commented on issue #23371: [SPARK-26223][SQL] Track metastore operation time in scan node

AmplabJenkins commented on issue #23371: [SPARK-26223][SQL] Track metastore operation time in scan node URL: https://github.com/apache/spark/pull/23371#issuecomment-449644273 Merged build finished. Test PASSed. This is an aut

[GitHub] SparkQA removed a comment on issue #23371: [SPARK-26223][SQL] Track metastore operation time in scan node

SparkQA removed a comment on issue #23371: [SPARK-26223][SQL] Track metastore operation time in scan node URL: https://github.com/apache/spark/pull/23371#issuecomment-449630135 **[Test build #100412 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/100412/test

[GitHub] SparkQA commented on issue #23371: [SPARK-26223][SQL] Track metastore operation time in scan node

SparkQA commented on issue #23371: [SPARK-26223][SQL] Track metastore operation time in scan node URL: https://github.com/apache/spark/pull/23371#issuecomment-449644159 **[Test build #100412 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/100412/testReport)

[GitHub] Ngone51 commented on a change in pull request #23375: [SPARK-26431][CORE] Update availableSlots by availableCpus for Barrier TaskSet

Ngone51 commented on a change in pull request #23375: [SPARK-26431][CORE] Update availableSlots by availableCpus for Barrier TaskSet URL: https://github.com/apache/spark/pull/23375#discussion_r243765522 ## File path: core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.

[GitHub] Ngone51 commented on a change in pull request #23375: [SPARK-26431][CORE] Update availableSlots by availableCpus for Barrier TaskSet

Ngone51 commented on a change in pull request #23375: [SPARK-26431][CORE] Update availableSlots by availableCpus for Barrier TaskSet URL: https://github.com/apache/spark/pull/23375#discussion_r243765368 ## File path: core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.

[GitHub] Ngone51 commented on a change in pull request #23375: [SPARK-26431][CORE] Update availableSlots by availableCpus for Barrier TaskSet

Ngone51 commented on a change in pull request #23375: [SPARK-26431][CORE] Update availableSlots by availableCpus for Barrier TaskSet URL: https://github.com/apache/spark/pull/23375#discussion_r243765368 ## File path: core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerImpl.

[GitHub] jiangxb1987 commented on a change in pull request #23375: [SPARK-26431][CORE] Update availableSlots by availableCpus for Barrier TaskSet

jiangxb1987 commented on a change in pull request #23375: [SPARK-26431][CORE] Update availableSlots by availableCpus for Barrier TaskSet URL: https://github.com/apache/spark/pull/23375#discussion_r243765132 ## File path: core/src/main/scala/org/apache/spark/scheduler/TaskSchedulerI

  1   2   >