Github user rekhajoshm commented on the pull request:
https://github.com/apache/spark/pull/8842#issuecomment-141831005
Ok @srowen .If you both review, validate its not correct/needed, thats
fine. @rxin
imo more than how the configuration is done or whether call should ever be
invoked by user/developer, its a matter of consistency.None of the conf/ files
have the apache license atm. Maybe they all can include it then?If not then all
files under conf/ can forgo check on license.
@srowen You are right. I have a local setup with conf files which I use for
tinkering, need run-tests to be working for pull request.You can verify this
pull makes the change spark-defaults.conf.* which works.There are additional
similar changes.
---
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
