Use VLong for byte serialization

Author: nicktindall

server/src/main/java/org/elasticsearch/common/blobstore/BlobStoreActionStats.java

@@ -28,7 +28,7 @@ public record BlobStoreActionStats(long operations, long requests) implements Wr
     public static final BlobStoreActionStats ZERO = new BlobStoreActionStats(0, 0);
 
     public BlobStoreActionStats(StreamInput in) throws IOException {
-        this(in.readLong(), in.readLong());
+        this(in.readVLong(), in.readVLong());
     }
 
     public BlobStoreActionStats {
@@ -38,8 +38,8 @@ public BlobStoreActionStats(StreamInput in) throws IOException {
 
     @Override
     public void writeTo(StreamOutput out) throws IOException {
-        out.writeLong(operations);
-        out.writeLong(requests);
+        out.writeVLong(operations);
+        out.writeVLong(requests);
     }
 
     public BlobStoreActionStats add(BlobStoreActionStats other) {

server/src/test/java/org/elasticsearch/common/blobstore/BlobStoreActionStatsTests.java

@@ -11,6 +11,8 @@
 
 import org.elasticsearch.test.ESTestCase;
 
+import java.io.IOException;
+
 public class BlobStoreActionStatsTests extends ESTestCase {
 
     public void testAdd() {
@@ -37,6 +39,13 @@ public void testIsZero() {
         assertFalse(randomBlobStoreActionStats(1, Long.MAX_VALUE).isZero());
     }
 
+    public void testSerialization() throws IOException {
+        final BlobStoreActionStats original = randomBlobStoreActionStats(Long.MAX_VALUE);
+        BlobStoreActionStats deserializedModel = copyWriteable(original, null, BlobStoreActionStats::new);
+        assertEquals(original, deserializedModel);
+        assertNotSame(original, deserializedModel);
+    }
+
     private BlobStoreActionStats randomBlobStoreActionStats(long upperBound) {
         return randomBlobStoreActionStats(0, upperBound);
     }