HHH-19935 Make sure that increment to current sequence can not be zero

Author: cigaly

hibernate-core/src/main/java/org/hibernate/id/uuid/UuidVersion7Strategy.java

@@ -54,6 +54,12 @@ public long millis() {
 			return lastTimestamp.toEpochMilli();
 		}
 
+		/**
+		 * Sub-miliseconds part of timestamp (micro- and nanoseconds) mapped to 12 bit integral value.
+		 * Calculated as nanos / 1000000 * 4096
+		 * @param timestamp
+		 * @return
+		 */
 		private static long nanos(Instant timestamp) {
 			return (long) ((timestamp.getNano() % 1_000_000L) * 0.004096);
 		}
@@ -64,9 +70,9 @@ public State getNextState() {
 				return new State( now, randomSequence() );
 			}
 			else {
-				final long nextSequence = lastSequence + Holder.numberGenerator.nextLong( 0xFFFF_FFFFL );
-				return nextSequence > MAX_RANDOM_SEQUENCE
-						? new State( lastTimestamp.plusNanos( 250 ), randomSequence() )
+				final long nextSequence = Holder.numberGenerator.nextLong( MAX_RANDOM_SEQUENCE + 1 );
+				return lastSequence >= nextSequence
+						? new State( lastTimestamp.plusNanos( 250 ), nextSequence )
 						: new State( lastTimestamp, nextSequence );
 			}
 		}
@@ -118,7 +124,7 @@ public UUID generateUuid(final SharedSessionContractImplementor session) {
 				| state.nanos() & 0xFFFL,
 				// LSB bits 0-1 - variant = 4
 				0x8000_0000_0000_0000L
-				// LSB bits 2-15 - pseudorandom counter
+				// LSB bits 2-63 - pseudorandom counter
 				| state.lastSequence
 		);
 	}