Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22162
@AndrewKL Thanks for your work! I think we can add enough tests for this
patch without using a spy library.
---
-
To
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22162
Can you fix the style errors?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22162
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/22162
**[Test build #95804 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95804/testReport)**
for PR 22162 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22162
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95804/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22162
**[Test build #95804 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95804/testReport)**
for PR 22162 at commit
Github user AndrewKL commented on the issue:
https://github.com/apache/spark/pull/22162
Hey Friends, I have updated the commit and fixed most of the issues
mentioned. I do have one remaining issue left which is the testing of the new
show functionality. Currently the provided
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22162
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95775/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22162
Build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22162
**[Test build #95775 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95775/testReport)**
for PR 22162 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22162
**[Test build #95775 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95775/testReport)**
for PR 22162 at commit
Github user AndrewKL commented on the issue:
https://github.com/apache/spark/pull/22162
Hey Friends,
Sorry for the delayed response. I've been in the desert 5 hours into the
middle of no where. I'll incorporate the feed back and update this.
---
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22162
ya, sure.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22162
We should wait @AndrewKL for few days?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22162
I have much bandwidh to take it, too. Is it ok to take it over? @mgaido91
not working on this now?
---
-
To unsubscribe,
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22162
sure, no worries @kiszk, I can take it if needed. Thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22162
Would someone please take it?
I have less bandwidth next two days since I will be in a training session
at my office.
---
-
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22162
sure @HyukjinKwon, thanks for pinging me anyway.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22162
@viirya, @kiszk, @mgaido91 and @maropu, would you be interested in taking
this over if this gets inactive for few more days?
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22162
ping @AndrewKL
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22162
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22162
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95254/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22162
**[Test build #95254 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95254/testReport)**
for PR 22162 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22162
**[Test build #95254 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95254/testReport)**
for PR 22162 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22162
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22162
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/22162
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/22162
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
28 matches
Mail list logo