Skip to content

Conversation

gavinking
Copy link
Member

@gavinking gavinking commented Sep 4, 2025

This is just what I came up with on the flight MEL->SIN.

Need to let it brew a bit more.

See:


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.


@hibernate-github-bot
Copy link

hibernate-github-bot bot commented Sep 4, 2025

Thanks for your pull request!

This pull request does not follow the contribution rules. Could you have a look?

❌ All commit messages should start with a JIRA issue key matching pattern HHH-\d+
    ↳ Offending commits: [49aa3bd, bfb9fca, a38954e, 6f147f2, c1c58a1, 9c5fc24, bd11c38, 2c0b1ea, 4f23a7c]

› This message was automatically generated.

@gavinking gavinking force-pushed the assigments branch 4 times, most recently from 2abb8d8 to e7ba103 Compare September 5, 2025 20:10
@gavinking gavinking changed the title rough prototype for Assignments and ProjectionSpecifications prototype for Assignments and ProjectionSpecifications Sep 28, 2025
scope.inTransaction( session -> {
var builder = session.getCriteriaBuilder();
var query = builder.createCriteriaDelete( Book.class );
var root = query.from( Book.class );

Check notice

Code scanning / CodeQL

Unread local variable Note test

Variable 'JpaRoot root' is never read.
@gavinking gavinking marked this pull request as ready for review September 29, 2025 08:21
@gavinking
Copy link
Member Author

So this has been sitting here for almost a month with no feedback.

I spent a bit of time yesterday on polishing it, and I think it would be better to just integrate it, with those great big warnings that ProjctionSpecification and SimpleProjectionSpecification are incubating and expected to change.

I'm more likely to get useful feedback once someone actually tries it out.

@gavinking gavinking merged commit bde5f78 into hibernate:main Sep 29, 2025
24 of 25 checks passed
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.

2 participants