joeyutong commented on PR #923: URL: https://github.com/apache/flink-agents/pull/923#issuecomment-5066894239
One ActionStateStore replay semantic seems worth making explicit before merging. When a completed action result is reused, the runtime returns the stored output Events and rebinds their lineage to the current triggering Event while preserving the output Event IDs. The resulting behavior is: ```text first execution: E1 -> A -> O1 reused execution: E2 -> A(reused) -> O1 ``` I think this is a reasonable semantic, but it is currently implicit. Could we document it and add a test that asserts `O1.id` is preserved while its `upstreamEventId` changes from `E1` to `E2`? This also establishes that the same logical Event ID may appear in multiple Event Log records across reuse, which reconstruction needs to tolerate. -- 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]
