njnu-seafish commented on PR #18549:
URL:
https://github.com/apache/dolphinscheduler/pull/18549#issuecomment-6119020937
@ruanwenjun @SbloodyS Addressed in f4b37e55b7. The alert content is no
longer the full result set; prepareTaskResultAlert now applies its own size
limit before serializing:
keeps at most 30 rows; beyond that a trailing {"...":"N more rows omitted"}
marker is appended, so the payload stays a valid JSON array (Feishu /
Prometheus / Webex channels still parse it);
truncates string field values longer than 100 chars with an ellipsis.
This caps what gets serialized into the alert, which is the OOM surface you
flagged. It is deliberately decoupled from QUERY_LIMIT / setMaxRows — the query
result and task log are unchanged; only the sent alert content is bounded.
The 30 / 100 defaults are private constants for now; aligned with your note
that system-side limits shouldn't be user-facing, they'll move to a service
config file in a follow-up PR rather than become task parameters.
--
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]
