-
Notifications
You must be signed in to change notification settings - Fork 3.6k
HHH-15086 Introduce configurable query cache layout #7543
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
CodeQL found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.
Thanks for your pull request! This pull request appears to follow the contribution rules. › This message was automatically generated. |
0cf03da
to
0a63c1e
Compare
...te-core/src/test/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowCollectionTest.java
Fixed
Show fixed
Hide fixed
...e-core/src/test/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowMultiselectTest.java
Fixed
Show fixed
Hide fixed
...te-core/src/test/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowCollectionTest.java
Fixed
Show fixed
Hide fixed
...e-core/src/test/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowMultiselectTest.java
Fixed
Show fixed
Hide fixed
...te-core/src/test/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowCollectionTest.java
Fixed
Show fixed
Hide fixed
...ava/org/hibernate/orm/test/jpa/query/CachedQueryShallowWithDiscriminatorPolymorphicTest.java
Fixed
Show fixed
Hide fixed
...ava/org/hibernate/orm/test/jpa/query/CachedQueryShallowWithDiscriminatorPolymorphicTest.java
Fixed
Show fixed
Hide fixed
...ava/org/hibernate/orm/test/jpa/query/CachedQueryShallowWithDiscriminatorPolymorphicTest.java
Fixed
Show fixed
Hide fixed
...ava/org/hibernate/orm/test/jpa/query/CachedQueryShallowWithDiscriminatorPolymorphicTest.java
Fixed
Show fixed
Hide fixed
.../src/main/java/org/hibernate/metamodel/mapping/internal/DiscriminatedAssociationMapping.java
Fixed
Show fixed
Hide fixed
...t/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowCollectionNestedJoinFetchTest.java
Fixed
Show fixed
Hide fixed
...t/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowCollectionNestedJoinFetchTest.java
Fixed
Show fixed
Hide fixed
...t/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowCollectionNestedJoinFetchTest.java
Fixed
Show fixed
Hide fixed
...t/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowCollectionNestedJoinFetchTest.java
Fixed
Show fixed
Hide fixed
...t/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowCollectionNestedJoinFetchTest.java
Fixed
Show fixed
Hide fixed
.../src/test/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowWithJoinFetchLazyTest.java
Fixed
Show fixed
Hide fixed
.../src/test/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowWithJoinFetchLazyTest.java
Fixed
Show fixed
Hide fixed
.../src/test/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowWithJoinFetchLazyTest.java
Fixed
Show fixed
Hide fixed
.../src/test/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowWithJoinFetchLazyTest.java
Fixed
Show fixed
Hide fixed
.../src/test/java/org/hibernate/orm/test/jpa/query/CachedQueryShallowWithJoinFetchLazyTest.java
Fixed
Show fixed
Hide fixed
8534062
to
46001c2
Compare
); | ||
final CollectionPersister persister = collectionAttributeMapping.getCollectionDescriptor(); | ||
// Try to reuse the previous collection key and collection if possible | ||
if ( oldKey != null && persister.getKeyType().isEqual( oldKey.getKey(), collectionKeyValue ) ) { |
Check notice
Code scanning / CodeQL
Deprecated method or constructor invocation
return produceInitializer( | ||
navigablePath, | ||
attribute, | ||
parentAccess, | ||
lockMode, | ||
collectionKeyAssembler, | ||
collectionValueKeyAssembler, | ||
creationState | ||
); |
Check notice
Code scanning / CodeQL
Deprecated method or constructor invocation
this.fetches = creationState.visitFetches( this ); | ||
assert fetches.size() == 2; | ||
|
||
discriminatorValueFetch = fetches.get( graphedPart.getDiscriminatorPart() ); |
Check notice
Code scanning / CodeQL
Deprecated method or constructor invocation
|
||
@Override | ||
public Fetch findFetch(Fetchable fetchable) { | ||
if ( graphedPart.getDiscriminatorPart() == fetchable ) { |
Check notice
Code scanning / CodeQL
Deprecated method or constructor invocation
getConcreteDescriptor().getEntityName(), | ||
uniqueKeyAttributePath, | ||
rowProcessingState.getEntityUniqueKey(), | ||
getConcreteDescriptor().getPropertyType( uniqueKeyAttributePath ), |
Check notice
Code scanning / CodeQL
Deprecated method or constructor invocation
void finishRowProcessing(RowProcessingState rowProcessingState); | ||
|
||
default void finishRowProcessing(RowProcessingState rowProcessingState, boolean wasAdded) { | ||
finishRowProcessing( rowProcessingState ); |
Check notice
Code scanning / CodeQL
Deprecated method or constructor invocation
* Callback at the end of processing the current "row" | ||
*/ | ||
default void finishRowProcessing(boolean wasAdded) { | ||
finishRowProcessing(); |
Check notice
Code scanning / CodeQL
Deprecated method or constructor invocation
https://hibernate.atlassian.net/browse/HHH-15086