[GitHub] spark issue #20636: [SPARK-23415][SQL][TEST] Make behavior of BufferHolderSp...

2018-06-16 Thread kiszk
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/20636 retest this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #20636: [SPARK-23415][SQL][TEST] Make behavior of BufferHolderSp...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20636 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20636: [SPARK-23415][SQL][TEST] Make behavior of BufferHolderSp...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20636 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91969/ Test FAILed. ---

[GitHub] spark issue #21551: Fix issue in 'docker-image-tool.sh'

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21551 **[Test build #91967 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91967/testReport)** for PR 21551 at commit

[GitHub] spark issue #21551: Fix issue in 'docker-image-tool.sh'

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21551 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21551: Fix issue in 'docker-image-tool.sh'

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21551 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91967/ Test PASSed. ---

[GitHub] spark issue #21441: [DO-NOT-MERGE] Run tests against hadoop-3.1 to see the t...

2018-06-16 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/21441 retest this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #21441: [DO-NOT-MERGE] Run tests against hadoop-3.1 to see the t...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21441 **[Test build #91973 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91973/testReport)** for PR 21441 at commit

[GitHub] spark issue #21564: [SPARK-24556][SQL] Always rewrite output partitioning in...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21564 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91970/ Test PASSed. ---

[GitHub] spark issue #21564: [SPARK-24556][SQL] Always rewrite output partitioning in...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21564 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark pull request #21564: [SPARK-24556][SQL] Always rewrite output partitio...

2018-06-16 Thread mgaido91
Github user mgaido91 commented on a diff in the pull request: https://github.com/apache/spark/pull/21564#discussion_r195898092 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala --- @@ -686,6 +686,68 @@ class PlannerSuite extends SharedSQLContext {

[GitHub] spark pull request #21564: [SPARK-24556][SQL] Always rewrite output partitio...

2018-06-16 Thread mgaido91
Github user mgaido91 commented on a diff in the pull request: https://github.com/apache/spark/pull/21564#discussion_r195898107 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala --- @@ -686,6 +686,68 @@ class PlannerSuite extends SharedSQLContext {

[GitHub] spark issue #21502: [SPARK-22575][SQL] Add destroy to Dataset

2018-06-16 Thread mgaido91
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/21502 Well, it happens especially with dynamic allocation, but there may be other causes like YARN preemption. Anytime a container is killed we can face this issue. Anyway, I plan to check the

[GitHub] spark pull request #21564: [SPARK-24556][SQL] Always rewrite output partitio...

2018-06-16 Thread yucai
Github user yucai commented on a diff in the pull request: https://github.com/apache/spark/pull/21564#discussion_r195900959 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala --- @@ -686,6 +686,68 @@ class PlannerSuite extends SharedSQLContext {

[GitHub] spark issue #21567: [SPARK-24560][CORE][MESOS] Fix some getTimeAsMs as getTi...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21567 **[Test build #91964 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91964/testReport)** for PR 21567 at commit

[GitHub] spark issue #21572: [SPARK-24534][K8S] Bypass non spark-on-k8s commands

2018-06-16 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/21572 Jenkins, test this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands,

[GitHub] spark issue #21502: [SPARK-22575][SQL] Add destroy to Dataset

2018-06-16 Thread mgaido91
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/21502 @maropu I think a monitor thread would be useless. Once the container is gone, there is nothing we can do. Another solution which may be worth to investigate is to clear the block manager for an

[GitHub] spark pull request #21564: [SPARK-24556][SQL] Always rewrite output partitio...

2018-06-16 Thread mgaido91
Github user mgaido91 commented on a diff in the pull request: https://github.com/apache/spark/pull/21564#discussion_r195901183 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala --- @@ -686,6 +686,68 @@ class PlannerSuite extends SharedSQLContext {

[GitHub] spark issue #21564: [SPARK-24556][SQL] Always rewrite output partitioning in...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21564 **[Test build #91970 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91970/testReport)** for PR 21564 at commit

[GitHub] spark issue #21555: [SPARK-24547] Allow for building spark on k8s docker ima...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21555 **[Test build #91966 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91966/testReport)** for PR 21555 at commit

[GitHub] spark issue #21551: Fix issue in 'docker-image-tool.sh'

2018-06-16 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/21551 ouch --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #21551: Fix issue in 'docker-image-tool.sh'

2018-06-16 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/21551 ok to test --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #21567: [SPARK-24560][CORE][MESOS] Fix some getTimeAsMs as getTi...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21567 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21567: [SPARK-24560][CORE][MESOS] Fix some getTimeAsMs as getTi...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21567 **[Test build #91971 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91971/testReport)** for PR 21567 at commit

[GitHub] spark issue #21567: [SPARK-24560][CORE][MESOS] Fix some getTimeAsMs as getTi...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21567 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91971/ Test FAILed. ---

[GitHub] spark pull request #21068: [SPARK-16630][YARN] Blacklist a node if executors...

2018-06-16 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/21068#discussion_r195896638 --- Diff: resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala --- @@ -0,0 +1,187 @@ +/* + *

[GitHub] spark pull request #21068: [SPARK-16630][YARN] Blacklist a node if executors...

2018-06-16 Thread felixcheung
Github user felixcheung commented on a diff in the pull request: https://github.com/apache/spark/pull/21068#discussion_r195896675 --- Diff: resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/YarnAllocatorBlacklistTracker.scala --- @@ -0,0 +1,187 @@ +/* + *

[GitHub] spark issue #21572: [SPARK-24534][K8S] Bypass non spark-on-k8s commands

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21572 Kubernetes integration test starting URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/234/ ---

[GitHub] spark issue #21572: [SPARK-24534][K8S] Bypass non spark-on-k8s commands

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21572 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/4129/

[GitHub] spark issue #21572: [SPARK-24534][K8S] Bypass non spark-on-k8s commands

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21572 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20636: [SPARK-23415][SQL][TEST] Make behavior of BufferHolderSp...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20636 **[Test build #91969 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91969/testReport)** for PR 20636 at commit

[GitHub] spark issue #21564: [SPARK-24556][SQL] Always rewrite output partitioning in...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21564 **[Test build #91970 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91970/testReport)** for PR 21564 at commit

[GitHub] spark pull request #21564: [SPARK-24556][SQL] Always rewrite output partitio...

2018-06-16 Thread yucai
Github user yucai commented on a diff in the pull request: https://github.com/apache/spark/pull/21564#discussion_r195898606 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala --- @@ -686,6 +686,68 @@ class PlannerSuite extends SharedSQLContext {

[GitHub] spark issue #21564: [SPARK-24556][SQL] Always rewrite output partitioning in...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21564 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91968/ Test PASSed. ---

[GitHub] spark issue #21564: [SPARK-24556][SQL] Always rewrite output partitioning in...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21564 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21564: [SPARK-24556][SQL] Always rewrite output partitioning in...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21564 **[Test build #91968 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91968/testReport)** for PR 21564 at commit

[GitHub] spark issue #21441: [DO-NOT-MERGE] Run tests against hadoop-3.1 to see the t...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21441 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/4131/

[GitHub] spark issue #21441: [DO-NOT-MERGE] Run tests against hadoop-3.1 to see the t...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21441 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21441: [DO-NOT-MERGE] Run tests against hadoop-3.1 to see the t...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21441 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21441: [DO-NOT-MERGE] Run tests against hadoop-3.1 to see the t...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21441 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/236/

[GitHub] spark issue #21568: [SPARK-24562][TESTS] Support different configs for same ...

2018-06-16 Thread mgaido91
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/21568 @rxin the PR does what was suggested in these 2 comments https://github.com/apache/spark/pull/20023#issuecomment-358306751 and https://github.com/apache/spark/pull/21529#issuecomment-396413144.

[GitHub] spark issue #21572: [SPARK-24534][K8S] Bypass non spark-on-k8s commands

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21572 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/234/

[GitHub] spark issue #21572: [SPARK-24534][K8S] Bypass non spark-on-k8s commands

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21572 Kubernetes integration test status failure URL: https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/234/ ---

[GitHub] spark pull request #21462: [SPARK-24428][K8S] Fix unused code

2018-06-16 Thread ifilonenko
Github user ifilonenko commented on a diff in the pull request: https://github.com/apache/spark/pull/21462#discussion_r195900804 --- Diff: resource-managers/kubernetes/docker/src/main/dockerfiles/spark/entrypoint.sh --- @@ -45,12 +45,10 @@ shift 1

[GitHub] spark issue #21572: [SPARK-24534][K8S] Bypass non spark-on-k8s commands

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21572 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21572: [SPARK-24534][K8S] Bypass non spark-on-k8s commands

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21572 **[Test build #91965 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91965/testReport)** for PR 21572 at commit

[GitHub] spark issue #21572: [SPARK-24534][K8S] Bypass non spark-on-k8s commands

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21572 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91965/ Test PASSed. ---

[GitHub] spark issue #21572: [SPARK-24534][K8S] Bypass non spark-on-k8s commands

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21572 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21502: [SPARK-22575][SQL] Add destroy to Dataset

2018-06-16 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21502 IMO I like that approach. If this issue happens only in dynamic allocation, how about adding a new option to turn off/on that checking? ---

[GitHub] spark issue #18900: [SPARK-21687][SQL] Spark SQL should set createTime for H...

2018-06-16 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/18900 Any update? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark pull request #18900: [SPARK-21687][SQL] Spark SQL should set createTim...

2018-06-16 Thread maropu
Github user maropu commented on a diff in the pull request: https://github.com/apache/spark/pull/18900#discussion_r195898963 --- Diff: sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala --- @@ -1019,6 +1021,8 @@ private[hive] object HiveClientImpl {

[GitHub] spark issue #21567: [SPARK-24560][CORE][MESOS] Fix some getTimeAsMs as getTi...

2018-06-16 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21567 retest this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #21567: [SPARK-24560][CORE][MESOS] Fix some getTimeAsMs as getTi...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21567 **[Test build #91971 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91971/testReport)** for PR 21567 at commit

[GitHub] spark issue #21502: [SPARK-22575][SQL] Add destroy to Dataset

2018-06-16 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21502 aha, thanks for the kindly explanation! --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21567: [SPARK-24560][CORE][MESOS] Fix some getTimeAsMs as getTi...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21567 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91964/ Test FAILed. ---

[GitHub] spark issue #21578: [SPARK-24571][SQL] Support Char literals

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21578 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91961/ Test FAILed. ---

[GitHub] spark issue #21555: [SPARK-24547] Allow for building spark on k8s docker ima...

2018-06-16 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/21555 can you edit the title to add `[k8s]` and fix the text which is truncated? --- - To unsubscribe, e-mail:

[GitHub] spark issue #21572: [SPARK-24534][K8S] Bypass non spark-on-k8s commands

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21572 **[Test build #91965 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91965/testReport)** for PR 21572 at commit

[GitHub] spark issue #21555: [SPARK-24547] Allow for building spark on k8s docker ima...

2018-06-16 Thread felixcheung
Github user felixcheung commented on the issue: https://github.com/apache/spark/pull/21555 ok to test --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #21578: [SPARK-24571][SQL] Support Char literals

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21578 **[Test build #91961 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91961/testReport)** for PR 21578 at commit

[GitHub] spark issue #21567: [SPARK-24560][CORE][MESOS] Fix some getTimeAsMs as getTi...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21567 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21578: [SPARK-24571][SQL] Support Char literals

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21578 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21564: [SPARK-24556][SQL] Always rewrite output partitioning in...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21564 **[Test build #91968 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91968/testReport)** for PR 21564 at commit

[GitHub] spark issue #20636: [SPARK-23415][SQL][TEST] Make behavior of BufferHolderSp...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20636 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20636: [SPARK-23415][SQL][TEST] Make behavior of BufferHolderSp...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20636 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/4130/

[GitHub] spark issue #20636: [SPARK-23415][SQL][TEST] Make behavior of BufferHolderSp...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20636 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/235/

[GitHub] spark issue #20636: [SPARK-23415][SQL][TEST] Make behavior of BufferHolderSp...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20636 **[Test build #91969 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91969/testReport)** for PR 20636 at commit

[GitHub] spark issue #20636: [SPARK-23415][SQL][TEST] Make behavior of BufferHolderSp...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20636 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark pull request #21564: [SPARK-24556][SQL] Always rewrite output partitio...

2018-06-16 Thread yucai
Github user yucai commented on a diff in the pull request: https://github.com/apache/spark/pull/21564#discussion_r195898387 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala --- @@ -686,6 +686,68 @@ class PlannerSuite extends SharedSQLContext {

[GitHub] spark pull request #21564: [SPARK-24556][SQL] Always rewrite output partitio...

2018-06-16 Thread mgaido91
Github user mgaido91 commented on a diff in the pull request: https://github.com/apache/spark/pull/21564#discussion_r195898686 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala --- @@ -686,6 +686,68 @@ class PlannerSuite extends SharedSQLContext {

[GitHub] spark issue #21568: [SPARK-24562][TESTS] Support different configs for same ...

2018-06-16 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21568 We need to strictly handle the second case, too? If we accepted the same output files for that case, we could have simpler output file name rules as I described

[GitHub] spark issue #21568: [SPARK-24562][TESTS] Support different configs for same ...

2018-06-16 Thread mgaido91
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/21568 Sorry, @maropu, I didn't get want you mean by > If we accepted the same output files for that case may you please explain me? Anyway, the problem with that proposal is not

[GitHub] spark pull request #21462: [SPARK-24428][K8S] Fix unused code

2018-06-16 Thread ifilonenko
Github user ifilonenko commented on a diff in the pull request: https://github.com/apache/spark/pull/21462#discussion_r195900749 --- Diff: resource-managers/kubernetes/docker/src/main/dockerfiles/spark/entrypoint.sh --- @@ -45,12 +45,10 @@ shift 1

[GitHub] spark issue #21555: [SPARK-24547] Allow for building spark on k8s docker ima...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21555 **[Test build #91966 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91966/testReport)** for PR 21555 at commit

[GitHub] spark issue #21555: [SPARK-24547] Allow for building spark on k8s docker ima...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21555 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91966/ Test PASSed. ---

[GitHub] spark issue #21555: [SPARK-24547] Allow for building spark on k8s docker ima...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21555 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark pull request #21564: [SPARK-24556][SQL] Always rewrite output partitio...

2018-06-16 Thread yucai
Github user yucai commented on a diff in the pull request: https://github.com/apache/spark/pull/21564#discussion_r195901356 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala --- @@ -686,6 +686,68 @@ class PlannerSuite extends SharedSQLContext {

[GitHub] spark pull request #21564: [SPARK-24556][SQL] Always rewrite output partitio...

2018-06-16 Thread mgaido91
Github user mgaido91 commented on a diff in the pull request: https://github.com/apache/spark/pull/21564#discussion_r195901400 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala --- @@ -686,6 +686,68 @@ class PlannerSuite extends SharedSQLContext {

[GitHub] spark issue #21564: [SPARK-24556][SQL] Always rewrite output partitioning in...

2018-06-16 Thread mgaido91
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/21564 LGTM --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #21564: [SPARK-24556][SQL] Always rewrite output partitioning in...

2018-06-16 Thread viirya
Github user viirya commented on the issue: https://github.com/apache/spark/pull/21564 Thanks for fixing this! LGTM --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #21551: Fix issue in 'docker-image-tool.sh'

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21551 **[Test build #91967 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91967/testReport)** for PR 21551 at commit

[GitHub] spark pull request #21564: [SPARK-24556][SQL] Always rewrite output partitio...

2018-06-16 Thread viirya
Github user viirya commented on a diff in the pull request: https://github.com/apache/spark/pull/21564#discussion_r195897400 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala --- @@ -686,6 +686,68 @@ class PlannerSuite extends SharedSQLContext {

[GitHub] spark issue #21502: [SPARK-22575][SQL] Add destroy to Dataset

2018-06-16 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21502 Is it the best to add a new api for that? Is that a bad idea to invoke a monitor thread to check that kind of things? --- - To

[GitHub] spark issue #21447: [SPARK-24339][SQL]Add project for transform/map/reduce s...

2018-06-16 Thread maropu
Github user maropu commented on the issue: https://github.com/apache/spark/pull/21447 kindly pining @gatorsmile @HyukjinKwon @ueshin --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21564: [SPARK-24556][SQL] Always rewrite output partitioning in...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21564 **[Test build #91972 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91972/testReport)** for PR 21564 at commit

[GitHub] spark issue #20146: [SPARK-11215][ML] Add multiple columns support to String...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/20146 **[Test build #91976 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91976/testReport)** for PR 20146 at commit

[GitHub] spark issue #20146: [SPARK-11215][ML] Add multiple columns support to String...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20146 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #20146: [SPARK-11215][ML] Add multiple columns support to String...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20146 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91976/ Test PASSed. ---

[GitHub] spark issue #21441: [DO-NOT-MERGE] Run tests against hadoop-3.1 to see the t...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21441 **[Test build #91973 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91973/testReport)** for PR 21441 at commit

[GitHub] spark issue #21441: [DO-NOT-MERGE] Run tests against hadoop-3.1 to see the t...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21441 Merged build finished. Test FAILed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21441: [DO-NOT-MERGE] Run tests against hadoop-3.1 to see the t...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21441 Test FAILed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91973/ Test FAILed. ---

[GitHub] spark issue #21576: [SPARK-24548][SQL] Fix incorrect schema of Dataset with ...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21576 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/4133/

[GitHub] spark issue #21576: [SPARK-24548][SQL] Fix incorrect schema of Dataset with ...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21576 Test PASSed. Refer to this link for build results (access rights to CI server needed): https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/238/

[GitHub] spark issue #21576: [SPARK-24548][SQL] Fix incorrect schema of Dataset with ...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21576 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21576: [SPARK-24548][SQL] Fix incorrect schema of Dataset with ...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21576 **[Test build #91977 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91977/testReport)** for PR 21576 at commit

[GitHub] spark issue #21576: [SPARK-24548][SQL] Fix incorrect schema of Dataset with ...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/21576 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

[GitHub] spark issue #21564: [SPARK-24556][SQL] Always rewrite output partitioning in...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21564 **[Test build #91972 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91972/testReport)** for PR 21564 at commit

[GitHub] spark issue #20146: [SPARK-11215][ML] Add multiple columns support to String...

2018-06-16 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/20146 retest this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #21534: [SPARK-24526][build] Spaces in the build dir causes fail...

2018-06-16 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/21534 **[Test build #91975 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91975/testReport)** for PR 21534 at commit

[GitHub] spark issue #20146: [SPARK-11215][ML] Add multiple columns support to String...

2018-06-16 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20146 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional

  1   2   3   >