honeymoose
/
OpenSearch
mirror of https://github.com/honeymoose/OpenSearch.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

[PLUGIN] Don't use atomic move when installing plugin 

the extract location might be on a different filesystem where
atomic move won't work. Yet this operation is not critical in terms
of visibility so there is no need to do this.
This commit is contained in:
Simon Willnauer 2014-12-03 09:48:49 +01:00
parent a436da7cdb
commit 921e028e99
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/main/java/org/elasticsearch/plugins/PluginManager.java
View File

@ -243,7 +243,7 @@ public class PluginManager {
if (Files.exists(toLocation)) { if (Files.exists(toLocation)) {
IOUtils.rm(toLocation); IOUtils.rm(toLocation);
} }
Files.move(binFile, toLocation, StandardCopyOption.ATOMIC_MOVE); Files.move(binFile, toLocation);
if (Files.getFileStore(toLocation).supportsFileAttributeView(PosixFileAttributeView.class)) { if (Files.getFileStore(toLocation).supportsFileAttributeView(PosixFileAttributeView.class)) {
final Set<PosixFilePermission> perms = new HashSet<>(); final Set<PosixFilePermission> perms = new HashSet<>();
perms.add(PosixFilePermission.OWNER_EXECUTE); perms.add(PosixFilePermission.OWNER_EXECUTE);