Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21175
Plz add `[CORE]` in the title.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/21175
@manbuyun you need to add the unit test into `ChunkedByteBufferSuite.scala`
and push a new commit.
---
-
To unsubscribe, e-mail:
Github user manbuyun commented on the issue:
https://github.com/apache/spark/pull/21175
test("writeFully() does not affect original buffer's position") {
val chunkedByteBuffer = new
ChunkedByteBuffer(Array(ByteBuffer.allocate(80*1024*1024)))
chunkedByteBuffer.writeFul
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21175
Would it be possible to add a unit test?
---
-
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/21175
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/21175
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
