IGNITE-27947 Apply code-reviews

** Fixed type
** Updated javadoc comment on ClientInboundMessageHandler
** Added IGNITE-28405 ticket mention

Author: tmgodinho

modules/client-handler/src/main/java/org/apache/ignite/client/handler/ClientInboundMessageHandler.java

@@ -306,8 +306,7 @@ public class ClientInboundMessageHandler
      *
      * <p><b>Mappings are removed:</b>
      * <ul>
-     *     <li>During a rollback request.</li>
-     *     <li>After the first request response is sent to the client.</li>
+     *     <li>During a commit or rollback request. Hook is added at creation.</li>
      * </ul>
      */
     private final Map<Long, Long> firstReqToTxResMap = new ConcurrentHashMap<>();

modules/client-handler/src/main/java/org/apache/ignite/client/handler/requests/table/DirectTransactionWithFirstRequest.java

@@ -32,7 +32,7 @@
 class DirectTransactionWithFirstRequest implements InternalTransaction, Wrapper {
     private final InternalTransaction base;
 
-    // We could alos just accept a lambda.
+    // We could also just accept a lambda.
     private final Map<Long, Long> reqToTxMap;
 
     private final long firstReqId;

modules/client/src/main/java/org/apache/ignite/internal/client/tx/ClientLazyTransaction.java

@@ -100,7 +100,7 @@ public void rollback() throws TransactionException {
     public CompletableFuture<Void> rollbackAsync() {
         var tx0 = tx;
 
-        // This is really fishy. It will probably let you reuse a transaction after calling a rollback :(
+        // TODO: IGNITE-28405 This is really fishy. It will probably let you reuse a transaction after calling a rollback :(
         if (tx0 == null) {
             // No operations were performed, nothing to rollback.
             return nullCompletedFuture();