Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22013
I'd merge this now.
@mn-mikke @mgaido91 If you have any other comments, let's have a follow-up
pr.
---
-
To unsubscribe, e-ma
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22013
Thanks! merging to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22013
LGTM.
@mn-mikke @mgaido91 Do you have any other comments on this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apac
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94819/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94819 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94819/testReport)**
for PR 22013 at commit
[`2f4943f`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94817/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94817 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94817/testReport)**
for PR 22013 at commit
[`58b60b2`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94811/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94811 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94811/testReport)**
for PR 22013 at commit
[`fb885f4`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94819 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94819/testReport)**
for PR 22013 at commit
[`2f4943f`](https://github.com/apache/spark/commit/2f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94817 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94817/testReport)**
for PR 22013 at commit
[`58b60b2`](https://github.com/apache/spark/commit/58
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94811 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94811/testReport)**
for PR 22013 at commit
[`fb885f4`](https://github.com/apache/spark/commit/fb
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94788/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94788 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94788/testReport)**
for PR 22013 at commit
[`e5d9b05`](https://github.com/apache/spark/commit/e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94788 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94788/testReport)**
for PR 22013 at commit
[`e5d9b05`](https://github.com/apache/spark/commit/e5
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/22013
Btw, we need one more right parenthesis after the second `array(1, 2, 3)`
and a space at `(k,v)` in the description?
---
-
To uns
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94775/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94775 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94775/testReport)**
for PR 22013 at commit
[`5db526b`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94775 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94775/testReport)**
for PR 22013 at commit
[`5db526b`](https://github.com/apache/spark/commit/5d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94765/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94765 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94765/testReport)**
for PR 22013 at commit
[`621213d`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94765 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94765/testReport)**
for PR 22013 at commit
[`621213d`](https://github.com/apache/spark/commit/62
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94758/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94758 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94758/testReport)**
for PR 22013 at commit
[`bb52630`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94758 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94758/testReport)**
for PR 22013 at commit
[`bb52630`](https://github.com/apache/spark/commit/bb
Github user codeatri commented on the issue:
https://github.com/apache/spark/pull/22013
Thanks for the review @ueshin! I have addressed all your comments.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apach
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94518/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94518 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94518/testReport)**
for PR 22013 at commit
[`1cbaf0c`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94518 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94518/testReport)**
for PR 22013 at commit
[`1cbaf0c`](https://github.com/apache/spark/commit/1c
Github user codeatri commented on the issue:
https://github.com/apache/spark/pull/22013
@hvanhovell @mn-mikke @mgaido91 Thanks for the review! I have addressed
all your comments and added appropriate test cases for the same.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94455/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94457/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94457 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94457/testReport)**
for PR 22013 at commit
[`f7fd231`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94455 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94455/testReport)**
for PR 22013 at commit
[`6526630`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94451/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94451 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94451/testReport)**
for PR 22013 at commit
[`9f6a8ab`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94457 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94457/testReport)**
for PR 22013 at commit
[`f7fd231`](https://github.com/apache/spark/commit/f7
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94455 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94455/testReport)**
for PR 22013 at commit
[`6526630`](https://github.com/apache/spark/commit/65
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94451 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94451/testReport)**
for PR 22013 at commit
[`9f6a8ab`](https://github.com/apache/spark/commit/9f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94405/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94404/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94405 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94405/testReport)**
for PR 22013 at commit
[`150a6a5`](https://github.com/apache/spark/commit/1
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94404 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94404/testReport)**
for PR 22013 at commit
[`5806ac4`](https://github.com/apache/spark/commit/5
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94405 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94405/testReport)**
for PR 22013 at commit
[`150a6a5`](https://github.com/apache/spark/commit/15
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94404 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94404/testReport)**
for PR 22013 at commit
[`5806ac4`](https://github.com/apache/spark/commit/58
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/94315/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94315 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94315/testReport)**
for PR 22013 at commit
[`0a19cc4`](https://github.com/apache/spark/commit/0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22013
**[Test build #94315 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/94315/testReport)**
for PR 22013 at commit
[`0a19cc4`](https://github.com/apache/spark/commit/0a
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/22013
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22013
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/22013
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/22013
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
66 matches
Mail list logo
