Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/8758#issuecomment-144741977
Good, yes passing through the args is right. Right now if you run
`start-history-server.sh foo` you will successfully set the log directory to
foo because of what the script does. That's what I'm trying to preserve. Or am
I also missing something there?
Right now there is no other arg parsing to break, right? nothing is passed
or parsed otherwise.
I tend to agree it's a little janky, but hey it generates a warning.
Compatibility is good if it's cheap and it seems easy here. But then again it's
been deprecated for forever. So I don't feel strongly about keeping it if there
are strong feelings for retiring this.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
