Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/18628
Thanks @jiangxb1987 , let me merge it to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18628
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81419/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18628
**[Test build #81419 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81419/testReport)**
for PR 18628 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18628
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18628
**[Test build #81419 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81419/testReport)**
for PR 18628 at commit
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18628
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18628
LGTM pending jenkins
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/18628
@cloud-fan @jiangxb1987, what do you think about this PR, I think it mostly
copies from HS2, and it is quite isolated unless we enabled spnego, so it
should be safe to merge.
---
If your
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/18628
Thanks for making sure this is consistent with other uses of
Configuration.get(); consistency is critical here
---
If your project is set up for it, you can reply to this email and have your
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18628
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18628
**[Test build #80193 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80193/testReport)**
for PR 18628 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18628
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80193/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18628
**[Test build #80193 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80193/testReport)**
for PR 18628 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18628
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79681/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18628
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18628
**[Test build #79681 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79681/testReport)**
for PR 18628 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18628
**[Test build #79681 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79681/testReport)**
for PR 18628 at commit
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/18628
This looks good to me, cc @cmirash @vanzin @steveloughran
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18628
looks good according to the existing similar code, but I'm not very
familiar with this part code of code, will leave it for a few days if someone
else has some comments.
Also cc
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18628
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/79628/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18628
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18628
**[Test build #79628 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79628/testReport)**
for PR 18628 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18628
**[Test build #79628 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/79628/testReport)**
for PR 18628 at commit
23 matches
Mail list logo