ascherbakoff commented on code in PR #6779:
URL: https://github.com/apache/ignite-3/pull/6779#discussion_r2536582264
##########
modules/client/src/main/java/org/apache/ignite/internal/client/tx/DirectTxUtils.java:
##########
@@ -131,6 +133,10 @@ public static void writeTx(@Nullable Transaction tx,
PayloadOutputChannel out, @
out.out().packLong(tx0.observableTimestamp());
out.out().packBoolean(tx.isReadOnly());
out.out().packLong(tx0.timeout());
+ if
(ctx.channel.protocolContext().isFeatureSupported(TX_CLIENT_GETALL_SUPPORTS_PRIORITY)
Review Comment:
I've considered this approach and discarded it.
This is intentional the flag is applied only for two operations, because:
1. the chance is low
2. pr scope is already big
Passing currently useless flag to all other operations doens't look like
valid thing to me.
We can always add it later if needed.
--
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]
