Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14257
Thank you, @liancheng and @rxin !
---
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
ena
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14257
I'm also going to merge this in branch-2.0 to avoid conflicts in fixing
bugs in this area.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as we
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/14257
LGTM, merging to master. Thanks!
---
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/14257
Hi, @liancheng and @rxin .
It's not ready for review again.
Thank you always!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14257
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62896/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14257
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/14257
**[Test build #62896 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62896/consoleFull)**
for PR 14257 at commit
[`29e953a`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14257
**[Test build #62896 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62896/consoleFull)**
for PR 14257 at commit
[`29e953a`](https://github.com/apache/spark/commit/2
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14257
It was Java internal error.
```
#
# A fatal error has been detected by the Java Runtime Environment:
#
# Internal Error (sharedRuntime.cpp:834), pid=49858, tid=1404775277708
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14257
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/14257
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62893/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14257
**[Test build #62893 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62893/consoleFull)**
for PR 14257 at commit
[`29e953a`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14257
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/14257
**[Test build #62893 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62893/consoleFull)**
for PR 14257 at commit
[`29e953a`](https://github.com/apache/spark/commit/2
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/14257
LGTM except for one minor comment. Thanks!
---
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 featur
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14257
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/14257
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62862/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14257
**[Test build #62862 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62862/consoleFull)**
for PR 14257 at commit
[`16b0f49`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14257
**[Test build #62862 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62862/consoleFull)**
for PR 14257 at commit
[`16b0f49`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14257
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/14257
**[Test build #62829 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62829/consoleFull)**
for PR 14257 at commit
[`6b3adbf`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14257
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62829/
Test PASSed.
---
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14257
cc @liancheng
---
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 th
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14257
**[Test build #62829 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62829/consoleFull)**
for PR 14257 at commit
[`6b3adbf`](https://github.com/apache/spark/commit/6
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14257
Now, the PR is updated with new golden SQL files.
---
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
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14257
I removed the new format stuff. I'll update this PR, soon.
---
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 dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14257
It's just for helping review processes.
---
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 featu
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14257
No problem. We can simply remove `object SQLBuilder` and modify two lines.
May I remove those?
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/14257
I'm less sure about the new format -- it leads to tons of whitespaces for
very long SQL queries.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
29 matches
Mail list logo