[hibernate-dev] [ORM] Synchronization on AbstractLoadPlanBasedLoader

[Steve Ebersole]

It's very possible that code comments may no longer be pertinent.

On Mon, Dec 14, 2015 at 10:26 AM Sanne Grinovero <sanne at hibernate.org>
wrote:

> Hi all,
> while reviewing an improvement by Stale about reducing
> synchronization, I'm having the impression that the synchronization
> could be completely removed.
>
> But there's a comment warning me against that, so for sake of safety
> I'm merging the improvement without risking going too far:
>
>  // synchronized to avoid multi-thread access issues; defined as
> method synch to avoid
>  // potential deadlock issues due to nature of code.
>
> I tried to figure what "potential deadlock" it's referring to, but I'm
> having the impression the comment might be outdated. So I've reduced
> the contention to the only include the code block about which I'm not
> confident.
> By looking into git history, it seems the comment isn't related to any
> specific fix but was included already when this class was first
> created.
>
> Would someone be able to point out what is the issue this is protecting
> against?
>
> That should allow us to provide an even better patch, although I'll
> apply the safe one for now so to at least have the benefits already
> when wrapping of result-sets is disabled.
>
> thanks,
> Sanne
> _______________________________________________
> hibernate-dev mailing list
> hibernate-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/hibernate-dev
>