Github user Fokko commented on the issue:
https://github.com/apache/spark/pull/22399
@srowen Any incentive to move this forward? Or are PR's like these not
appreciated? Let me know.
Most of the changes are cosmetic, but having <Java8 in the codebase give a
bad example to newcomers to the Spark project. Also adding the `Closeable` to
the `RowBasedKeyValueBatchSuite.java`:
https://github.com/apache/spark/pull/22399/files#diff-6c2c45f79666e2e52eb9f9411fa8b4baR49
makes the codebase a bit nicer in my opinion, since the class already has a
`.close()` method, it makes sense to also implement the `Closeable` interface.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
