HHH-13576 Similar debugf and tracef issues found via grepping

This commit is contained in:
Sanne Grinovero 2019-08-14 11:31:32 +01:00
parent e110ab0030
commit 6b489474f5
9 changed files with 24 additions and 22 deletions

View File

@ -83,13 +83,13 @@ public class XmlMappingBinderAccess {
xmlInputStream.close();
}
catch (IOException e) {
LOG.debugf( "Unable to close InputStream obtained from InputStreamAccess : " + xmlInputStreamAccess.getStreamName() );
LOG.debugf( "Unable to close InputStream obtained from InputStreamAccess : %s", xmlInputStreamAccess.getStreamName() );
}
}
}
public Binding bind(InputStream xmlInputStream) {
LOG.tracef( "reading mappings from InputStream" );
LOG.trace( "reading mappings from InputStream" );
final Origin origin = new Origin( SourceType.INPUT_STREAM, null );
return new InputStreamXmlSource( origin, xmlInputStream, false ).doBind( getMappingBinder() );
}

View File

@ -76,7 +76,7 @@ public final class Settings {
LOG.debugf( "Check Nullability in Core (should be disabled when Bean Validation is on): %s", enabledDisabled( sessionFactoryOptions.isCheckNullability() ) );
LOG.debugf( "Allow initialization of lazy state outside session : %s", enabledDisabled( sessionFactoryOptions.isInitializeLazyStateOutsideTransactionsEnabled() ) );
LOG.debugf( "Using BatchFetchStyle : " + sessionFactoryOptions.getBatchFetchStyle().name() );
LOG.debugf( "Using BatchFetchStyle : %", sessionFactoryOptions.getBatchFetchStyle().name() );
LOG.debugf( "Default batch fetch size: %s", sessionFactoryOptions.getDefaultBatchFetchSize() );
LOG.debugf( "Maximum outer join fetch depth: %s", sessionFactoryOptions.getMaximumFetchDepth() );
LOG.debugf( "Default null ordering: %s", sessionFactoryOptions.getDefaultNullPrecedence() );

View File

@ -40,12 +40,12 @@ public class JtaPlatformInitiator implements StandardServiceInitiator<JtaPlatfor
JtaPlatform platform = registry.getService( StrategySelector.class ).resolveStrategy( JtaPlatform.class, setting );
if ( platform == null ) {
LOG.debugf( "No JtaPlatform was specified, checking resolver" );
LOG.debug( "No JtaPlatform was specified, checking resolver" );
platform = registry.getService( JtaPlatformResolver.class ).resolveJtaPlatform( configurationValues, registry );
}
if ( platform == null ) {
LOG.debugf( "No JtaPlatform was specified, checking resolver" );
LOG.debug( "No JtaPlatform was specified, checking resolver" );
platform = getFallbackProvider( configurationValues, registry );
}

View File

@ -300,7 +300,7 @@ public class DefaultLoadEventListener implements LoadEventListener {
// entities with subclasses that define a ProxyFactory can create
// a HibernateProxy so long as NO_PROXY was not specified.
if ( event.getShouldUnwrapProxy() != null && event.getShouldUnwrapProxy() ) {
LOG.debugf( "Ignoring NO_PROXY for to-one association with subclasses to honor laziness" );
LOG.debug( "Ignoring NO_PROXY for to-one association with subclasses to honor laziness" );
}
return createProxy( event, persister, keyToLoad, persistenceContext );
}

View File

@ -35,15 +35,15 @@ public class EntityCopyObserverFactoryInitiator implements StandardServiceInitia
public EntityCopyObserverFactory initiateService(final Map configurationValues, final ServiceRegistryImplementor registry) {
final Object value = getConfigurationValue( configurationValues );
if ( value.equals( EntityCopyNotAllowedObserver.SHORT_NAME ) || value.equals( EntityCopyNotAllowedObserver.class.getName() ) ) {
LOG.debugf( "Configured EntityCopyObserver strategy: " + EntityCopyNotAllowedObserver.SHORT_NAME );
LOG.debugf( "Configured EntityCopyObserver strategy: %s", EntityCopyNotAllowedObserver.SHORT_NAME );
return EntityCopyNotAllowedObserver.FACTORY_OF_SELF;
}
else if ( value.equals( EntityCopyAllowedObserver.SHORT_NAME ) || value.equals( EntityCopyAllowedObserver.class.getName() ) ) {
LOG.debugf( "Configured EntityCopyObserver strategy: " + EntityCopyAllowedObserver.SHORT_NAME );
LOG.debugf( "Configured EntityCopyObserver strategy: %s", EntityCopyAllowedObserver.SHORT_NAME );
return EntityCopyAllowedObserver.FACTORY_OF_SELF;
}
else if ( value.equals( EntityCopyAllowedLoggedObserver.SHORT_NAME ) || value.equals( EntityCopyAllowedLoggedObserver.class.getName() ) ) {
LOG.debugf( "Configured EntityCopyObserver strategy: " + EntityCopyAllowedLoggedObserver.SHORT_NAME );
LOG.debugf( "Configured EntityCopyObserver strategy: %s", EntityCopyAllowedLoggedObserver.SHORT_NAME );
return EntityCopyAllowedLoggedObserver.FACTORY_OF_SELF;
}
else {
@ -52,7 +52,7 @@ public class EntityCopyObserverFactoryInitiator implements StandardServiceInitia
//and that they are indeed of the right type.
EntityCopyObserver exampleInstance = registry.getService( StrategySelector.class ).resolveStrategy( EntityCopyObserver.class, value );
Class observerType = exampleInstance.getClass();
LOG.debugf( "Configured EntityCopyObserver is a custom implementation of type " + observerType.getName() );
LOG.debugf( "Configured EntityCopyObserver is a custom implementation of type %s", observerType.getName() );
return new EntityObserversFactoryFromClass( observerType );
}
}

View File

@ -105,7 +105,7 @@ public final class IteratorImpl implements HibernateIterator {
try {
boolean isHolder = holderInstantiator.isRequired();
LOG.debugf( "Assembling results" );
LOG.debug( "Assembling results" );
if ( single && !isHolder ) {
currentResult = types[0].nullSafeGet( rs, names[0], session, null );
}
@ -124,7 +124,7 @@ public final class IteratorImpl implements HibernateIterator {
}
postNext();
LOG.debugf( "Returning current results" );
LOG.debug( "Returning current results" );
return currentResult;
}
catch (SQLException sqle) {

View File

@ -155,7 +155,7 @@ public class SessionFactoryRegistry {
final SessionFactory sessionFactory = sessionFactoryMap.get( uuid );
if ( sessionFactory == null && LOG.isDebugEnabled() ) {
LOG.debugf( "Not found: %s", uuid );
LOG.debugf( sessionFactoryMap.toString() );
LOG.debug( sessionFactoryMap.toString() );
}
return sessionFactory;
}

View File

@ -37,7 +37,7 @@ public class OrderByFragmentTranslator {
* @return The translation.
*/
public static OrderByTranslation translate(TranslationContext context, String fragment) {
LOG.tracef( "Beginning parsing of order-by fragment : " + fragment );
LOG.tracef( "Beginning parsing of order-by fragment : ", fragment );
GeneratedOrderByLexer lexer = new GeneratedOrderByLexer( new StringReader( fragment ) );

View File

@ -108,7 +108,7 @@ public class HibernatePersistenceProviderAdaptor implements PersistenceProviderA
if ( Classification.NONE.equals( platform.defaultCacheClassification() ) ) {
if ( !SharedCacheMode.NONE.equals( pu.getSharedCacheMode() ) ) {
JPA_LOGGER.tracef( "second level cache is not supported in platform, ignoring shared cache mode" );
JPA_LOGGER.trace( "second level cache is not supported in platform, ignoring shared cache mode" );
}
pu.setSharedCacheMode( SharedCacheMode.NONE );
}
@ -127,13 +127,15 @@ public class HibernatePersistenceProviderAdaptor implements PersistenceProviderA
JPA_LOGGER.tracef( "second level cache enabled for %s", pu.getScopedPersistenceUnitName() );
}
else {
JPA_LOGGER.tracef(
"second level cache disabled for %s, pu %s property = %s, pu.getSharedCacheMode = %s",
pu.getScopedPersistenceUnitName(),
AvailableSettings.JPA_SHARED_CACHE_MODE,
sharedCacheMode,
pu.getSharedCacheMode().toString()
);
if ( JPA_LOGGER.isTraceEnabled() ) {
JPA_LOGGER.tracef(
"second level cache disabled for %s, pu %s property = %s, pu.getSharedCacheMode = %s",
pu.getScopedPersistenceUnitName(),
AvailableSettings.JPA_SHARED_CACHE_MODE,
sharedCacheMode,
pu.getSharedCacheMode().toString()
);
}
}
}