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

timoninmaxim pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ignite.git


The following commit(s) were added to refs/heads/master by this push:
     new b6b8cfcdadb IGNITE-21485 : Refactoring of lost code and codestyle 
related to GridCacheVersion (#11236)
b6b8cfcdadb is described below

commit b6b8cfcdadbd46e46de6d4c0d8af33323e4748c6
Author: Vladimir Steshin <[email protected]>
AuthorDate: Fri Feb 9 17:13:14 2024 +0300

    IGNITE-21485 : Refactoring of lost code and codestyle related to 
GridCacheVersion (#11236)
---
 .../dht/atomic/DhtAtomicUpdateResult.java            | 20 --------------------
 .../preloader/GridDhtPartitionsExchangeFuture.java   |  2 +-
 .../cache/distributed/near/GridNearGetResponse.java  |  3 +--
 .../processors/cache/version/GridCacheVersion.java   |  5 +++++
 .../cache/version/GridCacheVersionManager.java       |  9 ---------
 5 files changed, 7 insertions(+), 32 deletions(-)

diff --git 
a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/atomic/DhtAtomicUpdateResult.java
 
b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/atomic/DhtAtomicUpdateResult.java
index 15db625dc46..c265bf3edd3 100644
--- 
a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/atomic/DhtAtomicUpdateResult.java
+++ 
b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/atomic/DhtAtomicUpdateResult.java
@@ -50,26 +50,6 @@ class DhtAtomicUpdateResult {
      */
     private int processedEntriesCount;
 
-    /**
-     *
-     */
-    DhtAtomicUpdateResult() {
-        // No-op.
-    }
-
-    /**
-     * @param retVal Return value.
-     * @param deleted Deleted entries.
-     * @param dhtFut DHT update future.
-     */
-    DhtAtomicUpdateResult(GridCacheReturn retVal,
-        Collection<IgniteBiTuple<GridDhtCacheEntry, GridCacheVersion>> deleted,
-        GridDhtAtomicAbstractUpdateFuture dhtFut) {
-        this.retVal = retVal;
-        this.deleted = deleted;
-        this.dhtFut = dhtFut;
-    }
-
     /**
      * @param expiry Expiry policy.
      */
diff --git 
a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/preloader/GridDhtPartitionsExchangeFuture.java
 
b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/preloader/GridDhtPartitionsExchangeFuture.java
index c1e49d21107..ede7f87973d 100644
--- 
a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/preloader/GridDhtPartitionsExchangeFuture.java
+++ 
b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/preloader/GridDhtPartitionsExchangeFuture.java
@@ -2787,7 +2787,7 @@ public class GridDhtPartitionsExchangeFuture extends 
GridDhtTopologyFutureAdapte
         while (true) {
             GridCacheVersion old = lastVer.get();
 
-            if (old == null || Long.compare(old.order(), ver.order()) < 0) {
+            if (old == null || old.order() < ver.order()) {
                 if (lastVer.compareAndSet(old, ver))
                     break;
             }
diff --git 
a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearGetResponse.java
 
b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearGetResponse.java
index 578c46b6ac3..50e3dcdef4e 100644
--- 
a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearGetResponse.java
+++ 
b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/near/GridNearGetResponse.java
@@ -45,8 +45,7 @@ import org.jetbrains.annotations.NotNull;
 /**
  * Get response.
  */
-public class GridNearGetResponse extends GridCacheIdMessage implements 
GridCacheDeployable,
-    GridCacheVersionable {
+public class GridNearGetResponse extends GridCacheIdMessage implements 
GridCacheDeployable, GridCacheVersionable {
     /** */
     private static final long serialVersionUID = 0L;
 
diff --git 
a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/version/GridCacheVersion.java
 
b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/version/GridCacheVersion.java
index a54a9f5a54a..9044a15cfd2 100644
--- 
a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/version/GridCacheVersion.java
+++ 
b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/version/GridCacheVersion.java
@@ -88,6 +88,11 @@ public class GridCacheVersion implements Message, 
Externalizable, CacheEntryVers
      * @param order Version order.
      */
     public GridCacheVersion(int topVer, int nodeOrderDrId, long order) {
+        assert topVer >= 0 : topVer;
+        assert order >= 0 : order;
+        assert (nodeOrderDrId >>> DR_ID_SHIFT) >= 0 : (nodeOrderDrId >>> 
DR_ID_SHIFT);
+        assert (nodeOrderDrId & NODE_ORDER_MASK) >= 0 : (nodeOrderDrId & 
NODE_ORDER_MASK);
+
         this.topVer = topVer;
         this.nodeOrderDrId = nodeOrderDrId;
         this.order = order;
diff --git 
a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/version/GridCacheVersionManager.java
 
b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/version/GridCacheVersionManager.java
index ac81bf95780..2ebdf79c665 100644
--- 
a/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/version/GridCacheVersionManager.java
+++ 
b/modules/core/src/main/java/org/apache/ignite/internal/processors/cache/version/GridCacheVersionManager.java
@@ -365,13 +365,4 @@ public class GridCacheVersionManager extends 
GridCacheSharedManagerAdapter {
     public boolean isStartVersion(GridCacheVersion ver) {
         return startVer.equals(ver);
     }
-
-    /**
-     * Update grid start time.
-     */
-    public void gridStartTime(long startTime) {
-        offset = (int)((startTime - TOP_VER_BASE_TIME) / 1000);
-
-        isolatedStreamerVer = new GridCacheVersion(1 + offset, 0, 1, 
dataCenterId);
-    }
 }

Reply via email to