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

Closed
saargolde opened this Issue Jun 26, 2012 · 1 comment

Projects

None yet

2 participants

@saargolde

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).

@saargolde saargolde pushed a commit to saargolde/RHadoop that referenced this issue Jun 26, 2012
= changed file copy in local backend from file.rename to file.copy (Issue 347d39f
@piccolbo

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

@piccolbo piccolbo closed this Jul 9, 2012
@piccolbo piccolbo added a commit that referenced this issue Jul 10, 2012
@piccolbo piccolbo closes #111 #112 25c6403
@piccolbo piccolbo added a commit that referenced this issue Jul 10, 2012
@piccolbo piccolbo closes #111 #112 afe0d9a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment