add since to some @deprecated operations and delete a couple

gavinking · gavinking · commit f1d7d86c2d1e · 2024-08-29T19:56:30.000+02:00
Signed-off-by: Gavin King &lt;gavin@hibernate.org&gt;
diff --git a/hibernate-core/src/main/java/org/hibernate/annotations/Cache.java b/hibernate-core/src/main/java/org/hibernate/annotations/Cache.java
@@ -103,6 +103,6 @@
 	 *
 	 * @deprecated Use {@link #includeLazy()} for the sake of typesafety.
 	 */
-	@Deprecated
+	@Deprecated(since="6.4")
 	String include() default "all";
 }
diff --git a/hibernate-core/src/main/java/org/hibernate/annotations/QueryHints.java b/hibernate-core/src/main/java/org/hibernate/annotations/QueryHints.java
@@ -99,7 +99,7 @@ private QueryHints() {
 	 * Use {@link org.hibernate.procedure.ProcedureCall} or
 	 * {@link jakarta.persistence.StoredProcedureQuery} instead.
 	 */
-	@Deprecated
+	@Deprecated(since="6")
 	public static final String CALLABLE_FUNCTION = HibernateHints.HINT_CALLABLE_FUNCTION;
 
 	/**
diff --git a/hibernate-core/src/main/java/org/hibernate/boot/archive/scan/spi/ScanOptions.java b/hibernate-core/src/main/java/org/hibernate/boot/archive/scan/spi/ScanOptions.java
@@ -37,6 +37,6 @@ public interface ScanOptions {
 	 * @deprecated With move to unified schema, this setting is now deprecated and will
 	 * be removed once support for reading {@code hbm.xml} files is fully removed.
 	 */
-	@Deprecated
+	@Deprecated(since="5")
 	boolean canDetectHibernateMappingFiles();
 }
diff --git a/hibernate-core/src/main/java/org/hibernate/boot/internal/MetadataBuilderImpl.java b/hibernate-core/src/main/java/org/hibernate/boot/internal/MetadataBuilderImpl.java
@@ -69,7 +69,6 @@
 import org.hibernate.dialect.Dialect;
 import org.hibernate.dialect.TimeZoneSupport;
 import org.hibernate.engine.config.spi.ConfigurationService;
-import org.hibernate.engine.config.spi.StandardConverters;
 import org.hibernate.engine.jdbc.env.internal.JdbcEnvironmentImpl;
 import org.hibernate.engine.jdbc.spi.JdbcServices;
 import org.hibernate.internal.CoreLogging;
diff --git a/hibernate-core/src/main/java/org/hibernate/boot/model/internal/AnnotatedColumns.java b/hibernate-core/src/main/java/org/hibernate/boot/model/internal/AnnotatedColumns.java
@@ -140,14 +140,6 @@ public void setTable(Table table) {
 		this.table = table;
 	}
 
-	/**
-	 * @deprecated Use {@link #setTable(Table)} instead
-	 */
-	@Deprecated
-	void setTableInternal(Table table) {
-		this.table = table;
-	}
-
 	public void addColumn(AnnotatedColumn child) {
 		columns.add( child );
 	}
diff --git a/hibernate-core/src/main/java/org/hibernate/boot/model/relational/SqlStringGenerationContext.java b/hibernate-core/src/main/java/org/hibernate/boot/model/relational/SqlStringGenerationContext.java
@@ -8,7 +8,6 @@
 
 import org.hibernate.boot.model.naming.Identifier;
 import org.hibernate.dialect.Dialect;
-import org.hibernate.engine.jdbc.env.spi.IdentifierHelper;
 
 /**
  * A context provided to methods responsible for generating SQL strings on startup.
@@ -21,16 +20,6 @@ public interface SqlStringGenerationContext {
 	 */
 	Dialect getDialect();
 
