Hi Roman, good catch. These methods should be in Transaction, I've fixed that.
вт, 11 окт. 2022 г. в 13:12, Roman Puchkovskiy <roman.puchkovs...@gmail.com >: > Hi Alexei. > > Great proposal, thank you for your efforts. > > One thing that I noted is that the proposed IgniteInstructions > interface contains parameterless commit*() and rollback*() methods. > The transactions are not thread-bound, so there seems to be no > implicit mechanism for the global IgniteInstructions to know about the > current transaction, hence it looks like either these methods need to > belong to the Transaction interface, or that they need Transaction to > be passed as a parameter. Or, if there is still some other mechanism, > I missed it while reading. > > пн, 26 сент. 2022 г. в 16:32, Alexei Scherbakov < > alexey.scherbak...@gmail.com>: > > > > Hello, igniters. > > > > After a long and hard work I've finally pushed the IEP > > < > https://cwiki.apache.org/confluence/display/IGNITE/IEP-91%3A+Transaction+protocol > > > > to > > the necessary detail level. > > Any useful feedback would be greatly appreciated. > > > > -- > > > > Best regards, > > Alexei Scherbakov > -- Best regards, Alexei Scherbakov
