Change in ovirt-engine[master]: core: use ThreadPoolUtil for external scheduler discovery

rgolan at redhat.com rgolan at redhat.com
Sun Aug 2 21:25:47 UTC 2015


Roy Golan has submitted this change and it was merged.

Change subject: core: use ThreadPoolUtil for external scheduler discovery
......................................................................


core: use ThreadPoolUtil for external scheduler discovery

Use Runnable instead of new Thread. Only our ThreadPoolUtil should
supply that resources and we should keep it.

Also SchedulingManager dependency has been dropped the responsibility of
reloading moved into SchedulingManager

Change-Id: If377e0241190d55b6d0960ff456717b3707edb34
Signed-off-by: Roy Golan <rgolan at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/SchedulingManager.java
R backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/external/ExternalSchedulerDiscovery.java
2 files changed, 41 insertions(+), 30 deletions(-)

Approvals:
  Jenkins CI: Passed CI tests
  Roy Golan: Verified
  Moti Asayag: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: If377e0241190d55b6d0960ff456717b3707edb34
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofrenkel at redhat.com>
Gerrit-Reviewer: Roman Mohr <rmohr at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list