Skip to content

Upgrade Hibernate ORM to 6.5 #1870

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

Closed
wants to merge 14 commits into from
Closed

Conversation

DavideD
Copy link
Member

@DavideD DavideD commented Apr 3, 2024

Fix #1857

Requires next Hibernate ORM release and it currently uses the snapshots.

One test related to the @Generated annotation is disabled for MySQL and MariaDB.

resultSetAccess.release();
}

public void finishRowProcessing(RowProcessingState rowProcessingState, boolean wasAdded) {

Check notice

Code scanning / CodeQL

Useless parameter

The parameter 'rowProcessingState' is never used.
Comment on lines +180 to +192
mutationGroup.forEachOperation( (position, operation) -> {
final EntityTableMapping tableDetails = (EntityTableMapping) operation.getTableDetails();
if ( tableInclusionChecker.include( tableDetails ) ) {
final int[] attributeIndexes = tableDetails.getAttributeIndexes();
for ( final int attributeIndex : attributeIndexes ) {
if ( propertyInclusions[attributeIndex] ) {
final AttributeMapping mapping = entityPersister().getAttributeMappings().get( attributeIndex );
decomposeAttribute( values[attributeIndex], session, jdbcValueBindings, mapping );
}
}
}
} );

Check notice

Code scanning / CodeQL

Deprecated method or constructor invocation

Invoking [MutationOperationGroup.forEachOperation](1) should be avoided because it has been deprecated.
Comment on lines +193 to +202
mutationGroup.forEachOperation( (position, jdbcOperation) -> {
if ( id == null ) {
assert entityPersister().getIdentityInsertDelegate() != null;
}
else {
final EntityTableMapping tableDetails = (EntityTableMapping) jdbcOperation.getTableDetails();
breakDownJdbcValue( id, session, jdbcValueBindings, tableDetails );
}
} );

Check notice

Code scanning / CodeQL

Deprecated method or constructor invocation

Invoking [MutationOperationGroup.forEachOperation](1) should be avoided because it has been deprecated.

mutationGroup.forEachOperation( (position, jdbcOperation) -> {
if ( id == null ) {
assert entityPersister().getIdentityInsertDelegate() != null;

Check notice

Code scanning / CodeQL

Deprecated method or constructor invocation

Invoking [EntityMutationTarget.getIdentityInsertDelegate](1) should be avoided because it has been deprecated.
@DavideD DavideD force-pushed the wip/orm-6.5 branch 2 times, most recently from f4ef46d to 3fa01d7 Compare April 3, 2024 16:14

default CompletionStage<GeneratedValues> performReactiveNonBatchedOperations(
Object modelReference,
ValuesAnalysis valuesAnalysis,

Check notice

Code scanning / CodeQL

Useless parameter

The parameter 'valuesAnalysis' is never used.
@@ -141,7 +141,8 @@
}
final JdbcValueBindings jdbcValueBindings = mutationExecutor.getJdbcValueBindings();
bindPartitionColumnValueBindings( loadedState, session, jdbcValueBindings );
applyId( id, rowId, mutationExecutor, getStaticDeleteGroup(), session );
applyId( id, rowId, mutationExecutor, getStaticMutationOperationGroup(), session );
String[] identifierColumnNames = entityPersister().getIdentifierColumnNames();

Check notice

Code scanning / CodeQL

Unread local variable

Variable 'String\[\] identifierColumnNames' is never read.
@DavideD DavideD closed this Apr 15, 2024
@DavideD DavideD deleted the wip/orm-6.5 branch April 19, 2024 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Check changes required to upgrade to Hibernate ORM 6.5
1 participant