Github user ala commented on the issue:
https://github.com/apache/spark/pull/16960
Thanks @jaceklaskowski - it's already done:
https://github.com/apache/spark/pull/17939
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user jaceklaskowski commented on the issue:
https://github.com/apache/spark/pull/16960
I'll have a look at this this week and send a PR unless you beat me to it
:) Thanks @ala!
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user ala commented on the issue:
https://github.com/apache/spark/pull/16960
True. There's a couple of lines that should be removed with this change,
that were left behind. numGeneratedRows should be gone.
---
If your project is set up for it, you can reply to this email and
Github user jaceklaskowski commented on the issue:
https://github.com/apache/spark/pull/16960
I think that the commit has left
[numGeneratedRows](https://github.com/apache/spark/blob/master/sql/core/src/main/scala/org/apache/spark/sql/execution/basicPhysicalOperators.scala#L344)
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/16960
Merging in master.
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16960
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16960
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73057/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16960
**[Test build #73057 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73057/testReport)**
for PR 16960 at commit
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/16960
LGTM - pending jenkins.
---
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/16960
**[Test build #73057 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73057/testReport)**
for PR 16960 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16960
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16960
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73005/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16960
**[Test build #73005 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73005/testReport)**
for PR 16960 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16960
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16960
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/73004/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16960
**[Test build #73004 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73004/testReport)**
for PR 16960 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16960
**[Test build #73005 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73005/testReport)**
for PR 16960 at commit
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/16960
cc @hvanhovell if you have a min to review this ...
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16960
**[Test build #73004 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/73004/testReport)**
for PR 16960 at commit
19 matches
Mail list logo
