Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/12831#issuecomment-216593697
Thank you, @srowen .
---
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
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12831
---
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
enabled and wishes so, or if the feature is
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/12831#issuecomment-216502775
Merged to master/2.0
---
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
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/12831#issuecomment-216291461
Thank you for review, @hvanhovell and @srowen .
For `PasswdAuthenticationProvider`, I didn't change that due to the same
reason mentioned by @srowen .
---
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/12831#issuecomment-216223534
I agree with all the changes, though some look like they wouldn't have
actually been lint errors? I assume we can't fix the
`PasswdAuthenticationProvider` file because
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/12831#discussion_r61728740
--- Diff: dev/checkstyle-suppressions.xml ---
@@ -36,4 +36,10 @@
files="src/test/java/org/apache/spark/sql/hive/test/Complex.java"/>
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12831#issuecomment-216123923
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12831#issuecomment-216123922
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12831#issuecomment-216123811
**[Test build #57505 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57505/consoleFull)**
for PR 12831 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12831#issuecomment-216108953
**[Test build #57505 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57505/consoleFull)**
for PR 12831 at commit
Github user dongjoon-hyun commented on the pull request:
https://github.com/apache/spark/pull/12831#issuecomment-216108755
PySpark error is irrelevant to this PR. I'll rebase this to retrigger
Jenkins.
---
If your project is set up for it, you can reply to this email and have your
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12831#issuecomment-216108047
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12831#issuecomment-216108046
Merged build finished. Test FAILed.
---
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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12831#issuecomment-216107981
**[Test build #57503 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/57503/consoleFull)**
for PR 12831 at commit
14 matches
Mail list logo