Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13756
This is part of #14482. Close it now
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
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/13756
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62985/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #62985 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62985/consoleFull)**
for PR 13756 at commit
[`08b5374`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #62985 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62985/consoleFull)**
for PR 13756 at commit
[`08b5374`](https://github.com/apache/spark/commit/0
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13756
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62746/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
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/13756
**[Test build #62746 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62746/consoleFull)**
for PR 13756 at commit
[`08b5374`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #62746 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62746/consoleFull)**
for PR 13756 at commit
[`08b5374`](https://github.com/apache/spark/commit/0
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13756
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
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13756
cc @liancheng @cloud-fan
---
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/13756
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/13756
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62182/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #62182 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62182/consoleFull)**
for PR 13756 at commit
[`08b5374`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #62182 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62182/consoleFull)**
for PR 13756 at commit
[`08b5374`](https://github.com/apache/spark/commit/0
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13756
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61785/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
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/13756
**[Test build #61785 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61785/consoleFull)**
for PR 13756 at commit
[`08b5374`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #61785 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61785/consoleFull)**
for PR 13756 at commit
[`08b5374`](https://github.com/apache/spark/commit/0
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13756
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61770/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
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/13756
**[Test build #61770 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61770/consoleFull)**
for PR 13756 at commit
[`08b5374`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #61770 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61770/consoleFull)**
for PR 13756 at commit
[`08b5374`](https://github.com/apache/spark/commit/0
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13756
ping @liancheng @cloud-fan
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #61634 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61634/consoleFull)**
for PR 13756 at commit
[`53417f1`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #61634 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61634/consoleFull)**
for PR 13756 at commit
[`53417f1`](https://github.com/apache/spark/commit/5
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13756
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
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/13756
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61179/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #61179 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61179/consoleFull)**
for PR 13756 at commit
[`53417f1`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #61179 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61179/consoleFull)**
for PR 13756 at commit
[`53417f1`](https://github.com/apache/spark/commit/5
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13756
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61153/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
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
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #61153 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61153/consoleFull)**
for PR 13756 at commit
[`c0e7e0c`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
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/13756
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61154/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #61154 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61154/consoleFull)**
for PR 13756 at commit
[`53417f1`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #61154 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61154/consoleFull)**
for PR 13756 at commit
[`53417f1`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #61153 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61153/consoleFull)**
for PR 13756 at commit
[`c0e7e0c`](https://github.com/apache/spark/commit/c
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13756
Your suggestion is very good! Let me try it tonight. 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 n
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/13756
I'm thinking about if it's possible to concentrate error checking logics at
one place for table creation. For example, we check duplicated table column
names at parser for SQL
statement(https://g
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61099/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
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/13756
**[Test build #61099 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61099/consoleFull)**
for PR 13756 at commit
[`24edb5f`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #61099 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61099/consoleFull)**
for PR 13756 at commit
[`24edb5f`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/61005/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
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/13756
**[Test build #61005 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61005/consoleFull)**
for PR 13756 at commit
[`ae15ea9`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #61005 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/61005/consoleFull)**
for PR 13756 at commit
[`ae15ea9`](https://github.com/apache/spark/commit/a
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13756
It sounds like `PreWriteCheck` rule is a good home for adding this checking.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If you
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13756
: ) Sharp eye! `blockBy` is the parameter name I used for another Project.
Sorry for the wrong name. I did it more than once.
Let me try to find a common place in Analyzer for this. Than
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/13756
I think it would be better to move these checks to the analyzer, so that
the SQL equivalents of those structures (partitioning and bucketing) can also
benefit from them.
---
If your project is s
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/13756
Do you mean `bucketBy` instead of `blockBy` in the PR title?
---
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/13756
@cloud-fan @yhuai @liancheng @clockfly Could you please review this PR?
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13756
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/13756
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60764/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #60764 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60764/consoleFull)**
for PR 13756 at commit
[`83082ff`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13756
**[Test build #60764 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60764/consoleFull)**
for PR 13756 at commit
[`83082ff`](https://github.com/apache/spark/commit/8
62 matches
Mail list logo