lots of cleanups and minor refactorings to results stuff #11548

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.

Sign up for GitHub

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

Merged

gavinking merged 9 commits into hibernate:main from gavinking:bigchset

Jan 4, 2026

Merged

lots of cleanups and minor refactorings to results stuff #11548

minor cleanups in results.spi

GitHub Advanced Security / CodeQL succeeded Jan 4, 2026 in 13s

3 new alerts

New alerts in code changed by this pull request

1 warning
2 notes

See annotations below for details.

View all branch alerts.

Annotations

Check warning on line 1177 in hibernate-core/src/main/java/org/hibernate/query/sqm/mutation/internal/temptable/TableBasedInsertHandler.java

Code scanning / CodeQL

Dereferenced variable may be null Warning

Variable

may be null at this access because of

assignment.

Check notice on line 168 in hibernate-core/src/main/java/org/hibernate/query/results/internal/dynamic/DynamicFetchBuilderLegacy.java

Code scanning / CodeQL

Possible confusion of local and field Note

Potentially confusing name: method

also refers to field

(as this.fetchBuilderMap).

Check notice on line 18 in hibernate-core/src/main/java/org/hibernate/sql/ast/tree/insert/Values.java

Code scanning / CodeQL

Exposing internal representation Note

getExpressions exposes the internal representation stored in field expressions. The value may be modified

.
getExpressions exposes the internal representation stored in field expressions. The value may be modified

.
getExpressions exposes the internal representation stored in field expressions. The value may be modified

.