Github user jbax commented on the pull request:
https://github.com/apache/spark/pull/13267#issuecomment-221454486
@rxin In your case think it's better to have this turned on by default.
Regarding your other questions:
1 - There's no timeline. 2.2.x will come out when new features are
requested by our users and implemented. Currently there's nothing in the
pipeline so we'll be on 2.1.x adding fixes and minor internal improvements over
time. We have no open bugs either.
2 - Yes. It fixes a couple of bugs you guys probably won't come across, but
it also improves the performance of the parser with whitespace trimming enabled
(it's enabled by default, by the way).
3 - It's OK and I don't see why it would be a problem, other than having
some client with a very uncommon use case (they are out there, that's why the
library has a lot of configuration options).
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
