Gavin King

Results 1199 comments of Gavin King

> for a change like this we will need spec text and TCK test(s) for it. But let's agree the design first. Yes of course. And there is also impact...

So in #719, after trying out several more radical options, what I ended up converging on was: 1. override and deprecate `executeUpdate()` on `TypedQuery` 2. deprecate all "selection-only" operations on...

The previous pull request for this is all horribly conflicted new by the introduction of `EntityAgent`, but reviewing what I did here, I think this should be revived. I'm thinking...

> A further optional step might be to introduce a dedicated interface for executing update/delete queries So even though we have not yet made the decision to pull the trigger...

> we should start thinking about whether this slightly more radical proposal makes sense I have tried this out in code, and I'm *way* happier with the result than I...

I would now like to prioritize this for inclusion in M1, assuming that others agree that this is something we should actually do (or at least try out in the...

> I would now like to prioritize this for inclusion in M1 Well, no, not before discussion. We need to carefully weigh this proposal against something much less radical. Less...

I have merged [#842](https://github.com/jakartaee/persistence/pull/842) so that we can use M1 as a vehicle to gather feedback about this change.

I have opened #805 implementing this.

After gathering initial feedback from @njr-11, @beikov, and @sebersole, I have decided to merge this functionality with perhaps somewhat less review than I usually prefer. Therefore, I want to emphasize...