Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21668
@cloud-fan ping
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21668
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/93382/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21668
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21668
**[Test build #93382 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93382/testReport)**
for PR 21668 at commit
[`0b1f751`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21668
**[Test build #93382 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/93382/testReport)**
for PR 21668 at commit
[`0b1f751`](https://github.com/apache/spark/commit/0b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21668
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/1199/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21668
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21668
ping
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21668
@cloud-fan If no problem, could you check #20345 and merge it first? Based
on that, I'd like to start refactoring for the approach.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21668
sounds reasonable, also cc @wzhfy @maryannxue
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additiona
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21668
One of refactoring ideas is to inject the functionality of
`ReorderJoin`(=`StarSchemaDetection`)
into `CostBasedJoinReorder`;
In [the batch rule `Join Reorder` (`Once`
strategy)](https:/
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21668
yea, ok. I'll reconsider this again. Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional com
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21668
yea this is a real problem, but I feel a better solution is to integrate
the StarSchemaDetection into CBO. How hard will it be?
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21668
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21668
**[Test build #92460 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92460/testReport)**
for PR 21668 at commit
[`f0db73b`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21668
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92460/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21668
**[Test build #92460 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92460/testReport)**
for PR 21668 at commit
[`f0db73b`](https://github.com/apache/spark/commit/f0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21668
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21668
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/572/
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21668
This comes from #20345.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
20 matches
Mail list logo