aicam opened a new pull request, #6858:
URL: https://github.com/apache/texera/pull/6858

   ### What changes were proposed in this PR?
   
   Raises the default maximum workflow WebSocket message size from **64 KB to 
1024 KB (1 MB)**, in both places the default is defined:
   
   - `common/config/src/main/resources/application.conf` — the base config 
default (`web-server.max-workflow-websocket-request-payload-size-kb`)
   - `bin/k8s/values.yaml` — the Helm chart env default 
(`MAX_WORKFLOW_WEBSOCKET_REQUEST_PAYLOAD_SIZE_KB`), which overrides the base 
default in Kubernetes deployments
   
   `WebsocketPayloadSizeTuner` applies this value to the Jetty WebSocket 
container's max text/binary message buffer. At the previous 64 KB default, the 
result/status frame pushed to the browser for a workflow with many operators or 
a sizable visualization payload (e.g. a Bar Chart) could exceed 65536 bytes; 
Jetty then drops the entire frame with `MessageTooLargeException`, so the final 
result/visualization never reaches the frontend even though the execution 
completes successfully on the backend. The overshoot is often only a couple KB, 
so even modest workflows can trip it. 1 MB gives ample headroom for large 
visualization payloads while remaining well within reasonable WebSocket frame 
sizes.
   
   This changes only a default value; the limit remains overridable via the 
`MAX_WORKFLOW_WEBSOCKET_REQUEST_PAYLOAD_SIZE_KB` environment variable.
   
   ### How was this PR tested?
   
   Manually. Ran a workflow ending in a Bar Chart whose serialized result frame 
exceeded 64 KB:
   - **Before:** the visualization did not render, and the computing unit 
logged `MessageTooLargeException: ... too large for configured max of [65536]` 
at completion.
   - **After:** the frame is delivered and the chart renders.
   
   Also verified with `helm template` that the rendered chart env value updates 
to `1024` and the chart still renders cleanly.
   
   ### Was this PR authored or co-authored using generative AI tooling?
   
   Generated-by: Claude Opus 4.8
   
   ### Any related issues (either this closes or is dependent on)?
   
   Closes #6857
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to