funky-eyes commented on issue #7954: URL: https://github.com/apache/incubator-seata/issues/7954#issuecomment-4656645983
> Hi👋, I’d like to check whether there is any progress on this issue. I’m willing to help follow it up. > > From a practical perspective, I see two levels of solution: > > 1. Keep the current mechanism, but make it clear in the docs/API that setUpdated(true) must be called explicitly after modifying actionContext. > 2. Add a clearer API or auto-mark updated at the BusinessActionContext mutation entry, so common cases work out of the box. > > If we also want to support cases where an object is stored in actionContext and later its internal fields are modified, then we’d need a stronger proxy/dirty-tracking mechanism. Otherwise, the more realistic approach is to explicitly update the context or re-put a snapshot value. > > I’d like to work on this issue. Could you please assign it to me? Of course! Looking forward to your 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
