Tassilo Karge

Results 14 comments of Tassilo Karge

@beikov this may be interesting to you, judging from the stuff from me you reviewed before

@sebersole mentioned in the discussion on Jira, that the PR would not be accepted with the current implementation. Therefore I would like to have a discussion about how to fix...

> Add a setting which opts in to this behavior. Easy. You mean a setting that is read by the EntityManagerFactoryBuilderImpl and the SessionFactoryOptionsBuilder to then generate EntityManager / Session...

Copying the iterator on access is now optional and a setting (`hibernate.safe_managed_entities_iterator`) and false by default, to avoid affecting any existing users.

I just fixed the checkstyle warning. @sebersole I wondered, if it is possible to use this line instead of my custom iterator: ```Java return Arrays.stream(entityEntryContext.reentrantSafeEntityEntries()).map(Entry::getKey).iterator(); ``` I am not sure...

I have added an additional test case showing that an entity entry that is created by `em.persist()` is created with `isExistsInDatabase() == false` instead of evaluating whether the entity is...

@FrankEssenberger I just came across the same issue, however I would solve it in a different place: The problem is that `to_dict()` is called from `api_client.py` in the method `sanitize_for_serialization`....

I created the pull request, if you want to take a look: https://github.com/OpenAPITools/openapi-generator/pull/19405

New PR to main as requested in #6458