feat: increase DEFAULT_CHUNK_SIZE to reduce transfer overhead (#87)

googleapis · Dec 13, 2019 · 09b327d · 09b327d
1 parent 9b4df8f
commit 09b327d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/google-cloud-core/src/main/java/com/google/cloud/BaseWriteChannel.java b/google-cloud-core/src/main/java/com/google/cloud/BaseWriteChannel.java
@@ -36,8 +36,8 @@ public abstract class BaseWriteChannel<
         ServiceOptionsT extends ServiceOptions<?, ServiceOptionsT>, EntityT extends Serializable>
     implements WriteChannel {
 
-  private static final int MIN_CHUNK_SIZE = 256 * 1024;
-  private static final int DEFAULT_CHUNK_SIZE = 8 * MIN_CHUNK_SIZE;
+  private static final int MIN_CHUNK_SIZE = 256 * 1024; // 256 KiB
+  private static final int DEFAULT_CHUNK_SIZE = 60 * MIN_CHUNK_SIZE; // 15MiB
 
   private final ServiceOptionsT options;
   private final EntityT entity;

diff --git a/google-cloud-core/src/test/java/com/google/cloud/BaseWriteChannelTest.java b/google-cloud-core/src/test/java/com/google/cloud/BaseWriteChannelTest.java
@@ -55,8 +55,8 @@ protected CustomServiceOptions(
   private static final Serializable ENTITY = 42L;
   private static final String UPLOAD_ID = "uploadId";
   private static final byte[] CONTENT = {0xD, 0xE, 0xA, 0xD};
-  private static final int MIN_CHUNK_SIZE = 256 * 1024;
-  private static final int DEFAULT_CHUNK_SIZE = 8 * MIN_CHUNK_SIZE;
+  private static final int MIN_CHUNK_SIZE = 256 * 1024; // 256 KiB
+  private static final int DEFAULT_CHUNK_SIZE = 60 * MIN_CHUNK_SIZE; // 15MiB
   private static final Random RANDOM = new Random();
   private static BaseWriteChannel channel;