Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/13146
Merging to master.
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63618 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63618/consoleFull)**
for PR 13146 at commit
[`1a39dc5`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63632/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63632 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63632/consoleFull)**
for PR 13146 at commit
[`1a39dc5`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63632 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63632/consoleFull)**
for PR 13146 at commit
[`1a39dc5`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
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 project does not have this feature
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63618/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63618 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63618/consoleFull)**
for PR 13146 at commit
[`1a39dc5`](https://github.com/apache/spark/commit/1
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/13146
retest this please
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63570/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63570 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63570/consoleFull)**
for PR 13146 at commit
[`1a39dc5`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63566/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63566 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63566/consoleFull)**
for PR 13146 at commit
[`4fa7bda`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63565/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63565 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63565/consoleFull)**
for PR 13146 at commit
[`a1dc1e1`](https://github.com/apache/spark/commit/
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/13146
Sorry, I don't use intellij.
---
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 wis
Github user zjffdu commented on the issue:
https://github.com/apache/spark/pull/13146
Thanks @vanzin , BTW do you know how to configure intellij to make the code
style works. I already set the indentation as 2 spaces, but still get the wrong
indentation.
---
If your project is set
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63570 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63570/consoleFull)**
for PR 13146 at commit
[`1a39dc5`](https://github.com/apache/spark/commit/1
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/13146
I'll merge once current test run passes; that should give you time to fix
the remaining style issue. :-)
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63566 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63566/consoleFull)**
for PR 13146 at commit
[`4fa7bda`](https://github.com/apache/spark/commit/4
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63565 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63565/consoleFull)**
for PR 13146 at commit
[`a1dc1e1`](https://github.com/apache/spark/commit/a
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/13146
Still a couple of style nits.
---
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 wi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63490/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63490 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63490/consoleFull)**
for PR 13146 at commit
[`551a0cc`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63490 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63490/consoleFull)**
for PR 13146 at commit
[`551a0cc`](https://github.com/apache/spark/commit/5
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/13146
A few minor things, otherwise LGTM.
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63420/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63420 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63420/consoleFull)**
for PR 13146 at commit
[`8119f6d`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63417/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63417 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63417/consoleFull)**
for PR 13146 at commit
[`3826f33`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63420 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63420/consoleFull)**
for PR 13146 at commit
[`8119f6d`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63417 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63417/consoleFull)**
for PR 13146 at commit
[`3826f33`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63406/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63406 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63406/consoleFull)**
for PR 13146 at commit
[`8e1600e`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63406 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63406/consoleFull)**
for PR 13146 at commit
[`8e1600e`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
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 project does not have this feature
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63403 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63403/consoleFull)**
for PR 13146 at commit
[`1e9053a`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63403/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63403 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63403/consoleFull)**
for PR 13146 at commit
[`1e9053a`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/63361/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63361 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63361/consoleFull)**
for PR 13146 at commit
[`0a6a96c`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #63361 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/63361/consoleFull)**
for PR 13146 at commit
[`0a6a96c`](https://github.com/apache/spark/commit/0
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/13146
I think it would be better to have constants defined in
`core/src/main/scala/org/apache/spark/internal/config/package.scala` for the
new configs. That avoids hardcoding the config name in a bunch of
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61015/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13146
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #61015 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61015/consoleFull)**
for PR 13146 at commit
[`5fe4cc6`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13146
**[Test build #61015 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61015/consoleFull)**
for PR 13146 at commit
[`5fe4cc6`](https://github.com/apache/spark/commit/5
56 matches
Mail list logo
