RevolutionAnalytics / RHadoop

RHadoop
https://github.com/RevolutionAnalytics/RHadoop/wiki
763 stars 278 forks source link

replace file.rename when working with local backend with a more robust alternative #111

Closed saargolde closed 12 years ago

saargolde commented 12 years ago

When working with the local backend, the output is copied over from the tmp folder to the desired output location using file.rename, which is brittle (does not work sometimes, when the tmp folder is on a different file system, for example).

Should probably be replaced with a combination of file.copy and then deleting the temp file using file.remove (similar to how it's done for the hadoop backed).

piccolbo commented 12 years ago

Patch looks good, please submit pull request (should be one more click for you)