small optimization: do not use native parsing for short messages

KSDaemon · KSDaemon · commit 761728645588 · 2024-12-10T12:28:08.000+02:00
diff --git a/packages/cubejs-cubestore-driver/src/WebSocketConnection.ts b/packages/cubejs-cubestore-driver/src/WebSocketConnection.ts
@@ -118,7 +118,7 @@ export class WebSocketConnection {
             throw new Error(`Cube Store missed message id: ${httpMessage.messageId()}`); // logging
           }
 
-          if (getEnv('nativeOrchestrator')) {
+          if (getEnv('nativeOrchestrator') && msg.length > 1000) {
             try {
               const nativeResMsg = parseCubestoreResultMessage(msg);
               resolvers.resolve(nativeResMsg);

Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,7 @@ export class WebSocketConnection {`
`118`	`118`	throw new Error(`Cube Store missed message id: ${httpMessage.messageId()}`); // logging
`119`	`119`	`}`
`120`	`120`
`121`		`- if (getEnv('nativeOrchestrator')) {`
	`121`	`+ if (getEnv('nativeOrchestrator') && msg.length > 1000) {`
`122`	`122`	`try {`
`123`	`123`	`const nativeResMsg = parseCubestoreResultMessage(msg);`
`124`	`124`	`resolvers.resolve(nativeResMsg);`