Github user maropu commented on the issue:
https://github.com/apache/spark/pull/10896
@yhuai Thanks your comment and I agree with you. We'll keep the discussion.
---
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 pro
Github user yhuai commented on the issue:
https://github.com/apache/spark/pull/10896
@maropu Thank you for working on this. Sorry that I did not get time to
look at it after you updated the pr. I looked at it today. I think this
optimization deserves a feature flag since it determines
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/10896
Sorry for my bad explanation. yes, I agree that we remove the aggregation
stuff from `EnsureRequirements` for simpler codes. I'd say, how about moving
the aggregation stuff (creating partial aggregat
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/10896
I agree that partial aggregate is also kind of optimization, and it's
tricky to put it in planner. I think it makes sense to clean it up, after we
have sufficient discussion and come to a consensu
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/10896
@cloud-fan @liancheng yea, adding a new rule after `EnsureRequirements`
sounds good to me. One question; creating a partial aggregation in the planner
and removing it in the new rule seem to be kind
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/10896
You could also argue the other way around, planning a partial aggregate is
also a premature optimization, and that the planning of such an `Aggregate`
could also be considered tricky code. BTW: t
Github user liancheng commented on the issue:
https://github.com/apache/spark/pull/10896
+1 for @cloud-fan's proposal. Instead of creating a performant plan using
tricky code, it's clearer to create a naive but correct physical plan first and
then optimize it.
---
If your project is
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/10896
After this PR, we create the partial aggregate operator in
`EnsureRequirements`, which makes the aggregation code harder to understand and
also mess up `EnsureRequirements`.
I have a simp
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/10896
A high-level question: After this PR, `EnsureRequirements` also need to
know about aggregate, and handle partial aggregate logic. This makes the
aggregation code harder to understand, instead of d
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/10896
LGTM - merging to master. Thanks!
---
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
enable
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/10896
@hvanhovell could you also give me comments on #13852?
---
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 th
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64388/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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/10896
**[Test build #64388 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64388/consoleFull)**
for PR 10896 at commit
[`ac68145`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #64388 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64388/consoleFull)**
for PR 10896 at commit
[`ac68145`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64324/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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/10896
**[Test build #64324 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64324/consoleFull)**
for PR 10896 at commit
[`d5e0ed3`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #64324 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64324/consoleFull)**
for PR 10896 at commit
[`d5e0ed3`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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/10896
**[Test build #64322 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64322/consoleFull)**
for PR 10896 at commit
[`8a81e23`](https://github.com/apache/spark/commit/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64322/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #64322 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64322/consoleFull)**
for PR 10896 at commit
[`8a81e23`](https://github.com/apache/spark/commit/8
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/10896
okay, done
---
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 the
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64283/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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/10896
**[Test build #64283 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64283/consoleFull)**
for PR 10896 at commit
[`69751bb`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #64283 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64283/consoleFull)**
for PR 10896 at commit
[`69751bb`](https://github.com/apache/spark/commit/6
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/10896
@hvanhovell okay, fixed.
---
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
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/10896
okay, thanks. I'll fix in hours.
---
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64218/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #64218 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64218/consoleFull)**
for PR 10896 at commit
[`86068d0`](https://github.com/apache/spark/commit/
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/10896
@maropu this is shaping up nicely. I left a few more small comments.
---
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 projec
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #64218 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64218/consoleFull)**
for PR 10896 at commit
[`86068d0`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64209/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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/10896
**[Test build #64209 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64209/consoleFull)**
for PR 10896 at commit
[`0375ac6`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #64209 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64209/consoleFull)**
for PR 10896 at commit
[`0375ac6`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64200/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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/10896
**[Test build #64200 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64200/consoleFull)**
for PR 10896 at commit
[`8b64305`](https://github.com/apache/spark/commit/
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/10896
@hvanhovell fixed. plz check again?
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64199/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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/10896
**[Test build #64199 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64199/consoleFull)**
for PR 10896 at commit
[`e37ef6a`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #64200 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64200/consoleFull)**
for PR 10896 at commit
[`8b64305`](https://github.com/apache/spark/commit/8
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #64199 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64199/consoleFull)**
for PR 10896 at commit
[`e37ef6a`](https://github.com/apache/spark/commit/e
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/10896
@hvanhovell okay, ready to review.
---
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 a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64079/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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/10896
**[Test build #64079 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64079/consoleFull)**
for PR 10896 at commit
[`91701eb`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #64079 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64079/consoleFull)**
for PR 10896 at commit
[`91701eb`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64076/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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/10896
**[Test build #64076 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64076/consoleFull)**
for PR 10896 at commit
[`2615a6e`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #64076 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64076/consoleFull)**
for PR 10896 at commit
[`2615a6e`](https://github.com/apache/spark/commit/2
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/10896
Could you update?
---
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,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #3228 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3228/consoleFull)**
for PR 10896 at commit
[`572db4c`](https://github.com/apache/spark/commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #3228 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3228/consoleFull)**
for PR 10896 at commit
[`572db4c`](https://github.com/apache/spark/commit/
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/10896
okay, thanks!!
---
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 hvanhovell commented on the issue:
https://github.com/apache/spark/pull/10896
Yeah, lets pick this up again. Thanks for the ping.
---
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 maropu commented on the issue:
https://github.com/apache/spark/pull/10896
@hvanhovell ping
---
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 i
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60811/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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/10896
**[Test build #60811 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60811/consoleFull)**
for PR 10896 at commit
[`572db4c`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #60811 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60811/consoleFull)**
for PR 10896 at commit
[`572db4c`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60809/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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/10896
**[Test build #60809 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60809/consoleFull)**
for PR 10896 at commit
[`103680c`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #60809 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60809/consoleFull)**
for PR 10896 at commit
[`103680c`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60797/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #60797 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60797/consoleFull)**
for PR 10896 at commit
[`9ac8025`](https://github.com/apache/spark/commit/
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/10896
@hvanhovell okay, ready to review. After the v2.0 release, plz 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 pro
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60800/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #60800 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60800/consoleFull)**
for PR 10896 at commit
[`9ac8025`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #60797 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60797/consoleFull)**
for PR 10896 at commit
[`9ac8025`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #60800 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60800/consoleFull)**
for PR 10896 at commit
[`9ac8025`](https://github.com/apache/spark/commit/9
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/10896
Jenkins, retest this please.
---
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 wis
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60791/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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/10896
**[Test build #60791 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60791/consoleFull)**
for PR 10896 at commit
[`cb47a54`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #60791 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60791/consoleFull)**
for PR 10896 at commit
[`cb47a54`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60766/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #60766 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60766/consoleFull)**
for PR 10896 at commit
[`adad55d`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #60766 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60766/consoleFull)**
for PR 10896 at commit
[`adad55d`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60713/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #60713 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60713/consoleFull)**
for PR 10896 at commit
[`f8ac8cb`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #60713 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60713/consoleFull)**
for PR 10896 at commit
[`f8ac8cb`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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 AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/60697/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #60697 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60697/consoleFull)**
for PR 10896 at commit
[`3ae9153`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/10896
**[Test build #60697 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/60697/consoleFull)**
for PR 10896 at commit
[`3ae9153`](https://github.com/apache/spark/commit/3
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/10896
@hvanhovell okay, I'll finish the fix soon.
---
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 hvanhovell commented on the issue:
https://github.com/apache/spark/pull/10896
@maropu I would like to move this one to Spark 2.1 and implement it like
@yhuai suggested.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/10896
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
1 - 100 of 116 matches
Mail list logo
