remove unnecessary logging

Author: gavinking

hibernate-core/src/main/java/org/hibernate/engine/jdbc/env/internal/NormalizingIdentifierHelperImpl.java

@@ -15,13 +15,10 @@
 import org.hibernate.engine.jdbc.env.spi.JdbcEnvironment;
 import org.hibernate.engine.jdbc.env.spi.NameQualifierSupport;
 
-import org.jboss.logging.Logger;
-
 /**
 * @author Steve Ebersole
 */
 public class NormalizingIdentifierHelperImpl implements IdentifierHelper {
-	private static final Logger log = Logger.getLogger( NormalizingIdentifierHelperImpl.class );
 
 	private final JdbcEnvironment jdbcEnvironment;
 
@@ -57,8 +54,6 @@ public NormalizingIdentifierHelperImpl(
 
 	@Override
 	public Identifier normalizeQuoting(Identifier identifier) {
-		log.tracef( "Normalizing identifier quoting [%s]", identifier );
-
 		if ( identifier == null ) {
 			return null;
 		}
@@ -68,17 +63,14 @@ public Identifier normalizeQuoting(Identifier identifier) {
 		}
 
 		if ( globallyQuoteIdentifiers ) {
-			log.tracef( "Forcing identifier [%s] to quoted for global quoting", identifier );
 			return Identifier.toIdentifier( identifier.getText(), true );
 		}
 
 		if ( autoQuoteKeywords && isReservedWord( identifier.getText() ) ) {
-			log.tracef( "Forcing identifier [%s] to quoted as recognized reserved word", identifier );
 			return Identifier.toIdentifier( identifier.getText(), true );
 		}
 
 		if ( autoQuoteInitialUnderscore && identifier.getText().startsWith("_") ) {
-			log.tracef( "Forcing identifier [%s] to quoted due to initial underscore", identifier );
 			return Identifier.toIdentifier( identifier.getText(), true );
 		}
 
@@ -110,10 +102,7 @@ public boolean isReservedWord(String word) {
 
 	@Override
 	public String toMetaDataCatalogName(Identifier identifier) {
-		log.tracef( "Normalizing identifier quoting for catalog name [%s]", identifier );
-
 		if ( !nameQualifierSupport.supportsCatalogs() ) {
-			log.trace( "Environment does not support catalogs; returning null" );
 			// null is used to tell DatabaseMetaData to not limit results based on catalog.
 			return null;
 		}
@@ -139,50 +128,24 @@ private String toMetaDataText(Identifier identifier) {
 		}
 		else if ( identifier.isQuoted() ) {
 			return switch ( quotedCaseStrategy ) {
-				case UPPER -> {
-					log.tracef( "Rendering quoted identifier [%s] in upper case for use in DatabaseMetaData",
-							identifier );
-					yield text.toUpperCase( Locale.ROOT );
-				}
-				case LOWER -> {
-					log.tracef( "Rendering quoted identifier [%s] in lower case for use in DatabaseMetaData",
-							identifier );
-					yield text.toLowerCase( Locale.ROOT );
-				}
-				default -> {
-					// default is mixed case
-					log.tracef( "Rendering quoted identifier [%s] in mixed case for use in DatabaseMetaData",
-							identifier );
-					yield text;
-				}
+				case UPPER -> text.toUpperCase( Locale.ROOT );
+				case LOWER -> text.toLowerCase( Locale.ROOT );
+				case MIXED -> text; // default
 			};
 		}
 		else {
 			return switch ( unquotedCaseStrategy ) {
-				case MIXED -> {
-					log.tracef( "Rendering unquoted identifier [%s] in mixed case for use in DatabaseMetaData", identifier );
-					yield text;
-				}
-				case LOWER -> {
-					log.tracef( "Rendering unquoted identifier [%s] in lower case for use in DatabaseMetaData", identifier );
-					yield text.toLowerCase( Locale.ROOT );
-				}
-				default -> {
-					// default is upper case
-					log.tracef( "Rendering unquoted identifier [%s] in upper case for use in DatabaseMetaData", identifier );
-					yield text.toUpperCase( Locale.ROOT );
-				}
+				case MIXED -> text;
+				case LOWER -> text.toLowerCase( Locale.ROOT );
+				case UPPER -> text.toUpperCase( Locale.ROOT ); // default
 			};
 		}
 	}
 
 	@Override
 	public String toMetaDataSchemaName(Identifier identifier) {
-		log.tracef( "Normalizing identifier quoting for schema name [%s]", identifier );
-
 		if ( !nameQualifierSupport.supportsSchemas() ) {
 			// null is used to tell DatabaseMetaData to not limit results based on schema.
-			log.trace( "Environment does not support catalogs; returning null" );
 			return null;
 		}
 
@@ -198,8 +161,6 @@ public String toMetaDataSchemaName(Identifier identifier) {
 
 	@Override
 	public String toMetaDataObjectName(Identifier identifier) {
-		log.tracef( "Normalizing identifier quoting for object name [%s]", identifier );
-
 		if ( identifier == null ) {
 			// if this method was called, the value is needed
 			throw new IllegalArgumentException( "null was passed as an object name" );