ok

benchmark/suite_test.go

@@ -22,11 +22,9 @@ func TestBenchmarkSuite(t *testing.T) {
 
 	ctx := context.Background()
 	makeTest("SingleClient", func(t *testing.T, server *server.Server, client codec.Conn) {
-		for i := 0; i < 10; i++ {
-			err := client.Do(ctx, nil, "test_ping", nil)
-			if err != nil {
-				t.Error(err)
-			}
+		err := client.Do(ctx, nil, "test_ping", nil)
+		if err != nil {
+			t.Error(err)
 		}
 	})
 }
@@ -41,9 +39,11 @@ func runBenchmarkSuite(b *testing.B, sm jrpctest.ServerMaker) {
 	}
 	makeBench("SingleClient", func(b *testing.B, server *server.Server, client codec.Conn) {
 		for i := 0; i < b.N; i++ {
-			err := client.Do(ctx, nil, "test_ping", nil)
-			if err != nil {
-				panic(err)
+			for j := 0; j < 100; j++ {
+				err := client.Do(ctx, nil, "test_ping", nil)
+				if err != nil {
+					panic(err)
+				}
 			}
 		}
 	})

pkg/server/responsewriter.go

@@ -13,7 +13,7 @@ import (
 )
 
 // 16mb... should be more than enough for any batch.
-// you shouldn't be batching more than this
+// you shouldn't be batching more than this. really, you shouldn't be using batching at all.
 // TODO: make this configurable
 const maxBatchSizeBytes = 1024 * 1024 * 1024 * 16