Change in ovirt-engine[master]: core: Remove synchronization of OVirtNodeInfo getInstance

tnisan at redhat.com tnisan at redhat.com
Thu Oct 27 09:32:12 UTC 2016


Tal Nisan has submitted this change and it was merged.

Change subject: core: Remove synchronization of OVirtNodeInfo getInstance
......................................................................


core: Remove synchronization of OVirtNodeInfo getInstance

Commit 0736d3c2 removed the pointless null check before the double
locking check, this patch now removes the synchronized from the
getInstance method as the actual initialization part is taking part in
a synchronized block

Change-Id: Ibba279dcca897b4496ff90f98449c29bb8adf5dc
Signed-off-by: Tal Nisan <tnisan at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/hostdeploy/OVirtNodeInfo.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Tal Nisan: Verified
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/65785
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibba279dcca897b4496ff90f98449c29bb8adf5dc
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list