Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21932
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21932
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94351/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21932
**[Test build #94351 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94351/testReport)**
for PR 21932 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21932
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21932
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21932
**[Test build #94351 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94351/testReport)**
for PR 21932 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21932
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21932
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94348/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21932
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21932
**[Test build #94348 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94348/testReport)**
for PR 21932 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21932
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21932
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21932
**[Test build #94348 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94348/testReport)**
for PR 21932 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21932
It sounds reasonable to me. cc @rxin
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21932
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93833/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21932
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21932
**[Test build #93833 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93833/testReport)**
for PR 21932 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21932
@rxin there are different patterns
1. only match a node if its children are resolved. This is the most common
pattern and we should transform up.
2. match a specific node without
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21932
**[Test build #93833 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93833/testReport)**
for PR 21932 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21932
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21932
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/21932
Do we really need this? It's almost always the case for resolution that
you'd want to do bottom up, so I thought Michael's original design to just call
it resolveOperators make a lot of sense.
---
Github user holdensmagicalunicorn commented on the issue:
https://github.com/apache/spark/pull/21932
@cloud-fan, thanks! I am a bot who has found some folks who might be able
to help with the review:@gatorsmile, @yhuai and @rxin
---
23 matches
Mail list logo