Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20044
In the past we've said 'no' to random small changes to the code like this.
Some changes like _ to null probably aren't worth it at all. (0) to .head is
the same, I think.
Removing unused imports is OK, and I personally am OK with .size to .length
for arrays because it avoids an implicit conversion. But, it needs to be done
consistently or not at all.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
