Merge pull request #893 from ronawho/opt-array-transfer-copies

Further reduce the number of copies for client-server array transfers
Bears-R-Us · Aug 3, 2021 · 6ecd185 · 6ecd185
2 parents 022e06d + 5bd8191
commit 6ecd185
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/src/arkouda_server.chpl b/src/arkouda_server.chpl
@@ -220,10 +220,8 @@ proc main() {
          * string encapsulating user, token, cmd, message format and args from the 
          * remaining payload.
          */
-        var (rawRequest, payload) = reqMsgRaw.splitMsgToTuple(b"BINARY_PAYLOAD",2);
-        if reqMsgRaw.endsWith(b"BINARY_PAYLOAD") {
-          payload = socket.recv(bytes);
-        }
+        var (rawRequest, _) = reqMsgRaw.splitMsgToTuple(b"BINARY_PAYLOAD",2);
+        var payload = if reqMsgRaw.endsWith(b"BINARY_PAYLOAD") then socket.recv(bytes) else b"";
         var user, token, cmd: string;
 
         // parse requests, execute requests, format responses