[GitHub] [spark] MaxGekk commented on issue #26256: [SPARK-29605][SQL] Optimize string to interval casting

2019-11-04 Thread GitBox
MaxGekk commented on issue #26256: [SPARK-29605][SQL] Optimize string to interval casting URL: https://github.com/apache/spark/pull/26256#issuecomment-549705876 @cloud-fan Can we continue with this optimization since #26283 has been closed?

[GitHub] [spark] cloud-fan closed pull request #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format

2019-11-04 Thread GitBox
cloud-fan closed pull request #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format URL: https://github.com/apache/spark/pull/26283 This is an automated message from the Apache Git

[GitHub] [spark] cloud-fan commented on issue #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format

2019-11-04 Thread GitBox
cloud-fan commented on issue #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format URL: https://github.com/apache/spark/pull/26283#issuecomment-549701170 This has bigger impacts than I expect, e.g. it breaks `functions.window` which is a

[GitHub] [spark] AmplabJenkins commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549700536 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549700536 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549700532 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549700532 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA removed a comment on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
SparkQA removed a comment on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549682257 **[Test build #113244 has

[GitHub] [spark] SparkQA commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
SparkQA commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549700274 **[Test build #113244 has

[GitHub] [spark] AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549699347 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549699340 Merged build finished. Test FAILed. This is

[GitHub] [spark] SparkQA commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
SparkQA commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549699187 **[Test build #113240 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/113240/testReport)**

[GitHub] [spark] AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549699340 Merged build finished. Test FAILed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549699347 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
SparkQA removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549673269 **[Test build #113240 has

[GitHub] [spark] zhengruifeng edited a comment on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
zhengruifeng edited a comment on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549694050 IIRC, when `Summarizer` was created, I had done similar tests, and at that time

[GitHub] [spark] MaxGekk commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time

2019-11-04 Thread GitBox
MaxGekk commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time URL: https://github.com/apache/spark/pull/26392#issuecomment-549697554 There is a transitive dependency from `joda-time` via hive-common for `hadoop-3.2`: ``` [info]

[GitHub] [spark] AmplabJenkins commented on issue #26360: [SPARK-29714][SQL][TESTS] Port insert.sql

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26360: [SPARK-29714][SQL][TESTS] Port insert.sql URL: https://github.com/apache/spark/pull/26360#issuecomment-549696311 Merged build finished. Test PASSed. This is an automated message from

[GitHub] [spark] AmplabJenkins commented on issue #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust URL: https://github.com/apache/spark/pull/26394#issuecomment-549696352 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust URL: https://github.com/apache/spark/pull/26394#issuecomment-549696356 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26360: [SPARK-29714][SQL][TESTS] Port insert.sql

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26360: [SPARK-29714][SQL][TESTS] Port insert.sql URL: https://github.com/apache/spark/pull/26360#issuecomment-549696311 Merged build finished. Test PASSed. This is an automated

[GitHub] [spark] AmplabJenkins commented on issue #26360: [SPARK-29714][SQL][TESTS] Port insert.sql

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26360: [SPARK-29714][SQL][TESTS] Port insert.sql URL: https://github.com/apache/spark/pull/26360#issuecomment-549696315 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust URL: https://github.com/apache/spark/pull/26394#issuecomment-549696352 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust URL: https://github.com/apache/spark/pull/26394#issuecomment-549696356 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26360: [SPARK-29714][SQL][TESTS] Port insert.sql

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26360: [SPARK-29714][SQL][TESTS] Port insert.sql URL: https://github.com/apache/spark/pull/26360#issuecomment-549696315 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust

2019-11-04 Thread GitBox
SparkQA commented on issue #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust URL: https://github.com/apache/spark/pull/26394#issuecomment-549695881 **[Test build #113247 has

[GitHub] [spark] SparkQA commented on issue #26360: [SPARK-29714][SQL][TESTS] Port insert.sql

2019-11-04 Thread GitBox
SparkQA commented on issue #26360: [SPARK-29714][SQL][TESTS] Port insert.sql URL: https://github.com/apache/spark/pull/26360#issuecomment-549695913 **[Test build #113248 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/113248/testReport)** for PR 26360 at

[GitHub] [spark] cloud-fan commented on a change in pull request #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust

2019-11-04 Thread GitBox
cloud-fan commented on a change in pull request #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust URL: https://github.com/apache/spark/pull/26394#discussion_r342409645 ## File path:

[GitHub] [spark] cloud-fan commented on issue #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust

2019-11-04 Thread GitBox
cloud-fan commented on issue #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust URL: https://github.com/apache/spark/pull/26394#issuecomment-549695040 cc @maryannxue @JkSelf This is an automated

[GitHub] [spark] cloud-fan opened a new pull request #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust

2019-11-04 Thread GitBox
cloud-fan opened a new pull request #26394: [SPARK-29752][SQL][TEST] make AdaptiveQueryExecSuite more robust URL: https://github.com/apache/spark/pull/26394 ### What changes were proposed in this pull request? instead of checking the exact number of local shuffle readers, we

[GitHub] [spark] zhengruifeng commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
zhengruifeng commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549694512 friendly ping @srowen @WeichenXu123

[GitHub] [spark] AmplabJenkins removed a comment on issue #26337: [SPARK-29679][SQL] Make interval type comparable and orderable

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26337: [SPARK-29679][SQL] Make interval type comparable and orderable URL: https://github.com/apache/spark/pull/26337#issuecomment-549694062 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26337: [SPARK-29679][SQL] Make interval type comparable and orderable

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26337: [SPARK-29679][SQL] Make interval type comparable and orderable URL: https://github.com/apache/spark/pull/26337#issuecomment-549694058 Merged build finished. Test PASSed. This

[GitHub] [spark] AmplabJenkins commented on issue #26337: [SPARK-29679][SQL] Make interval type comparable and orderable

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26337: [SPARK-29679][SQL] Make interval type comparable and orderable URL: https://github.com/apache/spark/pull/26337#issuecomment-549694058 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #26337: [SPARK-29679][SQL] Make interval type comparable and orderable

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26337: [SPARK-29679][SQL] Make interval type comparable and orderable URL: https://github.com/apache/spark/pull/26337#issuecomment-549694062 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] zhengruifeng commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
zhengruifeng commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549694050 IIRC, when `Summarizer` was created, I had done similar tests, and at that time `Summarizer` was

[GitHub] [spark] xyxiaoyou commented on issue #17186: [SPARK-19846][SQL] Add a flag to disable constraint propagation

2019-11-04 Thread GitBox
xyxiaoyou commented on issue #17186: [SPARK-19846][SQL] Add a flag to disable constraint propagation URL: https://github.com/apache/spark/pull/17186#issuecomment-549693775 > @xyxiaoyou SQLConf.get ? What I want to express is that I wonder if this issue has been resolved in

[GitHub] [spark] SparkQA commented on issue #26337: [SPARK-29679][SQL] Make interval type comparable and orderable

2019-11-04 Thread GitBox
SparkQA commented on issue #26337: [SPARK-29679][SQL] Make interval type comparable and orderable URL: https://github.com/apache/spark/pull/26337#issuecomment-549693710 **[Test build #113246 has

[GitHub] [spark] dongjoon-hyun commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time

2019-11-04 Thread GitBox
dongjoon-hyun commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time URL: https://github.com/apache/spark/pull/26392#issuecomment-549693133 I pinged you the on-going `Arrow upgrade` PR. Let's resume this PR after that PR.

[GitHub] [spark] MaxGekk commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time

2019-11-04 Thread GitBox
MaxGekk commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time URL: https://github.com/apache/spark/pull/26392#issuecomment-549692184 > Is that the last one? Or, just one dependency instance? I bumped arrow version to 0.15.1 locally, so, there is no

[GitHub] [spark] maropu commented on issue #26360: [SPARK-29714][SQL][TESTS] Port insert.sql

2019-11-04 Thread GitBox
maropu commented on issue #26360: [SPARK-29714][SQL][TESTS] Port insert.sql URL: https://github.com/apache/spark/pull/26360#issuecomment-549691716 > I'm not sure this insert.sql is worthy to add and keep this in Apache Spark repository. Yea, to be honest, I'm also not sure we need

[GitHub] [spark] dongjoon-hyun commented on issue #26133: [WIP][SPARK-29376][SQL][PYTHON] Upgrade Apache Arrow to version 0.15.0

2019-11-04 Thread GitBox
dongjoon-hyun commented on issue #26133: [WIP][SPARK-29376][SQL][PYTHON] Upgrade Apache Arrow to version 0.15.0 URL: https://github.com/apache/spark/pull/26133#issuecomment-549691365 HI, @BryanCutler and @HyukjinKwon . Can we remove `[WIP]`? 

[GitHub] [spark] dongjoon-hyun commented on issue #26133: [WIP][SPARK-29376][SQL][PYTHON] Upgrade Apache Arrow to version 0.15.0

2019-11-04 Thread GitBox
dongjoon-hyun commented on issue #26133: [WIP][SPARK-29376][SQL][PYTHON] Upgrade Apache Arrow to version 0.15.0 URL: https://github.com/apache/spark/pull/26133#issuecomment-549691047 cc @MaxGekk since he is trying to remove `joda-time` with this PR.

[GitHub] [spark] dongjoon-hyun commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time

2019-11-04 Thread GitBox
dongjoon-hyun commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time URL: https://github.com/apache/spark/pull/26392#issuecomment-549690514 Is that the last one? Or, just one dependency instance?

[GitHub] [spark] gatorsmile commented on issue #17186: [SPARK-19846][SQL] Add a flag to disable constraint propagation

2019-11-04 Thread GitBox
gatorsmile commented on issue #17186: [SPARK-19846][SQL] Add a flag to disable constraint propagation URL: https://github.com/apache/spark/pull/17186#issuecomment-549690510 @xyxiaoyou SQLConf.get ? This is an automated

[GitHub] [spark] MaxGekk commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time

2019-11-04 Thread GitBox
MaxGekk commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time URL: https://github.com/apache/spark/pull/26392#issuecomment-549689138 arrow vector requires joda time in 12.x: ``` [info] joda-time:joda-time:2.9.9 [info]

[GitHub] [spark] dongjoon-hyun closed pull request #26223: [WIP][SPARK-29564][SQL] Cluster deploy mode should support Spark Thrift server

2019-11-04 Thread GitBox
dongjoon-hyun closed pull request #26223: [WIP][SPARK-29564][SQL] Cluster deploy mode should support Spark Thrift server URL: https://github.com/apache/spark/pull/26223 This is an automated message from the Apache Git

[GitHub] [spark] dongjoon-hyun commented on issue #26223: [WIP][SPARK-29564][SQL] Cluster deploy mode should support Spark Thrift server

2019-11-04 Thread GitBox
dongjoon-hyun commented on issue #26223: [WIP][SPARK-29564][SQL] Cluster deploy mode should support Spark Thrift server URL: https://github.com/apache/spark/pull/26223#issuecomment-549688495 Hi, @LantaoJin . I also have the same concern like @jerryshao . I'm closing this PR for now.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format URL: https://github.com/apache/spark/pull/26283#issuecomment-549687007 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format URL: https://github.com/apache/spark/pull/26283#issuecomment-549686982 Merged build finished. Test FAILed.

[GitHub] [spark] AmplabJenkins commented on issue #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format URL: https://github.com/apache/spark/pull/26283#issuecomment-549686982 Merged build finished. Test FAILed.

[GitHub] [spark] SparkQA removed a comment on issue #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format

2019-11-04 Thread GitBox
SparkQA removed a comment on issue #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format URL: https://github.com/apache/spark/pull/26283#issuecomment-549664041 **[Test build #113239 has

[GitHub] [spark] AmplabJenkins commented on issue #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format URL: https://github.com/apache/spark/pull/26283#issuecomment-549687007 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] SparkQA commented on issue #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format

2019-11-04 Thread GitBox
SparkQA commented on issue #26283: [SPARK-29622][SQL] do not allow leading 'interval' in the interval string format URL: https://github.com/apache/spark/pull/26283#issuecomment-549686809 **[Test build #113239 has

[GitHub] [spark] dongjoon-hyun commented on issue #25573: [SPARK-28833][DOCS][SQL] Document ALTER VIEW command

2019-11-04 Thread GitBox
dongjoon-hyun commented on issue #25573: [SPARK-28833][DOCS][SQL] Document ALTER VIEW command URL: https://github.com/apache/spark/pull/25573#issuecomment-549686884 Gentle ping. This is an automated message from the Apache

[GitHub] [spark] xyxiaoyou commented on issue #17186: [SPARK-19846][SQL] Add a flag to disable constraint propagation

2019-11-04 Thread GitBox
xyxiaoyou commented on issue #17186: [SPARK-19846][SQL] Add a flag to disable constraint propagation URL: https://github.com/apache/spark/pull/17186#issuecomment-549686880 May I ask how to solve this issue in spark2.3? I see that this flag has been removed in PruneFilters and

[GitHub] [spark] dongjoon-hyun commented on issue #21027: [SPARK-23943][MESOS][DEPLOY] Improve observability of MesosRestServer/MesosClusterDi…

2019-11-04 Thread GitBox
dongjoon-hyun commented on issue #21027: [SPARK-23943][MESOS][DEPLOY] Improve observability of MesosRestServer/MesosClusterDi… URL: https://github.com/apache/spark/pull/21027#issuecomment-549686589 Hi, All. I'm closing this PR due to author's inactivity. Please feel free to reopen

[GitHub] [spark] MaxGekk commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time

2019-11-04 Thread GitBox
MaxGekk commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time URL: https://github.com/apache/spark/pull/26392#issuecomment-549686541 > Sorry, but I'm -1 to downgrade like this. @dongjoon-hyun You are so quick.

[GitHub] [spark] dongjoon-hyun closed pull request #21027: [SPARK-23943][MESOS][DEPLOY] Improve observability of MesosRestServer/MesosClusterDi…

2019-11-04 Thread GitBox
dongjoon-hyun closed pull request #21027: [SPARK-23943][MESOS][DEPLOY] Improve observability of MesosRestServer/MesosClusterDi… URL: https://github.com/apache/spark/pull/21027 This is an automated message from the Apache

[GitHub] [spark] dongjoon-hyun commented on issue #25898: [SPARK-29209][BUILD] Print build environment variables to Github

2019-11-04 Thread GitBox
dongjoon-hyun commented on issue #25898: [SPARK-29209][BUILD] Print build environment variables to Github URL: https://github.com/apache/spark/pull/25898#issuecomment-549685906 Gentle ping, @wangyum . This is an automated

[GitHub] [spark] AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549684968 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549684961 Merged build finished. Test PASSed. This is

[GitHub] [spark] AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549684968 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549684961 Merged build finished. Test PASSed. This is an

[GitHub] [spark] SparkQA commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
SparkQA commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549684586 **[Test build #113245 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/113245/testReport)**

[GitHub] [spark] dongjoon-hyun commented on issue #26360: [SPARK-29714][SQL][TESTS] Port insert.sql

2019-11-04 Thread GitBox
dongjoon-hyun commented on issue #26360: [SPARK-29714][SQL][TESTS] Port insert.sql URL: https://github.com/apache/spark/pull/26360#issuecomment-549684325 Actually, only 3 original queries works. I'm not sure this `insert.sql` is worthy to add and keep this in Apache Spark repository.

[GitHub] [spark] jiangxb1987 commented on issue #26078: [SPARK-29151][CORE] Support fractional resources for task resource scheduling

2019-11-04 Thread GitBox
jiangxb1987 commented on issue #26078: [SPARK-29151][CORE] Support fractional resources for task resource scheduling URL: https://github.com/apache/spark/pull/26078#issuecomment-549683186 LGTM This is an automated message

[GitHub] [spark] AmplabJenkins removed a comment on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549682706 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins removed a comment on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549682712 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] zhengruifeng edited a comment on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
zhengruifeng edited a comment on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549682239 test code ```scala import org.apache.spark.ml.feature._ scala> var df =

[GitHub] [spark] AmplabJenkins commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549682706 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549682712 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] jiangxb1987 commented on a change in pull request #26078: [SPARK-29151][CORE] Support fractional resources for task resource scheduling

2019-11-04 Thread GitBox
jiangxb1987 commented on a change in pull request #26078: [SPARK-29151][CORE] Support fractional resources for task resource scheduling URL: https://github.com/apache/spark/pull/26078#discussion_r342398775 ## File path:

[GitHub] [spark] SparkQA commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
SparkQA commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549682257 **[Test build #113244 has

[GitHub] [spark] zhengruifeng commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
zhengruifeng commented on issue #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393#issuecomment-549682239 test code ```scala import org.apache.spark.ml.feature._ scala> var df =

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #26360: [SPARK-29714][SQL][TESTS] Port insert.sql

2019-11-04 Thread GitBox
dongjoon-hyun commented on a change in pull request #26360: [SPARK-29714][SQL][TESTS] Port insert.sql URL: https://github.com/apache/spark/pull/26360#discussion_r342397723 ## File path: sql/core/src/test/resources/sql-tests/inputs/postgreSQL/insert.sql ## @@ -0,0 +1,652

[GitHub] [spark] AmplabJenkins removed a comment on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time URL: https://github.com/apache/spark/pull/26392#issuecomment-549680788 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins removed a comment on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time URL: https://github.com/apache/spark/pull/26392#issuecomment-549680792 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time URL: https://github.com/apache/spark/pull/26392#issuecomment-549680788 Merged build finished. Test PASSed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time URL: https://github.com/apache/spark/pull/26392#issuecomment-549680792 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] zhengruifeng opened a new pull request #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer

2019-11-04 Thread GitBox
zhengruifeng opened a new pull request #26393: [SPARK-29751][ML] Scalers use Summarizer instead of MultivariateOnlineSummarizer URL: https://github.com/apache/spark/pull/26393 ### What changes were proposed in this pull request? use `ml.Summarizer` instead of

[GitHub] [spark] SparkQA commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time

2019-11-04 Thread GitBox
SparkQA commented on issue #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time URL: https://github.com/apache/spark/pull/26392#issuecomment-549680472 **[Test build #113243 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/113243/testReport)**

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #26360: [SPARK-29714][SQL][TESTS] Port insert.sql

2019-11-04 Thread GitBox
dongjoon-hyun commented on a change in pull request #26360: [SPARK-29714][SQL][TESTS] Port insert.sql URL: https://github.com/apache/spark/pull/26360#discussion_r342397024 ## File path: sql/core/src/test/resources/sql-tests/inputs/postgreSQL/insert.sql ## @@ -0,0 +1,652

[GitHub] [spark] AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549679718 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549679693 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549679688 Merged build finished. Test FAILed. This is

[GitHub] [spark] AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549679714 Merged build finished. Test FAILed. This is

[GitHub] [spark] SparkQA removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
SparkQA removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549659899 **[Test build #113238 has

[GitHub] [spark] AmplabJenkins commented on issue #26374: [SPARK-29695][SQL] ALTER TABLE (SerDe properties) should look up catalog/table like v2 commands

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26374: [SPARK-29695][SQL] ALTER TABLE (SerDe properties) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26374#issuecomment-549679803 Test PASSed. Refer to this link for build results (access rights to CI server

[GitHub] [spark] AmplabJenkins removed a comment on issue #26374: [SPARK-29695][SQL] ALTER TABLE (SerDe properties) should look up catalog/table like v2 commands

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26374: [SPARK-29695][SQL] ALTER TABLE (SerDe properties) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26374#issuecomment-549679803 Test PASSed. Refer to this link for build results (access rights to CI

[GitHub] [spark] SparkQA removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
SparkQA removed a comment on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549658568 **[Test build #113237 has

[GitHub] [spark] dongjoon-hyun commented on a change in pull request #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time

2019-11-04 Thread GitBox
dongjoon-hyun commented on a change in pull request #26392: [WIP][SPARK-29750][BUILD] Avoid dependency from joda-time URL: https://github.com/apache/spark/pull/26392#discussion_r342396376 ## File path: dev/deps/spark-deps-hadoop-3.2 ## @@ -139,6 +139,7 @@

[GitHub] [spark] AmplabJenkins removed a comment on issue #26374: [SPARK-29695][SQL] ALTER TABLE (SerDe properties) should look up catalog/table like v2 commands

2019-11-04 Thread GitBox
AmplabJenkins removed a comment on issue #26374: [SPARK-29695][SQL] ALTER TABLE (SerDe properties) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26374#issuecomment-549679802 Merged build finished. Test PASSed.

[GitHub] [spark] AmplabJenkins commented on issue #26374: [SPARK-29695][SQL] ALTER TABLE (SerDe properties) should look up catalog/table like v2 commands

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26374: [SPARK-29695][SQL] ALTER TABLE (SerDe properties) should look up catalog/table like v2 commands URL: https://github.com/apache/spark/pull/26374#issuecomment-549679802 Merged build finished. Test PASSed.

[GitHub] [spark] SparkQA commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
SparkQA commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549679592 **[Test build #113237 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/113237/testReport)**

[GitHub] [spark] AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549679693 Test FAILed. Refer to this link for build results (access rights to CI server needed):

[GitHub] [spark] AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549679714 Merged build finished. Test FAILed. This is an

[GitHub] [spark] AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549679688 Merged build finished. Test FAILed. This is an

[GitHub] [spark] SparkQA commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
SparkQA commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549679570 **[Test build #113238 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/113238/testReport)**

[GitHub] [spark] AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax

2019-11-04 Thread GitBox
AmplabJenkins commented on issue #26338: [SPARK-29680][SQL] Remove ALTER TABLE CHANGE COLUMN syntax URL: https://github.com/apache/spark/pull/26338#issuecomment-549679718 Test FAILed. Refer to this link for build results (access rights to CI server needed):

  1   2   3   4   5   6   7   8   9   >