Vamsi-klu commented on PR #69307:
URL: https://github.com/apache/airflow/pull/69307#issuecomment-4889070574
Could the fallback avoid reusing the original `timestamp` when validation
failed? Right now any `ValidationError` falls into the except block, but then
it builds `StructuredLogMessage(..., timestamp=fields.get("timestamp"))`. If
the malformed field was `timestamp` rather than `event`, the fallback can raise
the same validation error again and still fail the whole log fetch.
---
Drafted-by: Codex (GPT-5); reviewed by @Vamsi-klu before posting
--
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]
