get rid of billions of obsolete WeakerAccess suppressions

This commit is contained in:
Gavin King 2022-01-23 00:53:38 +01:00
parent a4cb390552
commit e3a59f883a
165 changed files with 52 additions and 299 deletions

View File

@ -1866,7 +1866,6 @@ public class InFlightMetadataCollectorImpl implements InFlightMetadataCollector
}
}
@SuppressWarnings("WeakerAccess")
protected void secondPassCompileForeignKeys(
final Table table,
Set<ForeignKey> done,

View File

@ -94,7 +94,6 @@ public class MetadataImpl implements MetadataImplementor, Serializable {
private final Map<String, SqmFunctionDescriptor> sqlFunctionMap;
private final Database database;
@SuppressWarnings("WeakerAccess")
public MetadataImpl(
UUID uuid,
MetadataBuildingOptions metadataBuildingOptions,

View File

@ -45,7 +45,6 @@ public class SessionFactoryBuilderImpl implements SessionFactoryBuilderImplement
private final MetadataImplementor metadata;
private final SessionFactoryOptionsBuilder optionsBuilder;
@SuppressWarnings("WeakerAccess")
public SessionFactoryBuilderImpl(MetadataImplementor metadata, BootstrapContext bootstrapContext) {
this(
metadata,
@ -56,7 +55,6 @@ public class SessionFactoryBuilderImpl implements SessionFactoryBuilderImplement
);
}
@SuppressWarnings("WeakerAccess")
public SessionFactoryBuilderImpl(MetadataImplementor metadata, SessionFactoryOptionsBuilder optionsBuilder) {
this.metadata = metadata;
this.optionsBuilder = optionsBuilder;

View File

@ -145,7 +145,6 @@ import static org.hibernate.internal.log.DeprecationLogger.DEPRECATION_LOGGER;
*
* @author Steve Ebersole
*/
@SuppressWarnings("WeakerAccess")
public class SessionFactoryOptionsBuilder implements SessionFactoryOptions {
private static final CoreMessageLogger log = messageLogger( SessionFactoryOptionsBuilder.class );
@ -262,7 +261,6 @@ public class SessionFactoryOptionsBuilder implements SessionFactoryOptions {
private int queryStatisticsMaxSize;
@SuppressWarnings({"WeakerAccess", "deprecation"})
public SessionFactoryOptionsBuilder(StandardServiceRegistry serviceRegistry, BootstrapContext context) {
this.serviceRegistry = serviceRegistry;
this.jpaBootstrap = context.isJpaBootstrap();

View File

@ -33,7 +33,6 @@ public abstract class AbstractConverterDescriptor implements ConverterDescriptor
private final AutoApplicableConverterDescriptor autoApplicableDescriptor;
@SuppressWarnings("WeakerAccess")
public AbstractConverterDescriptor(
Class<? extends AttributeConverter> converterClass,
Boolean forceAutoApply,

View File

@ -26,7 +26,6 @@ import org.hibernate.internal.util.StringHelper;
/**
* @author Steve Ebersole
*/
@SuppressWarnings("WeakerAccess")
public class NamedQueryBinder {
public static void processNamedQuery(
HbmLocalMetadataBuildingContext context,

View File

@ -33,7 +33,6 @@ public class StrategySelectorImpl implements StrategySelector {
private static final Logger log = Logger.getLogger( StrategySelectorImpl.class );
@SuppressWarnings("WeakerAccess")
private static final StrategyCreator STANDARD_STRATEGY_CREATOR = strategyClass -> {
try {
return strategyClass.newInstance();

View File

@ -28,7 +28,6 @@ import org.xml.sax.SAXException;
*
* @author Steve Ebersole
*/
@SuppressWarnings("WeakerAccess")
public class LocalXsdResolver {
public static String latestJpaVerison() {

View File

@ -18,7 +18,6 @@ public abstract class AbstractInterceptor implements SessionAssociableIntercepto
private boolean allowLoadOutsideTransaction;
private String sessionFactoryUuid;
@SuppressWarnings("WeakerAccess")
public AbstractInterceptor(String entityName) {
this.entityName = entityName;
}

View File

@ -18,7 +18,6 @@ public abstract class AbstractLazyLoadInterceptor extends AbstractInterceptor im
super( entityName );
}
@SuppressWarnings("WeakerAccess")
public AbstractLazyLoadInterceptor(String entityName, SharedSessionContractImplementor session) {
super( entityName );
setSession( session );

View File

@ -52,7 +52,6 @@ public class RegionFactoryInitiator implements StandardServiceInitiator<RegionFa
}
@SuppressWarnings({"unchecked", "WeakerAccess"})
protected RegionFactory resolveRegionFactory(Map configurationValues, ServiceRegistryImplementor registry) {
final Properties p = new Properties();
p.putAll( configurationValues );
@ -126,7 +125,6 @@ public class RegionFactoryInitiator implements StandardServiceInitiator<RegionFa
return NoCachingRegionFactory.INSTANCE;
}
@SuppressWarnings({"WeakerAccess", "unused"})
protected RegionFactory getFallback(Map configurationValues, ServiceRegistryImplementor registry) {
return null;
}

View File

@ -175,13 +175,11 @@ public abstract class AbstractReadWriteAccess extends AbstractCachedDomainDataAc
}
}
@SuppressWarnings("WeakerAccess")
protected void decrementLock(SharedSessionContractImplementor session, Object key, SoftLockImpl lock) {
lock.unlock( getRegion().getRegionFactory().nextTimestamp() );
getStorageAccess().putIntoCache( key, lock, session );
}
@SuppressWarnings("WeakerAccess")
protected void handleLockExpiry(SharedSessionContractImplementor session, Object key, Lockable lock) {
SecondLevelCacheLogger.INSTANCE.softLockedCacheExpired( getRegion().getName(), key );
log.info( "Cached entry expired : " + key );

View File

@ -20,7 +20,7 @@ import org.hibernate.cache.spi.access.NaturalIdDataAccess;
* @author Steve Ebersole
*/
public class DomainDataRegionImpl extends DomainDataRegionTemplate {
@SuppressWarnings("WeakerAccess")
public DomainDataRegionImpl(
DomainDataRegionConfig regionConfig,
RegionFactoryTemplate regionFactory,

View File

@ -90,7 +90,6 @@ public class DomainDataRegionTemplate extends AbstractDomainDataRegion {
}
}
@SuppressWarnings("WeakerAccess")
protected EntityDataAccess generateReadOnlyEntityAccess(EntityDataCachingConfig accessConfig) {
return new EntityReadOnlyAccess(
this,
@ -100,7 +99,6 @@ public class DomainDataRegionTemplate extends AbstractDomainDataRegion {
);
}
@SuppressWarnings("WeakerAccess")
protected EntityDataAccess generateReadWriteEntityAccess(EntityDataCachingConfig accessConfig) {
return new EntityReadWriteAccess(
this,
@ -110,7 +108,6 @@ public class DomainDataRegionTemplate extends AbstractDomainDataRegion {
);
}
@SuppressWarnings("WeakerAccess")
protected EntityDataAccess generateNonStrictReadWriteEntityAccess(EntityDataCachingConfig accessConfig) {
return new EntityNonStrictReadWriteAccess(
this,
@ -120,7 +117,6 @@ public class DomainDataRegionTemplate extends AbstractDomainDataRegion {
);
}
@SuppressWarnings({"WeakerAccess"})
protected EntityDataAccess generateTransactionalEntityDataAccess(EntityDataCachingConfig entityAccessConfig) {
throw generateTransactionalNotSupportedException();
}
@ -155,7 +151,6 @@ public class DomainDataRegionTemplate extends AbstractDomainDataRegion {
}
}
@SuppressWarnings("WeakerAccess")
protected NaturalIdDataAccess generateReadOnlyNaturalIdAccess(NaturalIdDataCachingConfig accessConfig) {
return new NaturalIdReadOnlyAccess(
this,
@ -165,7 +160,6 @@ public class DomainDataRegionTemplate extends AbstractDomainDataRegion {
);
}
@SuppressWarnings("WeakerAccess")
protected NaturalIdDataAccess generateReadWriteNaturalIdAccess(NaturalIdDataCachingConfig accessConfig) {
return new NaturalIdReadWriteAccess(
this,
@ -175,7 +169,6 @@ public class DomainDataRegionTemplate extends AbstractDomainDataRegion {
);
}
@SuppressWarnings("WeakerAccess")
protected NaturalIdDataAccess generateNonStrictReadWriteNaturalIdAccess(NaturalIdDataCachingConfig accessConfig) {
return new NaturalIdNonStrictReadWriteAccess(
this,
@ -185,7 +178,6 @@ public class DomainDataRegionTemplate extends AbstractDomainDataRegion {
);
}
@SuppressWarnings({"WeakerAccess"})
protected NaturalIdDataAccess generateTransactionalNaturalIdDataAccess(NaturalIdDataCachingConfig accessConfig) {
throw generateTransactionalNotSupportedException();
}
@ -242,7 +234,6 @@ public class DomainDataRegionTemplate extends AbstractDomainDataRegion {
);
}
@SuppressWarnings({"WeakerAccess", "unused"})
protected CollectionDataAccess generateTransactionalCollectionDataAccess(CollectionDataCachingConfig accessConfig) {
throw generateTransactionalNotSupportedException();
}

View File

@ -141,7 +141,7 @@ import static org.hibernate.cfg.BinderHelper.toAliasTableMap;
* @author inger
* @author Emmanuel Bernard
*/
@SuppressWarnings({"unchecked", "WeakerAccess", "deprecation"})
@SuppressWarnings({"unchecked", "deprecation"})
public abstract class CollectionBinder {
private static final CoreMessageLogger LOG = Logger.getMessageLogger(CoreMessageLogger.class, CollectionBinder.class.getName());

View File

@ -14,7 +14,6 @@ import org.hibernate.engine.spi.SharedSessionContractImplementor;
/**
* @author Steve Ebersole
*/
@SuppressWarnings("WeakerAccess")
public class StandardTemporaryTableExporter implements TemporaryTableExporter {
private final Dialect dialect;

View File

@ -26,7 +26,6 @@ import org.hibernate.jdbc.AbstractWork;
/**
* @author Steve Ebersole
*/
@SuppressWarnings("WeakerAccess")
public class TemporaryTableHelper {
private final static CoreMessageLogger log = CoreLogging.messageLogger( TemporaryTableHelper.class );

View File

@ -54,7 +54,6 @@ public class EffectiveEntityGraph implements AppliedGraph, Serializable {
* @implSpec See {@link #EffectiveEntityGraph}
*/
@Incubating
@SuppressWarnings("WeakerAccess")
public EffectiveEntityGraph(boolean allowOverwrite) {
this.allowOverwrite = allowOverwrite;
}

View File

@ -53,7 +53,6 @@ public class JtaPlatformInitiator implements StandardServiceInitiator<JtaPlatfor
return platform;
}
@SuppressWarnings({"WeakerAccess", "unused"})
protected JtaPlatform getFallbackProvider(Map configurationValues, ServiceRegistryImplementor registry) {
return null;
}

View File

@ -592,7 +592,6 @@ public class DefaultLoadEventListener implements LoadEventListener {
*
* @return The object loaded from the datasource, or null if not found.
*/
@SuppressWarnings("WeakerAccess")
protected Object loadFromDatasource(
final LoadEvent event,
final EntityPersister persister) {

View File

@ -27,7 +27,6 @@ import org.hibernate.type.Type;
*
* @author Gavin King
*/
@SuppressWarnings("WeakerAccess")
public class WrapVisitor extends ProxyVisitor {
private static final CoreMessageLogger LOG = CoreLogging.messageLogger( WrapVisitor.class );
protected Object entity;

View File

@ -27,7 +27,6 @@ import org.hibernate.graph.spi.RootGraphImplementor;
*
* @author asusnjar
*/
@SuppressWarnings("WeakerAccess")
public final class EntityGraphs {
/**
* Merges multiple entity graphs into a single graph that specifies the fetching/loading of all attributes the input

View File

@ -30,7 +30,7 @@ import org.hibernate.graph.spi.RootGraphImplementor;
*
* @author asusnjar
*/
@SuppressWarnings({"unused", "WeakerAccess"})
@SuppressWarnings("unused")
public final class GraphParser {
// ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

View File

@ -14,7 +14,6 @@ import org.hibernate.HibernateException;
* @author asusnjar
*
*/
@SuppressWarnings("WeakerAccess")
public class InvalidGraphException extends HibernateException {
private static final long serialVersionUID = 1L;

View File

@ -36,7 +36,6 @@ public abstract class AbstractGraph<J> extends AbstractGraphNode<J> implements G
private final ManagedDomainType<J> managedType;
private Map<PersistentAttribute<?,?>, AttributeNodeImplementor<?>> attrNodeMap;
@SuppressWarnings("WeakerAccess")
public AbstractGraph(
ManagedDomainType<J> managedType,
boolean mutable,
@ -45,7 +44,6 @@ public abstract class AbstractGraph<J> extends AbstractGraphNode<J> implements G
this.managedType = managedType;
}
@SuppressWarnings("WeakerAccess")
protected AbstractGraph(boolean mutable, GraphImplementor<J> original) {
this( original.getGraphedType(), mutable, original.jpaMetamodel() );

View File

@ -36,7 +36,6 @@ public class AttributeNodeImpl<J>
private Map<Class<? extends J>, SubGraphImplementor<? extends J>> subGraphMap;
private Map<Class<? extends J>, SubGraphImplementor<? extends J>> keySubGraphMap;
@SuppressWarnings("WeakerAccess")
public <X> AttributeNodeImpl(
boolean mutable,
PersistentAttribute<X, J> attribute,
@ -150,7 +149,7 @@ public class AttributeNodeImpl<J>
return internalMakeSubgraph( managedType.findSubType( subType ) );
}
@SuppressWarnings({"WeakerAccess", "unchecked"})
@SuppressWarnings("unchecked")
protected <S extends J> void internalAddSubGraph(Class<S> subType, SubGraphImplementor<S> subGraph) {
log.tracef( "Adding sub-graph : ( (%s) %s )", subGraph.getGraphedType().getTypeName(), getAttributeName() );
@ -217,7 +216,7 @@ public class AttributeNodeImpl<J>
return internalMakeKeySubgraph( subType );
}
@SuppressWarnings({"WeakerAccess", "unchecked"})
@SuppressWarnings("unchecked")
protected <S extends J> void internalAddKeySubGraph(Class<S> subType, SubGraph<S> subGraph) {
log.tracef( "Adding key sub-graph : ( (%s) %s )", subType.getName(), getAttributeName() );

View File

@ -130,13 +130,11 @@ public class SequenceStyleGenerator
/**
* Indicates the name of the column holding the identifier values. The default value is {@link #DEF_VALUE_COLUMN}
*/
@SuppressWarnings("WeakerAccess")
public static final String VALUE_COLUMN_PARAM = "value_column";
/**
* The default value for {@link #VALUE_COLUMN_PARAM}
*/
@SuppressWarnings("WeakerAccess")
public static final String DEF_VALUE_COLUMN = "next_val";
@ -276,7 +274,7 @@ public class SequenceStyleGenerator
* @param jdbcEnv The JdbcEnvironment
* @return The sequence name
*/
@SuppressWarnings({"UnusedParameters", "WeakerAccess"})
@SuppressWarnings("UnusedParameters")
protected QualifiedName determineSequenceName(
Properties params,
Dialect dialect,
@ -359,7 +357,6 @@ public class SequenceStyleGenerator
* @param jdbcEnvironment The JDBC environment
* @return The value column name
*/
@SuppressWarnings({"UnusedParameters", "WeakerAccess"})
protected Identifier determineValueColumnName(Properties params, JdbcEnvironment jdbcEnvironment) {
final String name = ConfigurationHelper.getString( VALUE_COLUMN_PARAM, params, DEF_VALUE_COLUMN );
return jdbcEnvironment.getIdentifierHelper().toIdentifier( name );
@ -375,7 +372,6 @@ public class SequenceStyleGenerator
* @param params The params supplied in the generator config (plus some standard useful extras).
* @return The initial value
*/
@SuppressWarnings({"WeakerAccess"})
protected int determineInitialValue(Properties params) {
return ConfigurationHelper.getInt( INITIAL_PARAM, params, DEFAULT_INITIAL_VALUE );
}
@ -389,7 +385,6 @@ public class SequenceStyleGenerator
* @param params The params supplied in the generator config (plus some standard useful extras).
* @return The increment size
*/
@SuppressWarnings("WeakerAccess")
protected int determineIncrementSize(Properties params) {
return ConfigurationHelper.getInt( INCREMENT_PARAM, params, DEFAULT_INCREMENT_SIZE );
}
@ -403,7 +398,6 @@ public class SequenceStyleGenerator
* @param incrementSize The {@link #determineIncrementSize determined increment size}
* @return The optimizer strategy (name)
*/
@SuppressWarnings("WeakerAccess")
protected String determineOptimizationStrategy(Properties params, int incrementSize) {
return ConfigurationHelper.getString(
OPT_PARAM,
@ -420,7 +414,6 @@ public class SequenceStyleGenerator
* @param incrementSize The {@link #determineIncrementSize determined increment size}
* @return The adjusted increment size.
*/
@SuppressWarnings("WeakerAccess")
protected int determineAdjustedIncrementSize(String optimizationStrategy, int incrementSize) {
final int resolvedIncrementSize;
if ( Math.abs( incrementSize ) > 1 &&
@ -466,7 +459,6 @@ public class SequenceStyleGenerator
*
* @return An abstraction for the actual database structure in use (table vs. sequence).
*/
@SuppressWarnings("WeakerAccess")
protected DatabaseStructure buildDatabaseStructure(
Type type,
Properties params,
@ -505,7 +497,6 @@ public class SequenceStyleGenerator
);
}
@SuppressWarnings("WeakerAccess")
protected DatabaseStructure buildTableStructure(
Type type,
Properties params,

View File

@ -141,7 +141,6 @@ public class TableGenerator implements PersistentIdentifierGenerator {
/**
* The default {@link #TABLE_PARAM} value
*/
@SuppressWarnings("WeakerAccess")
public static final String DEF_TABLE = "hibernate_sequences";
/**
@ -175,20 +174,17 @@ public class TableGenerator implements PersistentIdentifierGenerator {
/**
* The default {@link #SEGMENT_VALUE_PARAM} value, unless {@link #CONFIG_PREFER_SEGMENT_PER_ENTITY} is specified
*/
@SuppressWarnings("WeakerAccess")
public static final String DEF_SEGMENT_VALUE = "default";
/**
* Indicates the length of the column defined by {@link #SEGMENT_COLUMN_PARAM}. Used in schema export. The
* default value is {@link #DEF_SEGMENT_LENGTH}
*/
@SuppressWarnings("WeakerAccess")
public static final String SEGMENT_LENGTH_PARAM = "segment_value_length";
/**
* The default {@link #SEGMENT_LENGTH_PARAM} value
*/
@SuppressWarnings("WeakerAccess")
public static final int DEF_SEGMENT_LENGTH = 255;
private boolean storeLastUsedValue;
@ -264,7 +260,6 @@ public class TableGenerator implements PersistentIdentifierGenerator {
*
* @return the column size.
*/
@SuppressWarnings({"UnusedDeclaration", "WeakerAccess"})
public final int getSegmentValueLength() {
return segmentValueLength;
}
@ -373,7 +368,6 @@ public class TableGenerator implements PersistentIdentifierGenerator {
* @param jdbcEnvironment The JDBC environment
* @return The table name to use.
*/
@SuppressWarnings({"UnusedParameters", "WeakerAccess"})
protected QualifiedName determineGeneratorTableName(Properties params, JdbcEnvironment jdbcEnvironment, ServiceRegistry serviceRegistry) {
String fallbackTableName = DEF_TABLE;
@ -415,7 +409,6 @@ public class TableGenerator implements PersistentIdentifierGenerator {
* @param jdbcEnvironment The JDBC environment
* @return The name of the segment column
*/
@SuppressWarnings({"UnusedParameters", "WeakerAccess"})
protected String determineSegmentColumnName(Properties params, JdbcEnvironment jdbcEnvironment) {
final String name = ConfigurationHelper.getString( SEGMENT_COLUMN_PARAM, params, DEF_SEGMENT_COLUMN );
return jdbcEnvironment.getIdentifierHelper().toIdentifier( name ).render( jdbcEnvironment.getDialect() );
@ -431,7 +424,6 @@ public class TableGenerator implements PersistentIdentifierGenerator {
* @param jdbcEnvironment The JDBC environment
* @return The name of the value column
*/
@SuppressWarnings({"UnusedParameters", "WeakerAccess"})
protected String determineValueColumnName(Properties params, JdbcEnvironment jdbcEnvironment) {
final String name = ConfigurationHelper.getString( VALUE_COLUMN_PARAM, params, DEF_VALUE_COLUMN );
return jdbcEnvironment.getIdentifierHelper().toIdentifier( name ).render( jdbcEnvironment.getDialect() );
@ -446,7 +438,6 @@ public class TableGenerator implements PersistentIdentifierGenerator {
* @param params The params supplied in the generator config (plus some standard useful extras).
* @return The name of the value column
*/
@SuppressWarnings("WeakerAccess")
protected String determineSegmentValue(Properties params) {
String segmentValue = params.getProperty( SEGMENT_VALUE_PARAM );
if ( StringHelper.isEmpty( segmentValue ) ) {
@ -462,7 +453,6 @@ public class TableGenerator implements PersistentIdentifierGenerator {
* @param params The params supplied in the generator config (plus some standard useful extras).
* @return The default segment value to use.
*/
@SuppressWarnings("WeakerAccess")
protected String determineDefaultSegmentValue(Properties params) {
final boolean preferSegmentPerEntity = ConfigurationHelper.getBoolean( CONFIG_PREFER_SEGMENT_PER_ENTITY, params, false );
final String defaultToUse = preferSegmentPerEntity ? params.getProperty( TABLE ) : DEF_SEGMENT_VALUE;
@ -479,22 +469,19 @@ public class TableGenerator implements PersistentIdentifierGenerator {
* @param params The params supplied in the generator config (plus some standard useful extras).
* @return The size of the segment column
*/
@SuppressWarnings("WeakerAccess")
protected int determineSegmentColumnSize(Properties params) {
return ConfigurationHelper.getInt( SEGMENT_LENGTH_PARAM, params, DEF_SEGMENT_LENGTH );
}
@SuppressWarnings("WeakerAccess")
protected int determineInitialValue(Properties params) {
return ConfigurationHelper.getInt( INITIAL_PARAM, params, DEFAULT_INITIAL_VALUE );
}
@SuppressWarnings("WeakerAccess")
protected int determineIncrementSize(Properties params) {
return ConfigurationHelper.getInt( INCREMENT_PARAM, params, DEFAULT_INCREMENT_SIZE );
}
@SuppressWarnings({"unchecked", "WeakerAccess"})
@SuppressWarnings("unchecked")
protected String buildSelectQuery(String formattedPhysicalTableName, SqlStringGenerationContext context) {
final String alias = "tbl";
final String query = "select " + StringHelper.qualify( alias, valueColumnName ) +
@ -506,14 +493,12 @@ public class TableGenerator implements PersistentIdentifierGenerator {
return context.getDialect().applyLocksToSql( query, lockOptions, updateTargetColumnsMap );
}
@SuppressWarnings("WeakerAccess")
protected String buildUpdateQuery(String formattedPhysicalTableName, SqlStringGenerationContext context) {
return "update " + formattedPhysicalTableName +
" set " + valueColumnName + "=? " +
" where " + valueColumnName + "=? and " + segmentColumnName + "=?";
}
@SuppressWarnings("WeakerAccess")
protected String buildInsertQuery(String formattedPhysicalTableName, SqlStringGenerationContext context) {
return "insert into " + formattedPhysicalTableName + " (" + segmentColumnName + ", " + valueColumnName + ") " + " values (?,?)";
}

View File

@ -114,7 +114,6 @@ import static java.lang.Boolean.TRUE;
*
* @author Steve Ebersole
*/
@SuppressWarnings("WeakerAccess")
public abstract class AbstractSharedSessionContract implements SharedSessionContractImplementor {
private static final EntityManagerMessageLogger log = HEMLogging.messageLogger( SessionImpl.class );

View File

@ -20,7 +20,6 @@ import java.util.Spliterators;
*
* @author Steve Ebersole
*/
@SuppressWarnings({"NullableProblems", "unused", "WeakerAccess"})
public class UniqueList<E> extends AbstractList<E> implements Set<E>, List<E> {
private final List<E> elements;

View File

@ -91,7 +91,6 @@ public class PersistenceXmlParser {
*
* @return The single persistence-unit descriptor
*/
@SuppressWarnings("WeakerAccess")
public static ParsedPersistenceXmlDescriptor locateIndividualPersistenceUnit(URL persistenceXmlUrl, Map integration) {
return locateIndividualPersistenceUnit( persistenceXmlUrl, PersistenceUnitTransactionType.RESOURCE_LOCAL, integration );
}
@ -106,7 +105,6 @@ public class PersistenceXmlParser {
*
* @return The single persistence-unit descriptor
*/
@SuppressWarnings("WeakerAccess")
public static ParsedPersistenceXmlDescriptor locateIndividualPersistenceUnit(
URL persistenceXmlUrl,
PersistenceUnitTransactionType transactionType,
@ -144,7 +142,6 @@ public class PersistenceXmlParser {
*
* @return The matching persistence-unit descriptor
*/
@SuppressWarnings("WeakerAccess")
public static ParsedPersistenceXmlDescriptor locateNamedPersistenceUnit(URL persistenceXmlUrl, String name, Map integration) {
return locateNamedPersistenceUnit( persistenceXmlUrl, name, PersistenceUnitTransactionType.RESOURCE_LOCAL, integration );
}
@ -159,7 +156,6 @@ public class PersistenceXmlParser {
*
* @return The matching persistence-unit descriptor
*/
@SuppressWarnings("WeakerAccess")
public static ParsedPersistenceXmlDescriptor locateNamedPersistenceUnit(
URL persistenceXmlUrl,
String name,

View File

@ -25,7 +25,6 @@ import org.hibernate.annotations.common.reflection.XMethod;
import org.hibernate.internal.util.ReflectHelper;
import org.hibernate.jpa.event.spi.CallbackDefinition;
import org.hibernate.jpa.event.spi.CallbackType;
import org.hibernate.mapping.PersistentClass;
import org.hibernate.mapping.Property;
import org.hibernate.property.access.spi.Getter;
@ -41,12 +40,11 @@ import org.jboss.logging.Logger;
public final class CallbackDefinitionResolverLegacyImpl {
private static final Logger log = Logger.getLogger( CallbackDefinitionResolverLegacyImpl.class );
@SuppressWarnings({"unchecked", "WeakerAccess"})
public static List<CallbackDefinition> resolveEntityCallbacks(ReflectionManager reflectionManager,
XClass entityClass, CallbackType callbackType) {
List<CallbackDefinition> callbackDefinitions = new ArrayList<>();
List<String> callbacksMethodNames = new ArrayList<>();
List<Class> orderedListeners = new ArrayList<>();
List<Class<?>> orderedListeners = new ArrayList<>();
XClass currentClazz = entityClass;
boolean stopListeners = false;
boolean stopDefaultListeners = false;
@ -61,8 +59,8 @@ public final class CallbackDefinitionResolverLegacyImpl {
//overridden method, remove the superclass overridden method
if ( callbackDefinition == null ) {
callbackDefinition = new EntityCallback.Definition( method, callbackType );
Class returnType = method.getReturnType();
Class[] args = method.getParameterTypes();
Class<?> returnType = method.getReturnType();
Class<?>[] args = method.getParameterTypes();
if ( returnType != Void.TYPE || args.length != 0 ) {
throw new RuntimeException(
"Callback methods annotated on the bean class must return void and take no arguments: "
@ -108,7 +106,9 @@ public final class CallbackDefinitionResolverLegacyImpl {
//handle default listeners
if ( !stopDefaultListeners ) {
List<Class> defaultListeners = (List<Class>) reflectionManager.getDefaults().get( EntityListeners.class );
@SuppressWarnings("unchecked")
List<Class<?>> defaultListeners = (List<Class<?>>)
reflectionManager.getDefaults().get( EntityListeners.class );
if ( defaultListeners != null ) {
int defaultListenerSize = defaultListeners.size();
@ -118,7 +118,7 @@ public final class CallbackDefinitionResolverLegacyImpl {
}
}
for ( Class listener : orderedListeners ) {
for ( Class<?> listener : orderedListeners ) {
CallbackDefinition callbackDefinition = null;
if ( listener != null ) {
XClass xListener = reflectionManager.toXClass( listener );
@ -133,8 +133,8 @@ public final class CallbackDefinitionResolverLegacyImpl {
if ( callbackDefinition == null ) {
callbackDefinition = new ListenerCallback.Definition( listener, method, callbackType );
Class returnType = method.getReturnType();
Class[] args = method.getParameterTypes();
Class<?> returnType = method.getReturnType();
Class<?>[] args = method.getParameterTypes();
if ( returnType != Void.TYPE || args.length != 1 ) {
throw new PersistenceException(
"Callback methods annotated in a listener bean class must return void and take one argument: "
@ -171,7 +171,7 @@ public final class CallbackDefinitionResolverLegacyImpl {
public static List<CallbackDefinition> resolveEmbeddableCallbacks(ReflectionManager reflectionManager,
Class<?> entityClass, Property embeddableProperty,
CallbackType callbackType) {
final Class embeddableClass = embeddableProperty.getType().getReturnedClass();
final Class<?> embeddableClass = embeddableProperty.getType().getReturnedClass();
final XClass embeddableXClass = reflectionManager.toXClass( embeddableClass );
final Getter embeddableGetter = embeddableProperty.getGetter( entityClass );
final List<CallbackDefinition> callbackDefinitions = new ArrayList<>();
@ -188,8 +188,8 @@ public final class CallbackDefinitionResolverLegacyImpl {
//overridden method, remove the superclass overridden method
if ( callbackDefinition == null ) {
callbackDefinition = new EmbeddableCallback.Definition( embeddableGetter, method, callbackType );
Class returnType = method.getReturnType();
Class[] args = method.getParameterTypes();
Class<?> returnType = method.getReturnType();
Class<?>[] args = method.getParameterTypes();
if ( returnType != Void.TYPE || args.length != 0 ) {
throw new RuntimeException(
"Callback methods annotated on the bean class must return void and take no arguments: "
@ -238,15 +238,16 @@ public final class CallbackDefinitionResolverLegacyImpl {
for ( ElementType type : target.value() ) {
if ( type.equals( ElementType.ANNOTATION_TYPE ) ) {
useAnnotationAnnotatedByListener = true;
break;
}
}
}
}
private static void getListeners(XClass currentClazz, List<Class> orderedListeners) {
private static void getListeners(XClass currentClazz, List<Class<?>> orderedListeners) {
EntityListeners entityListeners = currentClazz.getAnnotation( EntityListeners.class );
if ( entityListeners != null ) {
Class[] classes = entityListeners.value();
Class<?>[] classes = entityListeners.value();
int size = classes.length;
for ( int index = size - 1; index >= 0; index-- ) {
orderedListeners.add( classes[index] );
@ -257,7 +258,7 @@ public final class CallbackDefinitionResolverLegacyImpl {
for ( Annotation annot : annotations ) {
entityListeners = annot.getClass().getAnnotation( EntityListeners.class );
if ( entityListeners != null ) {
Class[] classes = entityListeners.value();
Class<?>[] classes = entityListeners.value();
int size = classes.length;
for ( int index = size - 1; index >= 0; index-- ) {
orderedListeners.add( classes[index] );

View File

@ -130,7 +130,6 @@ public class BasicValue extends SimpleValue implements JdbcTypeIndicators, Resol
this.enumerationStyle = enumerationStyle;
}
@SuppressWarnings("WeakerAccess")
public EnumType getEnumerationStyle() {
return enumerationStyle;
}

View File

@ -172,7 +172,6 @@ public class MappedSuperclass {
*
* @return {@code true} if a property with that name exists; {@code false} if not
*/
@SuppressWarnings("WeakerAccess")
public boolean hasProperty(String name) {
final Iterator itr = getDeclaredPropertyIterator();
while ( itr.hasNext() ) {
@ -193,7 +192,6 @@ public class MappedSuperclass {
*
* @return {@code true} if a property with that name exists; {@code false} if not
*/
@SuppressWarnings({"WeakerAccess", "RedundantIfStatement"})
public boolean isPropertyDefinedInHierarchy(String name) {
if ( hasProperty( name ) ) {
return true;

View File

@ -565,7 +565,6 @@ public abstract class PersistentClass implements AttributeContainer, Serializabl
*
* @return {@code true} if a property with that name exists; {@code false} if not
*/
@SuppressWarnings("WeakerAccess")
public boolean hasProperty(String name) {
final Property identifierProperty = getIdentifierProperty();
if ( identifierProperty != null && identifierProperty.getName().equals( name ) ) {
@ -606,7 +605,6 @@ public abstract class PersistentClass implements AttributeContainer, Serializabl
*
* @return {@code true} if a property with that name exists; {@code false} if not
*/
@SuppressWarnings({"WeakerAccess", "RedundantIfStatement"})
public boolean isPropertyDefinedInHierarchy(String name) {
if ( hasProperty( name ) ) {
return true;

View File

@ -217,7 +217,6 @@ public class MetadataContext {
*
* @return Tne corresponding JPA {@link org.hibernate.type.EntityType}, or null if not yet processed.
*/
@SuppressWarnings("WeakerAccess")
public EntityDomainType<?> locateEntityType(PersistentClass persistentClass) {
return entityTypesByPersistentClass.get( persistentClass );
}
@ -242,17 +241,16 @@ public class MetadataContext {
*
* @return The corresponding JPA {@link org.hibernate.type.EntityType}, or null.
*/
@SuppressWarnings({"unchecked", "WeakerAccess"})
@SuppressWarnings("unchecked")
public <E> EntityDomainType<E> locateEntityType(String entityName) {
return (EntityDomainType<E>) entityTypesByEntityName.get( entityName );
}
@SuppressWarnings("WeakerAccess")
public Map<String, EntityDomainType<?>> getEntityTypesByEntityName() {
return Collections.unmodifiableMap( entityTypesByEntityName );
}
@SuppressWarnings({"unchecked", "WeakerAccess"})
@SuppressWarnings("unchecked")
public void wrapUp() {
if ( LOG.isTraceEnabled() ) {
LOG.trace( "Wrapping up metadata context..." );

View File

@ -20,9 +20,8 @@ import org.hibernate.type.descriptor.java.JavaType;
public class PojoInstantiatorImpl<J> extends AbstractPojoInstantiator {
private static final CoreMessageLogger LOG = CoreLogging.messageLogger( PojoInstantiatorImpl.class );
private final Constructor constructor;
private final Constructor<?> constructor;
@SuppressWarnings("WeakerAccess")
public PojoInstantiatorImpl(JavaType javaType) {
super( javaType.getJavaTypeClass() );
@ -31,9 +30,8 @@ public class PojoInstantiatorImpl<J> extends AbstractPojoInstantiator {
: resolveConstructor( getMappedPojoClass() );
}
protected static Constructor resolveConstructor(Class mappedPojoClass) {
protected static Constructor<?> resolveConstructor(Class<?> mappedPojoClass) {
try {
//noinspection unchecked
return ReflectHelper.getDefaultConstructor( mappedPojoClass);
}
catch ( PropertyNotFoundException e ) {

View File

@ -20,7 +20,6 @@ public abstract class AbstractAttributeMapping implements AttributeMapping {
private final ManagedMappingType declaringType;
@SuppressWarnings("WeakerAccess")
public AbstractAttributeMapping(String name, ManagedMappingType declaringType) {
this.name = name;
this.declaringType = declaringType;

View File

@ -62,7 +62,6 @@ public class BasicAttributeMapping
private final JavaType domainTypeDescriptor;
@SuppressWarnings("WeakerAccess")
public BasicAttributeMapping(
String attributeName,
NavigableRole navigableRole,

View File

@ -68,7 +68,6 @@ public class EmbeddedAttributeMapping
private final EmbeddableMappingType embeddableMappingType;
private final PropertyAccess parentInjectionAttributePropertyAccess;
@SuppressWarnings("WeakerAccess")
public EmbeddedAttributeMapping(
String name,
NavigableRole navigableRole,

View File

@ -68,7 +68,6 @@ public class EmbeddedCollectionPart implements CollectionPart, EmbeddableValuedF
private final PropertyAccess parentInjectionAttributePropertyAccess;
private final String sqlAliasStem;
@SuppressWarnings("WeakerAccess")
public EmbeddedCollectionPart(
CollectionPersister collectionDescriptor,
Nature nature,

View File

@ -33,7 +33,6 @@ public class EmbeddedIdentifierMappingImpl
private final EmbeddableMappingType embeddableDescriptor;
private final PropertyAccess propertyAccess;
@SuppressWarnings("WeakerAccess")
public EmbeddedIdentifierMappingImpl(
EntityMappingType entityMapping,
String name,

View File

@ -87,7 +87,6 @@ public class EntityCollectionPart
private ModelPart fkTargetModelPart;
private ForeignKeyDescriptor fkDescriptor;
@SuppressWarnings("WeakerAccess")
public EntityCollectionPart(
CollectionPersister collectionDescriptor,
Nature nature,
@ -198,7 +197,6 @@ public class EntityCollectionPart
}
}
@SuppressWarnings("WeakerAccess")
public void finishInitialization(
CollectionPersister collectionDescriptor,
Collection bootValueMapping,

View File

@ -266,13 +266,13 @@ public class MappingModelCreationHelper {
.getBasicTypeRegistry()
.resolve( valueConverter.getRelationalJavaType(), resolution.getJdbcType() );
final GeneratedValueResolver generatedValueResolver;
if ( valueGeneration == null ) {
generatedValueResolver = NoGeneratedValueResolver.INSTANCE;
}
else if ( valueGeneration.getValueGenerator() == null ) {
// in-db generation
}
// final GeneratedValueResolver generatedValueResolver;
// if ( valueGeneration == null ) {
// generatedValueResolver = NoGeneratedValueResolver.INSTANCE;
// }
// else if ( valueGeneration.getValueGenerator() == null ) {
// // in-db generation
// }
return new BasicAttributeMapping(
attrName,
@ -1527,26 +1527,24 @@ public class MappingModelCreationHelper {
}
}
@SuppressWarnings("rawtypes")
private static class CollectionMappingTypeImpl implements CollectionMappingType {
private final JavaType collectionJtd;
private final CollectionSemantics semantics;
private final JavaType<?> collectionJtd;
private final CollectionSemantics<?,?> semantics;
@SuppressWarnings("WeakerAccess")
public CollectionMappingTypeImpl(
JavaType collectionJtd,
CollectionSemantics semantics) {
JavaType<?> collectionJtd,
CollectionSemantics<?,?> semantics) {
this.collectionJtd = collectionJtd;
this.semantics = semantics;
}
@Override
public CollectionSemantics getCollectionSemantics() {
public CollectionSemantics<?,?> getCollectionSemantics() {
return semantics;
}
@Override
public JavaType getMappedJavaType() {
public JavaType<?> getMappedJavaType() {
return collectionJtd;
}
}

View File

@ -112,13 +112,12 @@ public class PluralAttributeMappingImpl
private OrderByFragment orderByFragment;
private OrderByFragment manyToManyOrderByFragment;
@SuppressWarnings({"WeakerAccess", "rawtypes"})
public PluralAttributeMappingImpl(
String attributeName,
Collection bootDescriptor,
PropertyAccess propertyAccess,
StateArrayContributorMetadataAccess stateArrayContributorMetadataAccess,
CollectionMappingType collectionMappingType,
CollectionMappingType<?> collectionMappingType,
int stateArrayPosition,
CollectionPart elementDescriptor,
CollectionPart indexDescriptor,

View File

@ -15,7 +15,6 @@ public abstract class AbstractDomainType<J> implements SimpleDomainType<J> {
private final JpaMetamodel domainMetamodel;
private final JavaType<J> javaType;
@SuppressWarnings("WeakerAccess")
public AbstractDomainType(JavaType<J> javaType, JpaMetamodel domainMetamodel) {
this.javaType = javaType;
this.domainMetamodel = domainMetamodel;

View File

@ -336,12 +336,11 @@ public abstract class AbstractManagedType<J>
}
}
@SuppressWarnings({"SimplifiableIfStatement", "WeakerAccess"})
protected <Y> boolean isPrimitiveVariant(SingularAttribute<?,?> attribute, Class<Y> javaType) {
if ( attribute == null ) {
return false;
}
Class declaredType = attribute.getBindableJavaType();
Class<?> declaredType = attribute.getBindableJavaType();
if ( declaredType.isPrimitive() ) {
return ( Boolean.class.equals( javaType ) && Boolean.TYPE.equals( declaredType ) )

View File

@ -38,7 +38,6 @@ public abstract class AbstractAttribute<D,J,B> implements PersistentAttribute<D,
private final DomainType<B> valueType;
private transient Member member;
@SuppressWarnings("WeakerAccess")
protected AbstractAttribute(
ManagedDomainType<D> declaringType,
String name,

View File

@ -36,7 +36,6 @@ public abstract class AbstractPluralAttribute<D, C, E>
private final CollectionClassification classification;
private final SqmPathSource<E> elementPathSource;
@SuppressWarnings("WeakerAccess")
protected AbstractPluralAttribute(
PluralAttributeBuilder<D,C,E,?> builder,
MetadataContext metadataContext) {

View File

@ -23,18 +23,16 @@ import static jakarta.persistence.metamodel.Bindable.BindableType.SINGULAR_ATTRI
public class AnyMappingSqmPathSource<J> extends AbstractSqmPathSource<J> implements BindableType<J> {
private final SqmPathSource<?> keyPathSource;
@SuppressWarnings("WeakerAccess")
public AnyMappingSqmPathSource(
String localPathName,
AnyMappingDomainType<J> domainType,
BindableType jpaBindableType) {
super( localPathName, domainType, jpaBindableType );
keyPathSource = new BasicSqmPathSource( "id", (BasicDomainType) domainType.getKeyType(), SINGULAR_ATTRIBUTE );
keyPathSource = new BasicSqmPathSource<>( "id", (BasicDomainType<?>) domainType.getKeyType(), SINGULAR_ATTRIBUTE );
}
@Override
@Override @SuppressWarnings("unchecked")
public AnyMappingDomainType<J> getSqmPathType() {
//noinspection unchecked
return (AnyMappingDomainType<J>) super.getSqmPathType();
}

View File

@ -20,7 +20,7 @@ import org.hibernate.query.sqm.tree.domain.SqmPath;
public class BasicSqmPathSource<J>
extends AbstractSqmPathSource<J>
implements BindableType<J>, ReturnableType<J> {
@SuppressWarnings("WeakerAccess")
public BasicSqmPathSource(
String localPathName,
BasicDomainType<J> domainType,

View File

@ -36,7 +36,6 @@ public final class PersisterFactoryImpl implements PersisterFactory, ServiceRegi
/**
* The constructor signature for {@link EntityPersister} implementations
*/
@SuppressWarnings("WeakerAccess")
public static final Class<?>[] ENTITY_PERSISTER_CONSTRUCTOR_ARGS = new Class[] {
PersistentClass.class,
EntityDataAccess.class,
@ -47,7 +46,6 @@ public final class PersisterFactoryImpl implements PersisterFactory, ServiceRegi
/**
* The constructor signature for {@link CollectionPersister} implementations
*/
@SuppressWarnings("WeakerAccess")
public static final Class<?>[] COLLECTION_PERSISTER_CONSTRUCTOR_ARGS = new Class[] {
Collection.class,
CollectionDataAccess.class,
@ -143,7 +141,6 @@ public final class PersisterFactoryImpl implements PersisterFactory, ServiceRegi
}
@Override
@SuppressWarnings( {"unchecked"})
public CollectionPersister createCollectionPersister(
Collection collectionBinding,
CollectionDataAccess cacheAccessStrategy,

View File

@ -720,7 +720,6 @@ public class ProcedureCallImpl<R>
*
* @return The spaces
*/
@SuppressWarnings("WeakerAccess")
protected Set<String> synchronizedQuerySpaces() {
if ( synchronizedQuerySpaces == null ) {
synchronizedQuerySpaces = new HashSet<>();
@ -750,7 +749,6 @@ public class ProcedureCallImpl<R>
return this;
}
@SuppressWarnings("WeakerAccess")
protected void addSynchronizedQuerySpaces(EntityPersister persister) {
synchronizedQuerySpaces().addAll( Arrays.asList( (String[]) persister.getQuerySpaces() ) );
}

View File

@ -35,11 +35,9 @@ public class ProcedureParameterMetadataImpl implements ProcedureParameterMetadat
private ParameterStrategy parameterStrategy = ParameterStrategy.UNKNOWN;
private List<ProcedureParameterImplementor<?>> parameters;
@SuppressWarnings("WeakerAccess")
public ProcedureParameterMetadataImpl() {
}
@SuppressWarnings("WeakerAccess")
public ProcedureParameterMetadataImpl(NamedCallableQueryMemento memento, SharedSessionContractImplementor session) {
memento.getParameterMementos().forEach(
parameterMemento -> registerParameter( parameterMemento.resolve( session ) )
@ -119,7 +117,6 @@ public class ProcedureParameterMetadataImpl implements ProcedureParameterMetadat
return parameters.contains( parameter );
}
@SuppressWarnings("WeakerAccess")
public ParameterStrategy getParameterStrategy() {
return parameterStrategy;
}

View File

@ -21,7 +21,6 @@ import org.hibernate.query.sqm.tree.expression.SqmExpression;
public class DomainPathPart implements SemanticPathPart {
private SqmPath<?> currentPath;
@SuppressWarnings("WeakerAccess")
public DomainPathPart(SqmPath<?> basePath) {
this.currentPath = basePath;
assert currentPath != null;

View File

@ -18,7 +18,6 @@ public class FullyQualifiedReflectivePath implements SemanticPathPart, FullyQual
private final FullyQualifiedReflectivePathSource pathSource;
private final String localName;
@SuppressWarnings("WeakerAccess")
public FullyQualifiedReflectivePath(
FullyQualifiedReflectivePathSource pathSource,
String localName) {

View File

@ -12,6 +12,5 @@ import org.hibernate.query.hql.spi.SemanticPathPart;
/**
* @author Steve Ebersole
*/
@SuppressWarnings("WeakerAccess")
public interface FullyQualifiedReflectivePathSource extends DotIdentifierSequence, SemanticPathPart {
}

View File

@ -42,7 +42,6 @@ public class FullyQualifiedReflectivePathTerminal
private final Function<SemanticQueryWalker,?> handler;
@SuppressWarnings("WeakerAccess")
public FullyQualifiedReflectivePathTerminal(
FullyQualifiedReflectivePathSource pathSource,
String subPathName,

View File

@ -253,7 +253,6 @@ public class SemanticQueryBuilder<R> extends HqlParserBaseVisitor<Object> implem
* Main entry point into analysis of HQL/JPQL parse tree - producing a semantic model of the
* query.
*/
@SuppressWarnings("WeakerAccess")
public static <R> SqmStatement<R> buildSemanticModel(
HqlParser.StatementContext hqlParseTree,
SqmCreationOptions creationOptions,
@ -276,7 +275,6 @@ public class SemanticQueryBuilder<R> extends HqlParserBaseVisitor<Object> implem
private ParameterCollector parameterCollector;
private ParameterStyle parameterStyle;
@SuppressWarnings("WeakerAccess")
public SemanticQueryBuilder(SqmCreationOptions creationOptions, SqmCreationContext creationContext) {
this.creationOptions = creationOptions;
this.creationContext = creationContext;
@ -838,7 +836,6 @@ public class SemanticQueryBuilder<R> extends HqlParserBaseVisitor<Object> implem
return sqmQuerySpec;
}
@SuppressWarnings("WeakerAccess")
protected SqmSelectClause buildInferredSelectClause(SqmFromClause fromClause) {
// for now, this is slightly different than the legacy behavior where
// the root and each non-fetched-join was selected. For now, here, we simply
@ -1591,7 +1588,6 @@ public class SemanticQueryBuilder<R> extends HqlParserBaseVisitor<Object> implem
throw new UnsupportedOperationException( "Unexpected call to #visitJoin, see #consumeJoin" );
}
@SuppressWarnings("WeakerAccess")
protected <X> void consumeJoin(HqlParser.JoinContext parserJoin, SqmRoot<X> sqmRoot) {
final SqmJoinType joinType;
final int firstJoinTypeSymbolType;
@ -1689,7 +1685,6 @@ public class SemanticQueryBuilder<R> extends HqlParserBaseVisitor<Object> implem
throw new UnsupportedOperationException();
}
@SuppressWarnings("WeakerAccess")
protected void consumeJpaCollectionJoin(
HqlParser.JpaCollectionJoinContext ctx,
SqmRoot<?> sqmRoot) {

View File

@ -551,13 +551,11 @@ public abstract class AbstractCommonQueryContract implements CommonQueryContract
return this;
}
@SuppressWarnings("WeakerAccess")
protected boolean applyJpaCacheRetrieveModeHint(CacheRetrieveMode retrieveMode) {
getQueryOptions().setCacheRetrieveMode( retrieveMode );
return true;
}
@SuppressWarnings("WeakerAccess")
protected boolean applyJpaCacheStoreModeHint(CacheStoreMode storeMode) {
getQueryOptions().setCacheStoreMode( storeMode );
return true;
@ -748,13 +746,11 @@ public abstract class AbstractCommonQueryContract implements CommonQueryContract
return getQueryParameterBindings().getBinding( parameter );
}
@SuppressWarnings( {"WeakerAccess"} )
protected <P> QueryParameterBinding<P> locateBinding(String name) {
getSession().checkOpen();
return getQueryParameterBindings().getBinding( name );
}
@SuppressWarnings( {"WeakerAccess"} )
protected <P> QueryParameterBinding<P> locateBinding(int position) {
getSession().checkOpen();
return getQueryParameterBindings().getBinding( position );
@ -1421,7 +1417,6 @@ public abstract class AbstractCommonQueryContract implements CommonQueryContract
return this;
}
@SuppressWarnings("WeakerAccess")
protected BindableType<Object> determineType(String namedParam, Class<?> retType) {
BindableType<?> type = locateBinding( namedParam ).getBindType();
if ( type == null ) {

View File

@ -553,7 +553,7 @@ public abstract class AbstractQuery<R>
return qp != null && getQueryParameterBindings().isBound( qp );
}
@SuppressWarnings( {"WeakerAccess", "unchecked", "rawtypes"} )
@SuppressWarnings({"unchecked", "rawtypes"})
protected <P> QueryParameterBinding<P> locateBinding(Parameter<P> parameter) {
if ( parameter instanceof QueryParameterImplementor ) {
return locateBinding( (QueryParameterImplementor) parameter );
@ -570,19 +570,16 @@ public abstract class AbstractQuery<R>
);
}
@SuppressWarnings("WeakerAccess")
protected <P> QueryParameterBinding<P> locateBinding(QueryParameterImplementor<P> parameter) {
getSession().checkOpen();
return getQueryParameterBindings().getBinding( parameter );
}
@SuppressWarnings( {"WeakerAccess"} )
protected <P> QueryParameterBinding<P> locateBinding(String name) {
getSession().checkOpen();
return getQueryParameterBindings().getBinding( name );
}
@SuppressWarnings( {"WeakerAccess"} )
protected <P> QueryParameterBinding<P> locateBinding(int position) {
getSession().checkOpen();
return getQueryParameterBindings().getBinding( position );
@ -1246,7 +1243,6 @@ public abstract class AbstractQuery<R>
return this;
}
@SuppressWarnings("WeakerAccess")
protected BindableType<Object> determineType(String namedParam, Class<?> retType) {
BindableType<?> type = locateBinding( namedParam ).getBindType();
if ( type == null ) {
@ -1283,7 +1279,6 @@ public abstract class AbstractQuery<R>
return this;
}
@SuppressWarnings("WeakerAccess")
protected void afterQuery(boolean success) {
if ( sessionFlushMode != null ) {
getSession().setHibernateFlushMode( sessionFlushMode );

View File

@ -224,7 +224,6 @@ public abstract class AbstractSelectionQuery<R>
}
}
@SuppressWarnings("WeakerAccess")
protected static <T> T uniqueElement(List<T> list) throws NonUniqueResultException {
int size = list.size();
if ( size == 0 ) {

View File

@ -41,7 +41,6 @@ public class ParameterRecognizerImpl implements ParameterRecognizer {
private List<ParameterOccurrence> parameterList;
private final StringBuilder sqlStringBuffer = new StringBuilder();
@SuppressWarnings("WeakerAccess")
public ParameterRecognizerImpl(SessionFactoryImplementor factory) {
ordinalParameterImplicitPosition = 1;
}

View File

@ -80,7 +80,6 @@ public class ConcreteSqmSelectQueryPlan<R> implements SelectQueryPlan<R> {
private volatile CacheableSqmInterpretation cacheableSqmInterpretation;
@SuppressWarnings("WeakerAccess")
public ConcreteSqmSelectQueryPlan(
SqmSelectStatement<?> sqm,
String hql,

View File

@ -30,7 +30,6 @@ import org.hibernate.query.sqm.tree.expression.SqmPositionalParameter;
*
* @author Steve Ebersole
*/
@SuppressWarnings("WeakerAccess")
public class DomainParameterXref {
/**
* Create a DomainParameterXref for the parameters defined in the passed

View File

@ -32,7 +32,6 @@ public class SqmInterpretationsKey implements QueryInterpretationCache.Key {
Class<?> getResultType();
}
@SuppressWarnings("WeakerAccess")
public static SqmInterpretationsKey createInterpretationsKey(InterpretationsKeySource keySource) {
if ( ! isCacheable( keySource ) ) {
return null;
@ -83,7 +82,6 @@ public class SqmInterpretationsKey implements QueryInterpretationCache.Key {
return true;
}
@SuppressWarnings("WeakerAccess")
public static QueryInterpretationCache.Key generateNonSelectKey(InterpretationsKeySource keyDetails) {
// todo (6.0) : do we want to cache non-select plans? If so, what requirements?
// - very minimum is that it be a "simple" (non-multi-table) statement

View File

@ -64,7 +64,6 @@ import org.hibernate.type.spi.TypeConfiguration;
*
* @author Steve Ebersole
*/
@SuppressWarnings("WeakerAccess")
public class SqmUtil {
private SqmUtil() {
}

View File

@ -38,7 +38,6 @@ import org.hibernate.sql.results.graph.basic.BasicResult;
*
* @author Christian Beikov
*/
@SuppressWarnings("WeakerAccess")
public class CteDeleteHandler extends AbstractCteMutationHandler implements DeleteHandler {
protected CteDeleteHandler(

View File

@ -118,7 +118,6 @@ public class CteInsertHandler implements InsertHandler {
private final SqmCteTable cteTable;
private final DomainParameterXref domainParameterXref;
@SuppressWarnings("WeakerAccess")
public CteInsertHandler(
SqmCteTable cteTable,
SqmInsertStatement<?> sqmStatement,

View File

@ -45,7 +45,6 @@ import org.hibernate.sql.ast.tree.update.UpdateStatement;
*/
public class CteUpdateHandler extends AbstractCteMutationHandler implements UpdateHandler {
@SuppressWarnings("WeakerAccess")
public CteUpdateHandler(
SqmCteTable cteTable,
SqmUpdateStatement sqmStatement,

View File

@ -41,7 +41,6 @@ import org.hibernate.sql.exec.spi.StatementCreatorHelper;
* @author Vlad Mihalcea
* @author Steve Ebersole
*/
@SuppressWarnings("WeakerAccess")
public class InlineDeleteHandler implements DeleteHandler {
private final MatchingIdRestrictionProducer matchingIdsPredicateProducer;
private final SqmDeleteStatement sqmDeleteStatement;

View File

@ -47,7 +47,6 @@ import org.hibernate.sql.results.internal.SqlSelectionImpl;
/**
* @author Steve Ebersole
*/
@SuppressWarnings("WeakerAccess")
public final class ExecuteWithTemporaryTableHelper {
private ExecuteWithTemporaryTableHelper() {
}

View File

@ -78,7 +78,6 @@ public class RestrictedDeleteExecutionDelegate implements TableBasedDeleteHandle
private final Function<SharedSessionContractImplementor,String> sessionUidAccess;
private final MultiTableSqmMutationConverter converter;
@SuppressWarnings("WeakerAccess")
public RestrictedDeleteExecutionDelegate(
EntityMappingType entityDescriptor,
TemporaryTable idTable,

View File

@ -2132,7 +2132,6 @@ public abstract class BaseSqmToSqlAstConverter<T extends Statement> extends Base
protected Predicate additionalRestrictions;
@SuppressWarnings("WeakerAccess")
protected void consumeFromClauseRoot(SqmRoot<?> sqmRoot) {
log.tracef( "Resolving SqmRoot [%s] to TableGroup", sqmRoot );
final FromClauseIndex fromClauseIndex = getFromClauseIndex();
@ -2441,7 +2440,6 @@ public abstract class BaseSqmToSqlAstConverter<T extends Statement> extends Base
}
}
@SuppressWarnings("WeakerAccess")
protected TableGroup consumeExplicitJoin(
SqmJoin<?, ?> sqmJoin,
TableGroup lhsTableGroup,
@ -4378,7 +4376,6 @@ public abstract class BaseSqmToSqlAstConverter<T extends Statement> extends Base
return null;
}
@SuppressWarnings("WeakerAccess")
protected MappingModelExpressable<?> determineValueMapping(SqmParameter<?> sqmParameter) {
log.debugf( "Determining mapping-model type for SqmParameter : %s", sqmParameter );

View File

@ -106,7 +106,6 @@ public class SqlAstProcessingStateImpl
return normalize( expression );
}
@SuppressWarnings("WeakerAccess")
protected Expression normalize(Expression expression) {
final Clause currentClause = currentClauseAccess.get();
if ( currentClause == Clause.ORDER

View File

@ -41,7 +41,6 @@ public abstract class AbstractSqmAttributeJoin<O,T>
private SqmPredicate onClausePredicate;
@SuppressWarnings("WeakerAccess")
public AbstractSqmAttributeJoin(
SqmFrom<?,O> lhs,
SqmJoinable joinedNavigable,

View File

@ -38,7 +38,6 @@ public abstract class AbstractSqmPath<T> extends AbstractSqmExpression<T> implem
*/
private Map<String, SqmPath<?>> reusablePaths;
@SuppressWarnings("WeakerAccess")
protected AbstractSqmPath(
NavigablePath navigablePath,
SqmPathSource<T> referencedPathSource,

View File

@ -20,7 +20,6 @@ import org.hibernate.query.sqm.tree.from.SqmFrom;
*/
public abstract class AbstractSqmPluralJoin<O,C,E> extends AbstractSqmAttributeJoin<O,E> implements JpaJoin<O,E>, PluralJoin<O,C,E> {
@SuppressWarnings("WeakerAccess")
public AbstractSqmPluralJoin(
SqmFrom<?, O> lhs,
PluralPersistentAttribute<O,C,E> joinedNavigable,

View File

@ -15,7 +15,6 @@ import org.hibernate.query.sqm.SqmPathSource;
*/
public abstract class AbstractSqmSimplePath<T> extends AbstractSqmPath<T> implements SqmSimplePath<T> {
@SuppressWarnings("WeakerAccess")
public AbstractSqmSimplePath(
NavigablePath navigablePath,
SqmPathSource<T> referencedPathSource,
@ -24,7 +23,6 @@ public abstract class AbstractSqmSimplePath<T> extends AbstractSqmPath<T> implem
this( navigablePath, referencedPathSource, lhs, null, nodeBuilder );
}
@SuppressWarnings("WeakerAccess")
public AbstractSqmSimplePath(
NavigablePath navigablePath,
SqmPathSource<T> referencedPathSource,

View File

@ -19,7 +19,6 @@ public abstract class AbstractSqmSpecificPluralPartPath<T> extends AbstractSqmPa
private final SqmPath<?> pluralDomainPath;
private final PluralPersistentAttribute<?, ?, T> pluralAttribute;
@SuppressWarnings("WeakerAccess")
public AbstractSqmSpecificPluralPartPath(
NavigablePath navigablePath,
SqmPath<?> pluralDomainPath,
@ -34,12 +33,10 @@ public abstract class AbstractSqmSpecificPluralPartPath<T> extends AbstractSqmPa
this.pluralAttribute = referencedAttribute;
}
@SuppressWarnings("WeakerAccess")
public SqmPath<?> getPluralDomainPath() {
return pluralDomainPath;
}
@SuppressWarnings("WeakerAccess")
public PluralPersistentAttribute<?, ?, T> getPluralAttribute() {
return pluralAttribute;
}

View File

@ -39,7 +39,6 @@ public class SqmPluralValuedSimplePath<E> extends AbstractSqmSimplePath<E> {
this( navigablePath, referencedNavigable, lhs, null, nodeBuilder );
}
@SuppressWarnings("WeakerAccess")
public SqmPluralValuedSimplePath(
NavigablePath navigablePath,
PluralPersistentAttribute<?, ?, E> referencedNavigable,

View File

@ -20,7 +20,6 @@ public class SqmTreatedBagJoin<O,T, S extends T> extends SqmBagJoin<O,S> impleme
private final SqmBagJoin<O, T> wrappedPath;
private final EntityDomainType<S> treatTarget;
@SuppressWarnings("WeakerAccess")
public SqmTreatedBagJoin(
SqmBagJoin<O, T> wrappedPath,
EntityDomainType<S> treatTarget,

View File

@ -22,7 +22,6 @@ public class SqmTreatedListJoin<O,T, S extends T> extends SqmListJoin<O,S> imple
private final SqmListJoin<O,T> wrappedPath;
private final EntityDomainType<S> treatTarget;
@SuppressWarnings("WeakerAccess")
public SqmTreatedListJoin(
SqmListJoin<O, T> wrappedPath,
EntityDomainType<S> treatTarget,

View File

@ -18,7 +18,6 @@ public class SqmTreatedMapJoin<O, K, V, S extends V> extends SqmMapJoin<O, K, S>
private final SqmMapJoin<O, K, V> wrappedPath;
private final EntityDomainType<S> treatTarget;
@SuppressWarnings("WeakerAccess")
public SqmTreatedMapJoin(
SqmMapJoin<O, K, V> wrappedPath,
EntityDomainType<S> treatTarget,

View File

@ -18,7 +18,6 @@ public class SqmTreatedPluralPartJoin<O,T, S extends T> extends SqmPluralPartJoi
private final SqmPluralPartJoin<O,T> wrappedPath;
private final EntityDomainType<S> treatTarget;
@SuppressWarnings("WeakerAccess")
public SqmTreatedPluralPartJoin(
SqmPluralPartJoin<O,T> wrappedPath,
EntityDomainType<S> treatTarget,

View File

@ -20,7 +20,6 @@ public class SqmTreatedSetJoin<O,T, S extends T> extends SqmSetJoin<O,S> impleme
private final SqmSetJoin<O,T> wrappedPath;
private final EntityDomainType<S> treatTarget;
@SuppressWarnings("WeakerAccess")
public SqmTreatedSetJoin(
SqmSetJoin<O, T> wrappedPath,
EntityDomainType<S> treatTarget,

View File

@ -21,11 +21,11 @@ public class SqmTreatedSimplePath<T, S extends T>
private final EntityDomainType<S> treatTarget;
private final SqmPath<T> wrappedPath;
@SuppressWarnings({"unchecked", "WeakerAccess"})
public SqmTreatedSimplePath(
SqmPluralValuedSimplePath<T> wrappedPath,
EntityDomainType<S> treatTarget,
NodeBuilder nodeBuilder) {
//noinspection unchecked
super(
wrappedPath.getNavigablePath().treatAs(
treatTarget.getHibernateEntityName()
@ -38,11 +38,11 @@ public class SqmTreatedSimplePath<T, S extends T>
this.wrappedPath = wrappedPath;
}
@SuppressWarnings({"unchecked", "WeakerAccess"})
public SqmTreatedSimplePath(
SqmPath<T> wrappedPath,
EntityDomainType<S> treatTarget,
NodeBuilder nodeBuilder) {
//noinspection unchecked
super(
wrappedPath.getNavigablePath().treatAs(
treatTarget.getHibernateEntityName()

View File

@ -20,7 +20,6 @@ public class SqmTreatedSingularJoin<O,T, S extends T> extends SqmSingularJoin<O,
private final SqmSingularJoin<O,T> wrappedPath;
private final EntityDomainType<S> treatTarget;
@SuppressWarnings("WeakerAccess")
public SqmTreatedSingularJoin(
SqmSingularJoin<O,T> wrappedPath,
EntityDomainType<S> treatTarget,

View File

@ -43,7 +43,6 @@ public class SqmFromClause implements Serializable {
/**
* Inject the complete set of domain roots
*/
@SuppressWarnings({"WeakerAccess", "unused"})
public void setRoots(List<SqmRoot<?>> domainRoots) {
this.domainRoots = domainRoots;
}

View File

@ -24,7 +24,6 @@ public abstract class AbstractJpaTupleElement<T>
private SqmExpressable<T> expressableType;
private String alias;
@SuppressWarnings("WeakerAccess")
protected AbstractJpaTupleElement(SqmExpressable<? extends T> expressableType, NodeBuilder criteriaBuilder) {
super( criteriaBuilder );

View File

@ -45,7 +45,6 @@ public class SqmInListPredicate<T> extends AbstractNegatableSqmPredicate impleme
this( testExpression, listExpressions, false, nodeBuilder );
}
@SuppressWarnings("WeakerAccess")
public SqmInListPredicate(
SqmExpression<T> testExpression,
List<? extends SqmExpression<T>> listExpressions,

View File

@ -30,7 +30,6 @@ public class SqmInSubQueryPredicate<T> extends AbstractNegatableSqmPredicate imp
this( testExpression, subQueryExpression, false, nodeBuilder );
}
@SuppressWarnings("WeakerAccess")
public SqmInSubQueryPredicate(
SqmExpression<T> testExpression,
SqmSubQuery<T> subQueryExpression,

View File

@ -22,7 +22,6 @@ public class SqmMemberOfPredicate extends AbstractNegatableSqmPredicate {
this( leftHandExpression, pluralPath, false, nodeBuilder );
}
@SuppressWarnings("WeakerAccess")
public SqmMemberOfPredicate(
SqmExpression leftHandExpression,
SqmPath pluralPath,

View File

@ -59,7 +59,6 @@ public class SqmOrderByClause implements Serializable {
}
}
@SuppressWarnings("WeakerAccess")
public void setSortSpecifications(List<SqmSortSpecification> sortSpecifications) {
this.sortSpecifications = new ArrayList<>();
this.sortSpecifications.addAll( sortSpecifications );

View File

@ -16,7 +16,6 @@ public class SqmAssignment {
private final SqmPath targetPath;
private final SqmExpression value;
@SuppressWarnings("WeakerAccess")
public SqmAssignment(SqmPath targetPath, SqmExpression value) {
this.targetPath = targetPath;
this.value = value;

View File

@ -134,7 +134,6 @@ public abstract class AbstractCdiBeanContainer implements CdiBasedBeanContainer
BeanInstanceProducer fallbackProducer);
@SuppressWarnings("WeakerAccess")
protected final void forEachBean(Consumer<ContainedBeanImplementor<?>> consumer) {
registeredBeans.forEach( consumer );
}

View File

@ -228,7 +228,6 @@ public abstract class AbstractSqlAstTranslator<T extends JdbcOperation> implemen
return dialect;
}
@SuppressWarnings("WeakerAccess")
protected AbstractSqlAstTranslator(SessionFactoryImplementor sessionFactory, Statement statement) {
this.sessionFactory = sessionFactory;
this.statement = statement;
@ -347,7 +346,6 @@ public abstract class AbstractSqlAstTranslator<T extends JdbcOperation> implemen
setLimitParameter( null );
}
@SuppressWarnings("WeakerAccess")
public List<JdbcParameterBinder> getParameterBinders() {
return parameterBinders;
}
@ -2278,7 +2276,6 @@ public abstract class AbstractSqlAstTranslator<T extends JdbcOperation> implemen
}
}
@SuppressWarnings("WeakerAccess")
protected void renderOffset(Expression offsetExpression, boolean renderOffsetRowsKeyword) {
appendSql( " offset " );
clauseStack.push( Clause.OFFSET );
@ -2293,7 +2290,6 @@ public abstract class AbstractSqlAstTranslator<T extends JdbcOperation> implemen
}
}
@SuppressWarnings("WeakerAccess")
protected void renderFetch(
Expression fetchExpression,
Expression offsetExpressionToAdd,
@ -3591,7 +3587,6 @@ public abstract class AbstractSqlAstTranslator<T extends JdbcOperation> implemen
}
}
@SuppressWarnings("WeakerAccess")
protected void renderRootTableGroup(TableGroup tableGroup, List<TableGroupJoin> tableGroupJoinCollector) {
final LockMode effectiveLockMode = getEffectiveLockMode( tableGroup.getSourceAlias() );
final boolean usesLockHint = renderPrimaryTableReference( tableGroup, effectiveLockMode );
@ -3759,7 +3754,6 @@ public abstract class AbstractSqlAstTranslator<T extends JdbcOperation> implemen
return false;
}
@SuppressWarnings("WeakerAccess")
protected boolean renderNamedTableReference(NamedTableReference tableReference, LockMode lockMode) {
appendSql( tableReference.getTableExpression() );
registerAffectedTable( tableReference );
@ -3875,7 +3869,6 @@ public abstract class AbstractSqlAstTranslator<T extends JdbcOperation> implemen
affectedTableNames.add( tableExpression );
}
@SuppressWarnings("WeakerAccess")
protected void renderTableReferenceJoins(TableGroup tableGroup) {
final List<TableReferenceJoin> joins = tableGroup.getTableReferenceJoins();
if ( joins == null || joins.isEmpty() ) {
@ -3896,17 +3889,14 @@ public abstract class AbstractSqlAstTranslator<T extends JdbcOperation> implemen
}
}
@SuppressWarnings("WeakerAccess")
protected void processTableGroupJoins(TableGroup source) {
source.visitTableGroupJoins( tableGroupJoin -> processTableGroupJoin( tableGroupJoin, null ) );
}
@SuppressWarnings("WeakerAccess")
protected void processNestedTableGroupJoins(TableGroup source, List<TableGroupJoin> tableGroupJoinCollector) {
source.visitNestedTableGroupJoins( tableGroupJoin -> processTableGroupJoin( tableGroupJoin, tableGroupJoinCollector ) );
}
@SuppressWarnings("WeakerAccess")
protected void processTableGroupJoin(TableGroupJoin tableGroupJoin, List<TableGroupJoin> tableGroupJoinCollector) {
final TableGroup joinedGroup = tableGroupJoin.getJoinedGroup();
final TableGroup realTableGroup;

View File

@ -30,7 +30,6 @@ public class CteTableGroup extends AbstractTableGroup {
this( false, cteTableReference );
}
@SuppressWarnings("WeakerAccess")
public CteTableGroup(boolean canUseInnerJoins, NamedTableReference cteTableReference) {
super(
canUseInnerJoins,

View File

@ -33,7 +33,6 @@ public abstract class AbstractTableGroup extends AbstractColumnReferenceQualifie
private final SessionFactoryImplementor sessionFactory;
@SuppressWarnings("WeakerAccess")
public AbstractTableGroup(
boolean canUseInnerJoins,
NavigablePath navigablePath,

View File

@ -178,7 +178,6 @@ public class StandardTableGroup extends AbstractTableGroup {
return null;
}
@SuppressWarnings("WeakerAccess")
protected TableReference potentiallyCreateTableReference(String tableExpression) {
final TableReferenceJoin join = tableReferenceJoinCreator.apply( tableExpression, this );
if ( join != null ) {

Some files were not shown because too many files have changed in this diff Show More