Skip to content

Commit

Permalink
FORGE-1282: As per the javadoc, the XADisk Home should never be shared
Browse files Browse the repository at this point in the history
with multiple XADisk instances
  • Loading branch information
gastaldi committed Oct 24, 2013
1 parent f92ebc5 commit 7c03b67
Showing 1 changed file with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -41,8 +41,7 @@ public class FileResourceTransactionManager

public void startup(@Observes @Local PostStartup startup) throws Exception
{
File tempDirectory = OperatingSystemUtils.getTempDirectory();
File xaDiskHome = new File(tempDirectory, "xadisk");
File xaDiskHome = OperatingSystemUtils.createTempDir();
StandaloneFileSystemConfiguration config = new StandaloneFileSystemConfiguration(
xaDiskHome.getAbsolutePath(), "furnace-instance");
config.setTransactionTimeout(600);
Expand Down Expand Up @@ -72,5 +71,6 @@ public FileResourceTransactionImpl getCurrentTransaction(ResourceFactory resourc
transaction = new FileResourceTransactionImpl(fileSystem, resourceFactory);
}
return transaction;
}
}

}

1 comment on commit 7c03b67

@lincolnthree
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Interesting!!! this makes a lot of sense, but I'm surprised they did not take steps like this themselves!!

Please sign in to comment.