Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23225
If a test passes without/with this changes, it wouldn't verify the
regression.
---
-
To unsubscribe, e-mail:
Github user wangjiaochun commented on the issue:
https://github.com/apache/spark/pull/23225
1. I think test case writeEmptyIterator in UnsafeShuffleWriterSuite.java
cover this scenes
2. I will propose a JIRA soon.
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23225
Also, it needs a JIRA. it's not minor one.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23225
How come existing tests cover if the empty file is created or not?
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23225
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/23225
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/23225
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional