HHH-19551 - Address deficiencies in pessimistic locking

Author: sebersole

hibernate-community-dialects/src/main/java/org/hibernate/community/dialect/CacheDialect.java

@@ -327,6 +327,7 @@ protected LockingStrategy buildPessimisticWriteStrategy(EntityPersister lockable
 		return new PessimisticWriteUpdateLockingStrategy( lockable, lockMode );
 	}
 
+	@Override
 	protected LockingStrategy buildPessimisticReadStrategy(EntityPersister lockable, LockMode lockMode, Locking.Scope lockScope) {
 		// InterSystems Cache' does not current support "SELECT ... FOR UPDATE" syntax...
 		// Set your transaction mode to READ_COMMITTED before using

hibernate-community-dialects/src/main/java/org/hibernate/community/dialect/HSQLLegacyDialect.java

@@ -759,6 +759,7 @@ public boolean doesRoundTemporalOnOverflow() {
 		return false;
 	}
 
+	@Override
 	protected LockingStrategy buildReadStrategy(EntityPersister lockable, LockMode lockMode, Locking.Scope lockScope) {
 		return getVersion().isBefore( 2 )
 				? new ReadUncommittedLockingStrategy( lockable, lockMode )

hibernate-community-dialects/src/main/java/org/hibernate/community/dialect/RDMSOS2200Dialect.java

@@ -395,6 +395,7 @@ protected LockingStrategy buildPessimisticWriteStrategy(EntityPersister lockable
 		return new PessimisticWriteUpdateLockingStrategy( lockable, lockMode );
 	}
 
+	@Override
 	protected LockingStrategy buildPessimisticReadStrategy(EntityPersister lockable, LockMode lockMode, Locking.Scope lockScope) {
 		// RDMS has no known variation of "SELECT ... FOR UPDATE" syntax...
 		return new PessimisticReadUpdateLockingStrategy( lockable, lockMode );

hibernate-community-dialects/src/main/java/org/hibernate/community/dialect/TimesTenDialect.java

@@ -395,6 +395,7 @@ protected LockingStrategy buildPessimisticWriteStrategy(EntityPersister lockable
 		return new PessimisticWriteUpdateLockingStrategy( lockable, lockMode );
 	}
 
+	@Override
 	protected LockingStrategy buildPessimisticReadStrategy(EntityPersister lockable, LockMode lockMode, Locking.Scope lockScope) {
 		// TimesTen has no known variation of a "SELECT ... FOR UPDATE" syntax...
 		return new PessimisticReadUpdateLockingStrategy( lockable, lockMode );