Gavin King

Excellent, thanks everyone!

So I believe—but I'm not sure, without digging more deeply—that we actually permit the provider to throw `OptimisticLockException` synchronously from `persist()` or `remove()`. Usually that happens from `flush()` but I...

> for stateful entity manager, there is no database interaction until flush, remove() doesn't know the entity is stale. That is one possible implementation, sure. But what the _spec actually...

> Current mappedBy refers to the owning side of a bidirectional relationship. It's true that it is currently limited to that specific case. > At the database level this is...

> The way I understand your point is, all these mappedBys define a redundant, secondary attribute. Yes. > However in case of a bidirectional relationship the corresponding primary attribute is...

> How would you solve the ambiguity in `OneToOne` though? Aaaaah, you're right, that is a very good point. I had missed that. I fact, it could be reasonably disambiguated...

I'm not keen on this, it reads a bit ambiguously: does the expression `author.books` refer to the `Set`, or the contained `Book`s, i.e. is the collection flattened out or not?...

> Braced primarily because it is shortest among them. It's short, but it's also the option that looks most out of place in the SQL-like syntax. I personally don't find...

> Question is how do we handle `ZonedDateTime` and `Instant` but it may be implicit with `OffsetDateTime` already. I don't think we actually need `Instant` literals, all you would need...

Right, this is an argument that @sebersole also makes.