Github user nchammas commented on the pull request:

    https://github.com/apache/spark/pull/3707#issuecomment-68189975
  
    > @nchammas I spoke too soon earlier regarding it correctly handling 
relative paths. I fixed it and is now pwd-preserving.
    
    :+1:
    
    On average, it would be better to avoid changing the working directly at 
all when possible, or change and restore it using `pushd` and `popd` as opposed 
to `cd`. That makes it less likely that future changes to the script will 
introduce errors related to path handling.
    
    For now, though, I think this is fine.


---
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]

Reply via email to