Github user florianverhein commented on the pull request:
https://github.com/apache/spark/pull/4583#issuecomment-74982303
This seems arbitrary. Also, how would you copy a file into /? The behaviour
is exactly what rsync does (with a trailing /), so I don't think its a good
idea to deviate from this. Now note that I add the trailing slash (so it's
consistent with current deploy.files). I could remove that (and keep the
isdirectory check), which would mean the user can choose the desired behaviour
(leaving off the / should do exactly what you want I think). We should probably
rename the option then, as it's no longer limited to "deploying the contents of
a directory into /". But I don't see what this buys us.
---
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]
