Github user vanzin commented on the pull request:
https://github.com/apache/spark/pull/13407
Actually, looking at it again, I think the fix makes the code a little
confusing. Why is `appResource` being used to store the argument to an option?
I know it all aligns in the end, but it looks weird, and it might be possible
to break this.
I think it would be better to keep another flag saying whether
`appResource` is required or not, and only checking that (around L159)
depending on the value of the flag. Then the existing cases (no args and help)
could be changed to set that flag too.
---
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]
