Skip to content

Commit

Permalink
Remove FUSE workaround now that FUSE is updated
Browse files Browse the repository at this point in the history
  • Loading branch information
chrisbasoglu committed Feb 25, 2016
1 parent 246907d commit abdc9d3
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions Source/Common/fileutil.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -608,18 +608,17 @@ void renameOrDie(const std::string& from, const std::string& to)
// Delete destination file if it exists
// WORKAROUND: "rename" should do this but this is a workaround
// to the HDFS FUSE implementation's bug of failing to do so
// workaround for FUSE rename when running on Philly
if (ProgressTracing::GetTracingFlag())
{
fprintf(stderr, "rename %s to %s\n", from.c_str(), to.c_str());
}

unlinkOrDie(to);
if (rename(from.c_str(), to.c_str()) != 0)
{
RuntimeError("error renaming file '%s': %s", from.c_str(), strerror(errno));
}

// workaround for FUSE rename when running on Philly
if (ProgressTracing::GetTracingFlag())
{
fprintf(stderr, "sleep for 120 sec as workaround for FUSE write.\n");
sleep(120);
}
#endif
}

Expand Down

0 comments on commit abdc9d3

Please sign in to comment.