-	/**
-	 * @return The helper for dealing with identifiers in the current JDBC environment.
-	 * <p>
-	 * Note that the Identifiers returned from this helper already account for auto-quoting.
-	 *
-	 * @deprecated Use {@link #toIdentifier(String)} instead.
-	 */
-	@Deprecated
-	IdentifierHelper getIdentifierHelper();
-
 	/**
 	 * Generate an Identifier instance from its simple name as obtained from mapping
 	 * information.
diff --git a/hibernate-core/src/main/java/org/hibernate/boot/model/relational/internal/SqlStringGenerationContextImpl.java b/hibernate-core/src/main/java/org/hibernate/boot/model/relational/internal/SqlStringGenerationContextImpl.java
@@ -138,11 +138,6 @@ public Dialect getDialect() {
 		return dialect;
 	}
 
-	@Override
-	public IdentifierHelper getIdentifierHelper() {
-		return identifierHelper;
-	}
-
 	@Override
 	public Identifier toIdentifier(String text) {
 		return identifierHelper != null ? identifierHelper.toIdentifier( text ) : Identifier.toIdentifier( text );
diff --git a/hibernate-core/src/main/java/org/hibernate/bytecode/internal/BytecodeProviderInitiator.java b/hibernate-core/src/main/java/org/hibernate/bytecode/internal/BytecodeProviderInitiator.java
@@ -17,28 +17,14 @@
 import org.hibernate.bytecode.spi.BytecodeProvider;
 import org.hibernate.service.spi.ServiceRegistryImplementor;
 
-import static org.hibernate.internal.util.NullnessUtil.castNonNull;
-
 public final class BytecodeProviderInitiator implements StandardServiceInitiator<BytecodeProvider> {
 
 	/**
 	 * @deprecated Register a {@link BytecodeProvider} through Java {@linkplain java.util.ServiceLoader services}.
 	 */
-	@Deprecated( forRemoval = true )
+	@Deprecated( forRemoval = true, since = "6.2" )
 	public static final String BYTECODE_PROVIDER_NAME_BYTEBUDDY = "bytebuddy";
 
-	/**
-	 * @deprecated Register a {@link BytecodeProvider} through Java {@linkplain java.util.ServiceLoader services}.
-	 */
-	@Deprecated( forRemoval = true )
-	public static final String BYTECODE_PROVIDER_NAME_NONE = "none";
-
-	/**
-	 * @deprecated Deprecated with no replacement
-	 */
-	@Deprecated( forRemoval = true )
-	public static final String BYTECODE_PROVIDER_NAME_DEFAULT = BYTECODE_PROVIDER_NAME_BYTEBUDDY;
-
 	/**
 	 * Singleton access
 	 */
diff --git a/hibernate-core/src/main/java/org/hibernate/cache/RegionFactory.java b/hibernate-core/src/main/java/org/hibernate/cache/RegionFactory.java
@@ -13,6 +13,6 @@
  *
  * @deprecated Moved, but still need this definition for ehcache 
  */
-@Deprecated
+@Deprecated(since="4")
 public interface RegionFactory extends org.hibernate.cache.spi.RegionFactory {
 }
diff --git a/hibernate-core/src/main/java/org/hibernate/cfg/AvailableSettings.java b/hibernate-core/src/main/java/org/hibernate/cfg/AvailableSettings.java
@@ -61,14 +61,14 @@ public interface AvailableSettings
 	/**
 	 * @deprecated Use {@link #JAKARTA_LOCK_SCOPE} instead
 	 */
-	@Deprecated
+	@Deprecated(since="6.3")
 	@SuppressWarnings("DeprecatedIsStillUsed")
 	String JPA_LOCK_SCOPE = LegacySpecHints.HINT_JAVAEE_LOCK_SCOPE;
 
 	/**
 	 * @deprecated Use {@link #JAKARTA_LOCK_TIMEOUT} instead
 	 */
-	@Deprecated
+	@Deprecated(since="6.3")
 	@SuppressWarnings("DeprecatedIsStillUsed")
 	String JPA_LOCK_TIMEOUT = LegacySpecHints.HINT_JAVAEE_LOCK_TIMEOUT;
 
@@ -137,7 +137,7 @@ public interface AvailableSettings
 	 *
 	 * @since 5.2
 	 */
-	@Deprecated(forRemoval = true)
+	@Deprecated(since="6.6", forRemoval = true)
 	String ALLOW_REFRESH_DETACHED_ENTITY = "hibernate.allow_refresh_detached_entity";
 
 	/**
diff --git a/hibernate-core/src/main/java/org/hibernate/cfg/BatchSettings.java b/hibernate-core/src/main/java/org/hibernate/cfg/BatchSettings.java
@@ -68,6 +68,6 @@ public interface BatchSettings {
 	/**
 	 * @deprecated Use {@link #BUILDER} instead
 	 */
-	@Deprecated
+	@Deprecated(since="6.4")
 	String BATCH_STRATEGY = "hibernate.jdbc.factory_class";
 }
