Skip to content

Commit 9e81679

Browse files
authored
Enable zstd compression on gmessage topic by default (#919)
The data shows average of 1.5x reduction in message size on 2k network for partial gmessage sizes. Based on the testing, so far it's harmless and beneficial to have compression on gmessage topic enabled.
1 parent 6c27189 commit 9e81679

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

manifest/manifest.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ var (
5555
}
5656

5757
DefaultPubSubConfig = PubSubConfig{
58-
CompressionEnabled: false,
58+
CompressionEnabled: true,
5959
ChainCompressionEnabled: true,
6060
GMessageSubscriptionBufferSize: 128,
6161
ValidatedMessageBufferSize: 128,

manifest/manifest_test.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -124,8 +124,8 @@ func TestManifest_CID(t *testing.T) {
124124
t.Parallel()
125125

126126
const (
127-
wantLocalDevnetCid = "baguqfiheaiqh6vtsf7fionpvi2gp3xwz5vbtbkaabcb7gidub2skahnzjclq5wy"
128-
wantAfterUpdateCid = "baguqfiheaiqoplbaxrgczvmuiihl3zypbrbqmorvs2yeynm7bcnhkxeq6q2sz2y"
127+
wantLocalDevnetCid = "baguqfiheaiqoktbekgcqvdpzlfnxvcocafjj5n6enxsokutvkkfqi6vrowr4gay"
128+
wantAfterUpdateCid = "baguqfiheaiqiqtpn555ipnmjfp6ehzfymrtb5p5gyu2gmq74nohxlxwvhy3dx3y"
129129
)
130130
subject := manifest.LocalDevnetManifest()
131131
// Use a fixed network name for deterministic CID calculation.

0 commit comments

Comments
 (0)