vldpyatkov commented on code in PR #4929:
URL: https://github.com/apache/ignite-3/pull/4929#discussion_r1895382742
##########
modules/runner/src/main/java/org/apache/ignite/internal/app/IgniteImpl.java:
##########
@@ -1140,8 +1140,7 @@ public class IgniteImpl implements Ignite {
clientHandlerModule = new ClientHandlerModule(
qryEngine,
distributedTblMgr,
- // TODO: IGNITE-20232 The observable timestamp should be
different for each client.
- new IgniteTransactionsImpl(txManager, new
HybridTimestampTracker()),
+ txManager,
Review Comment:
Yes, I added a comment on the ticket.
After this patch, we won't track an observation timestamp on the server side
at all. The only exclusion is JDBC, but about it, I created another ticket
IGNITE-24053.
--
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]
