diff options
author | Harald Musum <musum@oath.com> | 2017-12-09 10:58:17 +0100 |
---|---|---|
committer | Harald Musum <musum@oath.com> | 2017-12-09 10:58:17 +0100 |
commit | ebe646e39c4227097b2f67e19fffb399e963fef2 (patch) | |
tree | e25b794edc878efabc744ffbf9ce49e21c5eb178 /filedistribution | |
parent | 46947a0782d08c2a58514f12ca788d7dfd9f11a2 (diff) |
Fix some issues related to RPC methods for file distribution
Diffstat (limited to 'filedistribution')
-rw-r--r-- | filedistribution/src/main/java/com/yahoo/vespa/filedistribution/FileReceiver.java | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/filedistribution/src/main/java/com/yahoo/vespa/filedistribution/FileReceiver.java b/filedistribution/src/main/java/com/yahoo/vespa/filedistribution/FileReceiver.java index b28e192f8e4..87bffa145a1 100644 --- a/filedistribution/src/main/java/com/yahoo/vespa/filedistribution/FileReceiver.java +++ b/filedistribution/src/main/java/com/yahoo/vespa/filedistribution/FileReceiver.java @@ -9,16 +9,13 @@ import com.yahoo.jrt.Request; import com.yahoo.jrt.Supervisor; import com.yahoo.log.LogLevel; import net.jpountz.xxhash.StreamingXXHash64; -import net.jpountz.xxhash.XXHash64; import net.jpountz.xxhash.XXHashFactory; import java.io.File; import java.io.IOException; -import java.nio.ByteBuffer; import java.nio.file.FileAlreadyExistsException; import java.nio.file.Files; import java.nio.file.StandardOpenOption; -import java.nio.file.attribute.PosixFilePermissions; import java.util.ArrayList; import java.util.HashMap; import java.util.List; @@ -37,7 +34,6 @@ public class FileReceiver { private final Supervisor supervisor; private final FileReferenceDownloader downloader; private final File downloadDirectory; - private final XXHash64 hasher = XXHashFactory.fastestInstance().hash64(); private final AtomicInteger nextSessionId = new AtomicInteger(1); private final Map<Integer, Session> sessions = new HashMap<>(); @@ -161,7 +157,7 @@ public class FileReceiver { .paramDesc(4, "error-description", "Error description.") .returnDesc(0, "ret", "0 if success, 1 if crc mismatch, 2 otherwise")); // Temporary method until we have chunking - methods.add(new Method(RECEIVE_METHOD, "sssxlis", "i", handler, "receiveFile") + methods.add(new Method(RECEIVE_METHOD, "sssXlis", "i", handler, "receiveFile") .methodDesc("receive file reference content") .paramDesc(0, "file reference", "file reference to download") .paramDesc(1, "filename", "filename") @@ -300,6 +296,7 @@ public class FileReceiver { synchronized (sessions) { sessions.remove(sessionId); } + req.returnValues().add(new Int32Value(retval)); } private final Session getSession(Integer sessionId) { |