Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16579
Thank you, @gatorsmile , @srowen , and @viirya .
---
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 t
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16579
Thanks! 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 w
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16579
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/16579
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71824/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16579
**[Test build #71824 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71824/testReport)**
for PR 16579 at commit
[`7879201`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16579
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71821/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16579
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/16579
**[Test build #71824 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71824/testReport)**
for PR 16579 at commit
[`7879201`](https://github.com/apache/spark/commit/78
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16579
The only failure is irrelevant to this PR.
```
[info] - set spark.sql.warehouse.dir *** FAILED *** (5 minutes, 0 seconds)
[info] Timeout of './bin/spark-submit' '--class'
'org.apa
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16579
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 wishe
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16579
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71822/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16579
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/16579
**[Test build #71822 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71822/testReport)**
for PR 16579 at commit
[`7879201`](https://github.com/apache/spark/commit/7
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16579
LGTM pending test
---
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,
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16579
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 and wishes so, or if the featur
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16579
**[Test build #71822 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71822/testReport)**
for PR 16579 at commit
[`7879201`](https://github.com/apache/spark/commit/78
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16579
**[Test build #71821 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71821/testReport)**
for PR 16579 at commit
[`7061cd9`](https://github.com/apache/spark/commit/70
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16579
Thank you, @viirya .
I noticed that `spark.sessionState.conf.clear()` is useless. I removed that.
---
If your project is set up for it, you can reply to this email and have your
reply appe
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16579
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71816/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16579
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/16579
**[Test build #71816 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71816/testReport)**
for PR 16579 at commit
[`387ab59`](https://github.com/apache/spark/commit/3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16579
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/16579
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71815/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16579
**[Test build #71815 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71815/testReport)**
for PR 16579 at commit
[`528b0fd`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16579
**[Test build #71816 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71816/testReport)**
for PR 16579 at commit
[`387ab59`](https://github.com/apache/spark/commit/38
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16579
**[Test build #71815 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71815/testReport)**
for PR 16579 at commit
[`528b0fd`](https://github.com/apache/spark/commit/52
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16579
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71809/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16579
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/16579
**[Test build #71809 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71809/testReport)**
for PR 16579 at commit
[`0214fad`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16579
**[Test build #71809 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71809/testReport)**
for PR 16579 at commit
[`0214fad`](https://github.com/apache/spark/commit/02
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16579
@gatorsmile . I updated with `` and added the test case.
If you can revert the change on `SetCommand.scala`, this test case will
fail.
---
If your project is set up for it, you can reply
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16579
Please try it and then we can check whether it makes sense to do it in the
same test case or not.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16579
Yep!
---
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
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16579
Yes. BTW, to do that, we need to add the test case in `SET -v` unit test.
Is it okay?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16579
Could you just add a test case?
---
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 dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16579
Yes. It was because the `stringConf` with default `null` is in YARN module.
So, when I run a Unit test or `sql` module test. It doesn't meet this.
Yesterday, I reproduced this in my lo
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/16579
Just wondering what is the reason why you did not hit this in your local
environment? Do you know which conf caused this?
---
If your project is set up for it, you can reply to this email and ha
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16579
Thank you for keeping your attention and approval again, @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 proje
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16579
Hi, @gatorsmile .
This is the original PR which has two fixes together now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16579
For `SET -v` without sorting, please refer #16624 , too.
---
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 no
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/16579
Hi, @srowen and @gatorsmile .
Finally, this PR resolved all issues.
Could you review this again?
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16579
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/16579
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71781/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16579
**[Test build #71781 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71781/testReport)**
for PR 16579 at commit
[`fb578d1`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16579
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/16579
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71777/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16579
**[Test build #71777 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/71777/testReport)**
for PR 16579 at commit
[`fa4914b`](https://github.com/apache/spark/commit/f
47 matches
Mail list logo