From a69fbb0a2b8bc3e6db1e5eb29c34284b8e99f0f5 Mon Sep 17 00:00:00 2001 From: Louis Ryan Date: Wed, 24 Jun 2015 14:34:06 -0700 Subject: [PATCH] Use 1MB as the max payload size in benchmarks to avoid exceeding Netty's buffer pool default max buffer size. --- .../jmh/java/io/grpc/benchmarks/netty/AbstractBenchmark.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/benchmarks/src/jmh/java/io/grpc/benchmarks/netty/AbstractBenchmark.java b/benchmarks/src/jmh/java/io/grpc/benchmarks/netty/AbstractBenchmark.java index f7cadf0cc8..db9a3df4aa 100644 --- a/benchmarks/src/jmh/java/io/grpc/benchmarks/netty/AbstractBenchmark.java +++ b/benchmarks/src/jmh/java/io/grpc/benchmarks/netty/AbstractBenchmark.java @@ -48,7 +48,9 @@ public abstract class AbstractBenchmark { * Standard payload sizes. */ public enum PayloadSize { - SMALL(10), MEDIUM(1024), LARGE(65536), JUMBO(16777216); + // Max out at 1MB to avoid creating payloads larger than Netty's buffer pool can handle + // by default + SMALL(10), MEDIUM(1024), LARGE(65536), JUMBO(1048576); private final int bytes; PayloadSize(int bytes) {