Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22427
Merged to master/2.4.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: r
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22427
Thank you, @maropu !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: re
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22427
Thanks for the explanation! The change looks good to me.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For ad
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22427
Thank you for review, @maropu .
1. Yes. It's the same. The first one limits the memory usage for write
operation. The second one limits the memory usage for compression operation.
2. Ye
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22427
Just a question; I'm not familiar with both internal logics though, these
parameters (`Memory buffer for writing` and `Compression chunk size`) are
internally treated in the same manner? Also, they a
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22427
cc @rdblue
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22427
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/22427
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96092/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22427
**[Test build #96092 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96092/testReport)**
for PR 22427 at commit
[`fb14cd5`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22427
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3127/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22427
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/22427
**[Test build #96092 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96092/testReport)**
for PR 22427 at commit
[`fb14cd5`](https://github.com/apache/spark/commit/fb
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22427
Could you review this, @gatorsmile , @cloud-fan , @dbtsai , @maropu and
@wangyum ?
---
-
To unsubscribe, e-mail: reviews-u
13 matches
Mail list logo