Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20016
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user chetkhatri commented on the issue:
https://github.com/apache/spark/pull/20016
@srowen I think, we can merge this now.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user chetkhatri commented on the issue:
https://github.com/apache/spark/pull/20016
@srowen Thank you for re-run, now it passes all.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20016
**[Test build #4019 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4019/testReport)**
for PR 20016 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20016
**[Test build #4019 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4019/testReport)**
for PR 20016 at commit
Github user chetkhatri commented on the issue:
https://github.com/apache/spark/pull/20016
@srowen Thanks for response, correct - i went through error of jenkins and
found error
Online[https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4018/console]
which fixed
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20016
@chetkhatri not sure what you mean. The whole change here is considered,
and doesn't pass style checks. Look at the test results and fix the errors.
---
Github user chetkhatri commented on the issue:
https://github.com/apache/spark/pull/20016
@srowen why only recent commit is going to merge, can't we get "squash
merge" ?
Please re-run test build. and let me know if still seems wrong.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20016
**[Test build #4018 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4018/testReport)**
for PR 20016 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20016
**[Test build #4018 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4018/testReport)**
for PR 20016 at commit
Github user chetkhatri commented on the issue:
https://github.com/apache/spark/pull/20016
@HyukjinKwon @mgaido91 @srowen All the changes are addressed and committed,
please do review and needful.
---
-
To
Github user chetkhatri commented on the issue:
https://github.com/apache/spark/pull/20016
+ @holdenk @sameeragarwal
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user chetkhatri commented on the issue:
https://github.com/apache/spark/pull/20016
@HyukjinKwon I agree with you ! But since Spark is scala project - A lot's
of developers refer examples available here and if those are Java Developers
they might don't understand that this is
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20016
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
14 matches
Mail list logo