Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/18918
gentle ping @heary-cao
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: re
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/18918
what exactly are you proposing?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/18918
cc @cloud-fan @gatorsmile @viirya I update the description of this PR,
Could your take a look?
---
If your project is set up for it, you can reply to this email and have your
reply appear on Git
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
Can one of the admins verify this patch?
---
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 feat
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/18918
cc @gatorsmile
Spark unit tests fails twice. this is because the new unit test throws an
exception:
`java.lang.IllegalArgumentException: requirement failed: Nondeterministic
expression
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80693/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
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/18918
**[Test build #80693 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80693/testReport)**
for PR 18918 at commit
[`97a3270`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18918
**[Test build #80693 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80693/testReport)**
for PR 18918 at commit
[`97a3270`](https://github.com/apache/spark/commit/97
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80686/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18918
**[Test build #80686 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80686/testReport)**
for PR 18918 at commit
[`df7ecaa`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
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/18918
**[Test build #80686 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80686/testReport)**
for PR 18918 at commit
[`df7ecaa`](https://github.com/apache/spark/commit/df
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/18918
sorry, Rang not rand
---
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 s
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80681/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
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/18918
**[Test build #80681 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80681/testReport)**
for PR 18918 at commit
[`82b82af`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18918
**[Test build #80681 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80681/testReport)**
for PR 18918 at commit
[`82b82af`](https://github.com/apache/spark/commit/82
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80670/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
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/18918
**[Test build #80670 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80670/testReport)**
for PR 18918 at commit
[`4daec54`](https://github.com/apache/spark/commit/4
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/18918
@gatorsmile
I have fixt it in PhysicalOperation . and extract a new object
FilterOperation to idle it.
but Unit testing is not good to added.
---
If your project is set up for it, you
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18918
**[Test build #80670 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80670/testReport)**
for PR 18918 at commit
[`4daec54`](https://github.com/apache/spark/commit/4d
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18918
Yes. We should fix it in `object PhysicalOperation`
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80628/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
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/18918
**[Test build #80628 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80628/testReport)**
for PR 18918 at commit
[`bf81c45`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18918
**[Test build #80628 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80628/testReport)**
for PR 18918 at commit
[`bf81c45`](https://github.com/apache/spark/commit/bf
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/18918
@gatorsmile
I tried to modify PhysicalOperation's collectProjectsAndFilters, Added an
match case:
```
case Project(fields, child: LeafNode) if
!fields.forall(_.deterministic) =>
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18918
This is the plan of a Hive serde table.
The fix should not be done in optimizer. We should fix it in the place that
causes the issue.
---
If your project is set up for it, you can re
Github user heary-cao commented on the issue:
https://github.com/apache/spark/pull/18918
@gatorsmile
data source tables:
HiveTableScans result plan:HiveTableScan [c030#204L, d004#205, d005#206,
d025#207, c002#208, d023#209, d024#210, c005#211L, c008#212, c009#213,
c010#214,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/80545/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
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/18918
**[Test build #80545 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80545/testReport)**
for PR 18918 at commit
[`b79b9af`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/18918
**[Test build #80545 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/80545/testReport)**
for PR 18918 at commit
[`b79b9af`](https://github.com/apache/spark/commit/b7
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18918
ok to 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, or if
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18918
How about data source tables?
---
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 an
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/18918
Can one of the admins verify this patch?
---
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 feat
42 matches
Mail list logo