Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21988
why do we need multiple PRs? typically we do that only then the change is
non-trivial or cannot be backported by cherry-picking.
---
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/21988
And put `[BRANCH-2.3]` into the title of the PR #21989 too?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21988
@RussellSpitzer, btw mind if I ask to put `[BRANCH-2.2]` inthe PR title to
just make it less confused?
---
-
To
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21988
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94153/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21988
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/21988
**[Test build #94153 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94153/testReport)**
for PR 21988 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21988
**[Test build #94153 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94153/testReport)**
for PR 21988 at commit
Github user RussellSpitzer commented on the issue:
https://github.com/apache/spark/pull/21988
Local PEP didn't seem to mind this code ... Fixed up the indentation so
hopefully jenkins will like it now
---
-
To
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21988
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21988
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21988
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94144/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21988
**[Test build #94144 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94144/testReport)**
for PR 21988 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21988
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/21988
**[Test build #94144 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94144/testReport)**
for PR 21988 at commit
14 matches
Mail list logo