Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23040
thanks, merging to master!
---
-
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/23040
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/23040
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98989/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23040
**[Test build #98989 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98989/testReport)**
for PR 23040 at commit
Github user LinhongLiu commented on the issue:
https://github.com/apache/spark/pull/23040
cc @cloud-fan @srowen
review is fixed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23040
**[Test build #98989 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98989/testReport)**
for PR 23040 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23040
also cc @jiangxb1987 @zsxwing
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23040
LGTM except one comment
---
-
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/23040
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/23040
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98878/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23040
**[Test build #98878 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98878/testReport)**
for PR 23040 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23040
**[Test build #98878 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98878/testReport)**
for PR 23040 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23040
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23040
It's good to fix a potential bug, can you add a unit test?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user LinhongLiu commented on the issue:
https://github.com/apache/spark/pull/23040
Problem:
ChunkedByteBuffer has signature `ChunkedByteBuffer(var chunks:
Array[ByteBuffer])`. This means user is allowed to pass any kind of `chunks` to
it. Then we will face the problem
Github user advancedxy commented on the issue:
https://github.com/apache/spark/pull/23040
cc @ericl and @JoshRosen, this bug was introduced by
https://github.com/apache/spark/pull/14099/files
After loosing empty chunk check, the ChunkedByteBufferInputStream doesn't
handle
Github user linhong-intel commented on the issue:
https://github.com/apache/spark/pull/23040
cc @xuanyuanking
---
-
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/23040
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/23040
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/23040
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
20 matches
Mail list logo