Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22219
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/22219
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97893/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22219
**[Test build #97893 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97893/testReport)**
for PR 22219 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22219
**[Test build #97893 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97893/testReport)**
for PR 22219 at commit
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22219
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/22219
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/22219
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/22219
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97530/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22219
**[Test build #97530 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97530/testReport)**
for PR 22219 at commit
Github user Dooyoung-Hwang commented on the issue:
https://github.com/apache/spark/pull/22219
I refactored collectionResultAsSeqView function with using implicit class.
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22219
**[Test build #97530 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97530/testReport)**
for PR 22219 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22219
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97492/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22219
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/22219
**[Test build #97492 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97492/testReport)**
for PR 22219 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22219
**[Test build #97492 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97492/testReport)**
for PR 22219 at commit
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22219
cc @srinathshankar @yuchenhuo
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22219
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97452/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22219
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/22219
**[Test build #97452 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97452/testReport)**
for PR 22219 at commit
Github user Dooyoung-Hwang commented on the issue:
https://github.com/apache/spark/pull/22219
Dear reviewers (cc : @dongjoon-hyun )
I updated these.
1. No behavior changes, if the new config is off. So, [PR
SPARK-25353](https://github.com/apache/spark/pull/22347) is not
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22219
**[Test build #97452 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97452/testReport)**
for PR 22219 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22219
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/22219
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97388/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22219
**[Test build #97388 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97388/testReport)**
for PR 22219 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22219
**[Test build #97388 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97388/testReport)**
for PR 22219 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22219
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97376/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22219
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/22219
**[Test build #97376 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97376/testReport)**
for PR 22219 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22219
**[Test build #97376 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97376/testReport)**
for PR 22219 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22219
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/22219
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97363/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22219
**[Test build #97363 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97363/testReport)**
for PR 22219 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22219
**[Test build #97363 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97363/testReport)**
for PR 22219 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22219
nope not yet. It needs some more review iterations.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22219
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user tooptoop4 commented on the issue:
https://github.com/apache/spark/pull/22219
@HyukjinKwon can this be merged?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22219
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22219
@Dooyoung-Hwang Would it possible to add a test case to verify result with
and without incrementalCollects by changing a value of
`spark.sql.thriftServer.batchDeserializeLimit`?
---
Github user Dooyoung-Hwang commented on the issue:
https://github.com/apache/spark/pull/22219
@kiszk @viirya @HyukjinKwon @cloud-fan
Could you review this patch?
---
-
To unsubscribe, e-mail:
Github user Dooyoung-Hwang commented on the issue:
https://github.com/apache/spark/pull/22219
Change the accessor of collectCountAndIterator to private[sql]. And updated
doc of feature that I define in ThriftServer.
---
Github user Dooyoung-Hwang commented on the issue:
https://github.com/apache/spark/pull/22219
Add test cases.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22219
Would it possible to prepare test cases? IIUC, this feature can be enabled
without thriftServer by writing some test code.
---
-
Github user Dooyoung-Hwang commented on the issue:
https://github.com/apache/spark/pull/22219
Yes, I verified results of a variety of queries, and memory & performance.
This patch passed all our query test. And there was no performance
degradation in our test cases.
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/22219
Did you verify this feature manually?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22219
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/22219
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/22219
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
47 matches
Mail list logo