Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23127
thanks, merging to master!
---
-
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/23127
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99339/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
**[Test build #99339 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99339/testReport)**
for PR 23127 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
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/23127
**[Test build #99339 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99339/testReport)**
for PR 23127 at commit
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23127
> jenkins retest this please
@juliuszsompolski It won't help, you need to fix python tests it seems
```
Failed example:
df.explain()
Differences (ndiff with
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99329/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23127
**[Test build #99329 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99329/testReport)**
for PR 23127 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
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/23127
**[Test build #99329 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99329/testReport)**
for PR 23127 at commit
Github user juliuszsompolski commented on the issue:
https://github.com/apache/spark/pull/23127
jenkins retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user juliuszsompolski commented on the issue:
https://github.com/apache/spark/pull/23127
jenkins retest this
---
-
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/23127
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99321/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
**[Test build #99321 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99321/testReport)**
for PR 23127 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99320/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
**[Test build #99320 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99320/testReport)**
for PR 23127 at commit
Github user juliuszsompolski commented on the issue:
https://github.com/apache/spark/pull/23127
Talked with @hvanhovell offline and set `LocalTableScanExec` and
`InputAdapter` to not create an unsafe projection, and `RDDScanExec` and
`RowDataSourceScanExec` to always do so, which
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23127
**[Test build #99321 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99321/testReport)**
for PR 23127 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user juliuszsompolski commented on the issue:
https://github.com/apache/spark/pull/23127
@cloud-fan Thanks. Actually, I had to revert earlier updates because the
plan no longer changes for LocalTableScanExec that is alone in a
wholestagecodegen.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
**[Test build #99320 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99320/testReport)**
for PR 23127 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23127
there are still 2 golden file test failures because of the plan change...
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99286/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
**[Test build #99286 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99286/testReport)**
for PR 23127 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23127
**[Test build #99286 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99286/testReport)**
for PR 23127 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
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/23127
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99275/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
**[Test build #99275 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99275/testReport)**
for PR 23127 at commit
Github user juliuszsompolski commented on the issue:
https://github.com/apache/spark/pull/23127
@cloud-fan @rednaxelafx
Actually, the input to a codegen stage can be an internal row so I can't
make the inputRDD be `RDD[UnsafeRow], but the output needs to be UnsafeRow.
Doing it
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
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/23127
**[Test build #99275 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99275/testReport)**
for PR 23127 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99259/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
**[Test build #99259 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99259/testReport)**
for PR 23127 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99258/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
**[Test build #99258 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99258/testReport)**
for PR 23127 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
**[Test build #99259 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99259/testReport)**
for PR 23127 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23127
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23127
**[Test build #99258 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99258/testReport)**
for PR 23127 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23127
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/23127
Looks good. One more higher level question that can also be addressed in a
follow-up.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
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/23127
**[Test build #99218 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99218/testReport)**
for PR 23127 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23127
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99218/
Test FAILed.
---
60 matches
Mail list logo