diff --git a/hibernate-core/src/main/java/org/hibernate/dialect/Dialect.java b/hibernate-core/src/main/java/org/hibernate/dialect/Dialect.java
@@ -852,7 +852,7 @@ public String getCheckCondition(String columnName, long min, long max) {
 	 * @return a SQL expression that will occur in a {@code check} constraint
 	 * @deprecated use {@link #getCheckCondition(String, Long[])} instead
 	 */
-	@Deprecated(forRemoval = true)
+	@Deprecated(since="6.5", forRemoval = true)
 	public String getCheckCondition(String columnName, long[] values) {
 		Long[] boxedValues = new Long[values.length];
 		for ( int i = 0; i<values.length; i++ ) {
diff --git a/hibernate-core/src/main/java/org/hibernate/dialect/MySQLDialect.java b/hibernate-core/src/main/java/org/hibernate/dialect/MySQLDialect.java
@@ -204,7 +204,7 @@ public MySQLDialect(DialectResolutionInfo info) {
 		registerKeywords( info );
 	}
 
-	@Deprecated
+	@Deprecated(since="6.6")
 	protected static DatabaseVersion createVersion(DialectResolutionInfo info) {
 		return createVersion( info, MINIMUM_VERSION );
 	}
@@ -443,7 +443,7 @@ protected void registerColumnTypes(TypeContributions typeContributions, ServiceR
 		ddlTypeRegistry.addDescriptor( new NativeOrdinalEnumDdlTypeImpl( this ) );
 	}
 
-	@Deprecated
+	@Deprecated(since="6.4")
 	protected static int getCharacterSetBytesPerCharacter(DatabaseMetaData databaseMetaData) {
 		if ( databaseMetaData != null ) {
 			try (java.sql.Statement s = databaseMetaData.getConnection().createStatement() ) {
diff --git a/hibernate-core/src/main/java/org/hibernate/dialect/identity/DB2390IdentityColumnSupport.java b/hibernate-core/src/main/java/org/hibernate/dialect/identity/DB2390IdentityColumnSupport.java
@@ -10,6 +10,6 @@
  * @author Jan Schatteman
  * @deprecated use {@code DB2zIdentityColumnSupport}
  */
-@Deprecated
+@Deprecated(since="6.4")
 public class DB2390IdentityColumnSupport extends DB2zIdentityColumnSupport {
 }
diff --git a/hibernate-core/src/main/java/org/hibernate/dialect/sequence/LegacyDB2SequenceSupport.java b/hibernate-core/src/main/java/org/hibernate/dialect/sequence/LegacyDB2SequenceSupport.java
@@ -15,7 +15,7 @@
  *
  * @deprecated use {@code DB2SequenceSupport}
  */
-@Deprecated
+@Deprecated(since="6.4")
 public class LegacyDB2SequenceSupport implements SequenceSupport {
 
 	public static final SequenceSupport INSTANCE = new LegacyDB2SequenceSupport();
diff --git a/hibernate-core/src/main/java/org/hibernate/query/NullPrecedence.java b/hibernate-core/src/main/java/org/hibernate/query/NullPrecedence.java
@@ -19,7 +19,7 @@
  *
  * @deprecated Use Jakarta Persistence {@linkplain Nulls} instead.
  */
-@Deprecated
+@Deprecated(since="7")
 public enum NullPrecedence {
 	/**
 	 * Null precedence not specified. Relies on the RDBMS implementation.
diff --git a/hibernate-core/src/main/java/org/hibernate/type/descriptor/java/JavaType.java b/hibernate-core/src/main/java/org/hibernate/type/descriptor/java/JavaType.java
@@ -319,28 +319,15 @@ default <X> T coerce(X value, CoercionContext coercionContext) {
 		return (T) value;
 	}
 
-	/**
-	 * Creates the {@link JavaType} for the given {@link ParameterizedType}
-	 * based on this {@link JavaType} registered for the raw type.
-	 *
-	 * @deprecated Use {@link #createJavaType(ParameterizedType, TypeConfiguration)} instead
-	 */
-	@Deprecated(since = "6.1")
-	default JavaType<T> createJavaType(ParameterizedType parameterizedType) {
-		return this;
-	}
-
 	/**
 	 * Creates the {@link JavaType} for the given {@link ParameterizedType}
 	 * based on this {@link JavaType} registered for the raw type.
 	 *
 	 * @since 6.1
 	 */
 	@Incubating
-	default JavaType<T> createJavaType(
-			ParameterizedType parameterizedType,
-			TypeConfiguration typeConfiguration) {
-		return createJavaType( parameterizedType );
+	default JavaType<T> createJavaType(ParameterizedType parameterizedType, TypeConfiguration typeConfiguration) {
+		return this;
 	}
 
 	/**
diff --git a/hibernate-core/src/main/java/org/hibernate/type/descriptor/jdbc/InstantAsTimestampJdbcType.java b/hibernate-core/src/main/java/org/hibernate/type/descriptor/jdbc/InstantAsTimestampJdbcType.java
@@ -14,7 +14,7 @@
  * @deprecated Use {@link TimestampUtcAsJdbcTimestampJdbcType}
  * @author Christian Beikov
  */
-@Deprecated(forRemoval = true)
+@Deprecated(since="6.4", forRemoval = true)
 public class InstantAsTimestampJdbcType extends TimestampUtcAsJdbcTimestampJdbcType {
 	public static final InstantAsTimestampJdbcType INSTANCE = new InstantAsTimestampJdbcType();
 }
diff --git a/hibernate-core/src/main/java/org/hibernate/type/descriptor/jdbc/InstantAsTimestampWithTimeZoneJdbcType.java b/hibernate-core/src/main/java/org/hibernate/type/descriptor/jdbc/InstantAsTimestampWithTimeZoneJdbcType.java
@@ -14,7 +14,7 @@
  * @deprecated Use {@link TimestampUtcAsOffsetDateTimeJdbcType}
  * @author Christian Beikov
  */
-@Deprecated(forRemoval = true)
+@Deprecated(since="6.4", forRemoval = true)
 public class InstantAsTimestampWithTimeZoneJdbcType extends TimestampUtcAsOffsetDateTimeJdbcType {
 	public static final InstantAsTimestampWithTimeZoneJdbcType INSTANCE = new InstantAsTimestampWithTimeZoneJdbcType();
 }
diff --git a/hibernate-core/src/test/java/org/hibernate/orm/test/dialect/PostgreSQLDialectTestCase.java b/hibernate-core/src/test/java/org/hibernate/orm/test/dialect/PostgreSQLDialectTestCase.java
@@ -23,7 +23,6 @@
 import org.hibernate.dialect.PostgreSQLDialect;
 import org.hibernate.QueryTimeoutException;
 import org.hibernate.dialect.unique.AlterTableUniqueDelegate;
-import org.hibernate.engine.jdbc.env.spi.IdentifierHelper;
 import org.hibernate.exception.LockAcquisitionException;
 import org.hibernate.exception.spi.SQLExceptionConversionDelegate;
 import org.hibernate.mapping.Table;
@@ -160,11 +159,6 @@ public Dialect getDialect() {
 			return null;
 		}
 
-		@Override
-		public IdentifierHelper getIdentifierHelper() {
-			return null;
-		}
-
 		@Override
 		public Identifier toIdentifier(String text) {
 			return null;

Original file line number	Diff line number	Diff line change
`@@ -103,6 +103,6 @@`
`103`	`103`	`*`
`104`	`104`	`* @deprecated Use {@link #includeLazy()} for the sake of typesafety.`
`105`	`105`	`*/`
`106`		`- @Deprecated`
	`106`	`+ @Deprecated(since="6.4")`
`107`	`107`	`String include() default "all";`
`108`	`108`	`}`
Original file line number	Diff line number	Diff line change
`@@ -37,6 +37,6 @@ public interface ScanOptions {`
`37`	`37`	`* @deprecated With move to unified schema, this setting is now deprecated and will`
`38`	`38`	`* be removed once support for reading {@code hbm.xml} files is fully removed.`
`39`	`39`	`*/`
`40`		`- @Deprecated`
	`40`	`+ @Deprecated(since="5")`
`41`	`41`	`boolean canDetectHibernateMappingFiles();`
`42`	`42`	`}`
Original file line number	Diff line number	Diff line change
`@@ -140,14 +140,6 @@ public void setTable(Table table) {`
`140`	`140`	`this.table = table;`
`141`	`141`	`}`
`142`	`142`
`143`		`- /**`
`144`		`- * @deprecated Use {@link #setTable(Table)} instead`
`145`		`- */`
`146`		`- @Deprecated`
`147`		`- void setTableInternal(Table table) {`
`148`		`- this.table = table;`
`149`		`- }`
`150`		`-`
`151`	`143`	`public void addColumn(AnnotatedColumn child) {`
`152`	`144`	`columns.add( child );`
`153`	`145`	`}`
Original file line number	Diff line number	Diff line change
`@@ -13,6 +13,6 @@`
`13`	`13`	`*`
`14`	`14`	`* @deprecated Moved, but still need this definition for ehcache`
`15`	`15`	`*/`
`16`		`-@Deprecated`
	`16`	`+@Deprecated(since="4")`
`17`	`17`	`public interface RegionFactory extends org.hibernate.cache.spi.RegionFactory {`
`18`	`18`	`}`