You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ignite.apache.org by pt...@apache.org on 2021/01/28 10:10:30 UTC

[ignite] 08/09: TODOs

This is an automated email from the ASF dual-hosted git repository.

ptupitsyn pushed a commit to branch ignite-14067
in repository https://gitbox.apache.org/repos/asf/ignite.git

commit 846067782791d0f80271e8757d059201ceb8df61
Author: Pavel Tupitsyn <pt...@apache.org>
AuthorDate: Thu Jan 28 13:08:28 2021 +0300

    TODOs
---
 .../processors/cache/distributed/dht/GridDhtTxPrepareFuture.java        | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxPrepareFuture.java b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxPrepareFuture.java
index 79d68c9..6be9eed 100644
--- a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxPrepareFuture.java
+++ b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/GridDhtTxPrepareFuture.java
@@ -1757,6 +1757,8 @@ public final class GridDhtTxPrepareFuture extends GridCacheCompoundFuture<Ignite
 
         for (IgniteTxEntry entry : entries) {
             try {
+                // TODO: Make it even better - pass lessPending to localCandidatesMax so it adds the items directly,
+                // without allocating another list.
                 Collection<GridCacheMvccCandidate> candidates = entry.cached().localCandidatesMax(baseVer);
                 for (GridCacheMvccCandidate cand : candidates) {
                     lessPending.add(cand.version());