Rob Bygrave

Results 140 comments of Rob Bygrave

An empty parentIdList implies the LoadManyRequests batch is empty. That implies the batch was always empty or that the batch was previously lazy loaded (or some sort of recursion where...

> attempt writing a test case again next week if time is allotted to me to continue attempting to fix this issue Yup cool. Note that this is about how...

Ok. It isn't clear why it was closed in the first place per say - there isn't any related commit or PR etc. Lets reopen it.

For BeanPersistListener no. For a BeanPersistController / BeanPersistAdapter *yes* via BeanPersistRequest.updatedValues(). That is, look to use a BeanPersistController postUpdate() instead of BeanPersistListener if you want access to that detail. Cheers,...

So you are either looking for either: A) A change to ChangeLog so that a bean update has access to the bean itself B) A change to BeanPersistListener to have...

> needs some standard account fields to have the full information Just to note that ChangeLog has: ```java /** * Arbitrary user context information expected to be optionally populated by...

Ok, I think we can close this issue for now. We might want to reopen / revisit and come up with a more elegant solution down the track. Perhaps something...

> Since ScalarTypeJsonMap.typeFor does not take into account mutationDetection strategy at all, unlike in createJsonObjectMapperType(), it seems to be the reason why Map field is behaving this way. Yes. >...

Hi, for this specific PR the comment about "this only being used for updateable resultset" makes me wonder if we should make no change here. That is, the use of...

> @rbygrave , thanks for the PR. Sorry, it was sitting so long. No worries. I myself got bogged down with lots of stuff and I know how busy you...