Github user rxin commented on the issue:
https://github.com/apache/spark/pull/22344
guys - the whole sequence of prs for this feature are contributing a lot of
cryptic code with arcane documentation everywhere. i worry a lot about the
maintainability of the code that's coming in. can you submit a pr to improve
the readability? otherwise i think we should revert all of them.
please document the algorithm, the change, the config flags. it shouldn't
require reading the actual implementation to understand the config flags.
we as committers should also do a better job at gating unreadable code.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
