sangkyoonnam commented on issue #1189:
URL: https://github.com/apache/flink-agents/issues/1189#issuecomment-6031787038

   Thanks for the probe and the control runs. I'd like to take this if no one 
else is working on it.
   
   It overlaps my open #1182, which reworks the same `try` block in 
`_record_call_completion` (payloads go through `_serialize_call_payloads`, and 
a detected Java interruption records nothing). My plan is to keep handling 
Python-side serialization failures there, but let an error from 
`recordCallCompletion` reach the caller instead of being swallowed, with a test 
that injects a persister write failure and asserts the durable call raises on 
both the sync and the individually awaited async path. I'd stack it on #1182 as 
a follow-up unless you'd rather see it folded into that PR.
   


-- 
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