[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-12-02 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19839 Will actually open a PR shortly :). --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional command

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-12-02 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19839 @srowen, seems this one causes a build failure for branch-2.1: https://amplab.cs.berkeley.edu/jenkins/job/spark-branch-2.1-test-sbt-hadoop-2.2/641/console https://amplab.cs.berkeley

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-11-30 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/19839 Merged to master/2.2/2.1 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: revie

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-11-30 Thread Victsm
Github user Victsm commented on the issue: https://github.com/apache/spark/pull/19839 @srowen @kiszk @mgaido91 Is this patch ready to merge? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

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

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-11-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19839 Merged build finished. Test PASSed. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional comma

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-11-29 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19839 **[Test build #84322 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84322/testReport)** for PR 19839 at commit [`ede95ae`](https://github.com/apache/spark/commit/e

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-11-29 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19839 **[Test build #84322 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/84322/testReport)** for PR 19839 at commit [`ede95ae`](https://github.com/apache/spark/commit/ed

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-11-29 Thread HyukjinKwon
Github user HyukjinKwon commented on the issue: https://github.com/apache/spark/pull/19839 test this please --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-11-29 Thread kiszk
Github user kiszk commented on the issue: https://github.com/apache/spark/pull/19839 Yea, we would like to use a version that includes [this](https://github.com/janino-compiler/janino/issues/33). However, it is not released yet. SGTM to use the latest released version after r

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-11-29 Thread mgaido91
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/19839 No, it is unreleased. Yes, sure, we can also update it now and later too. --- - To unsubscribe, e-mail: reviews-unsubscr...@spar

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-11-29 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/19839 You'll need to run `./dev/test-dependencies.sh --replace-manifest` --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-11-29 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19839 **[Test build #3997 has finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3997/testReport)** for PR 19839 at commit [`5f54a89`](https://github.com/apache/spark/commit/

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-11-29 Thread SparkQA
Github user SparkQA commented on the issue: https://github.com/apache/spark/pull/19839 **[Test build #3997 has started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3997/testReport)** for PR 19839 at commit [`5f54a89`](https://github.com/apache/spark/commit/5

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-11-29 Thread srowen
Github user srowen commented on the issue: https://github.com/apache/spark/pull/19839 Which version? if it's unreleased, I don't see a harm in updating this now and later too. If it's released, we can just update it here. --- -

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-11-28 Thread mgaido91
Github user mgaido91 commented on the issue: https://github.com/apache/spark/pull/19839 I think we don't need this since we need to upgrade to the next janino release for the issue related to SPARK-18016. --- - To u

[GitHub] spark issue #19839: SPARK-22373 Bump Janino dependency version to fix thread...

2017-11-28 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/19839 Can one of the admins verify this patch? --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional