stashslash commented on issue #346: compensation method invoke before compensable method URL: https://github.com/apache/servicecomb-pack/issues/346#issuecomment-447701073 @zhfeng im not sure compensate how to catch exception from compensable, because i think only compensable itself can know if happen exception and handle COMPENSATING status by itself much better. so, like you snippets, if something wrong happens in compensable, it'll hanging COMPENSATING status, right? and why not let Omega to report COMPENSATING status around invoke compensable method, like this i think i just need `bookings.get(id).cancel();` in compensate method.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
