[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

2018-01-29 Thread squito
Github user squito commented on the issue: https://github.com/apache/spark/pull/20399 I am merging this now to master & 2.3 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands,

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

2018-01-29 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20399 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/347/

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

2018-01-29 Thread vanzin
Github user vanzin commented on the issue: https://github.com/apache/spark/pull/20399 I was hoping that one of the other committers who +1'ed the patch would push it instead of me. (Ignoring the info vs. debug discussion.) ---

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

2018-01-29 Thread sameeragarwal
Github user sameeragarwal commented on the issue: https://github.com/apache/spark/pull/20399 @vanzin and reviewers -- is this ready to go? We're waiting on RC3 for this. Thanks! --- - To unsubscribe, e-mail:

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

2018-01-25 Thread jerryshao
Github user jerryshao commented on the issue: https://github.com/apache/spark/pull/20399 LGTM. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

2018-01-25 Thread vanzin
Github user vanzin commented on the issue: https://github.com/apache/spark/pull/20399 Because the code is cleaner that way. --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands,

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

2018-01-25 Thread jerryshao
Github user jerryshao commented on the issue: https://github.com/apache/spark/pull/20399 Originally we were using reflection for this `HiveDelegationTokenProvider`. But in that PR we changed to directly use Hive classes, is there any particular reason? ---

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

2018-01-25 Thread vanzin
Github user vanzin commented on the issue: https://github.com/apache/spark/pull/20399 Some of that could be cleaned up, but more exceptions are being caught there in different method calls, so it still can help. ---

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

2018-01-25 Thread jerryshao
Github user jerryshao commented on the issue: https://github.com/apache/spark/pull/20399 So seems `try-catch` mechanism in `HiveDelegationTokenProvider` is not useful. --- - To unsubscribe, e-mail:

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

2018-01-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20399 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/253/

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

2018-01-25 Thread vanzin
Github user vanzin commented on the issue: https://github.com/apache/spark/pull/20399 @jerryshao --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail:

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

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

[GitHub] spark issue #20399: [SPARK-23209][core] Allow credential manager to work whe...

2018-01-25 Thread AmplabJenkins
Github user AmplabJenkins commented on the issue: https://github.com/apache/spark/pull/20399 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/248/