Change in ovirt-engine[master]: utils: do not reject tasks

Code Review gerrit at ovirt.org
Fri Apr 7 07:38:21 UTC 2017


>From Martin Peřina <mperina at redhat.com>:

Martin Peřina has submitted this change and it was merged.

Change subject: utils: do not reject tasks
......................................................................


utils: do not reject tasks

The usage of RejectedExecutionHandler was not correct and and as a
result we used ThreadPoolExecutor.AbortPolicy which is default. Using
ThreadPoolExecutor.CallerRunsPolicy was intended and seems like good
idea so we need to make sure that we are going to use this policy.


Bug-Url: https://bugzilla.redhat.com/1438497
Change-Id: Icc4ed0b241b0961c49daa9ec29a905a0a15487f2
Signed-off-by: Piotr Kliczewski <piotr.kliczewski at gmail.com>
---
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/threadpool/ThreadPoolUtil.java
1 file changed, 2 insertions(+), 10 deletions(-)

Approvals:
  Piotr Kliczewski: Verified
  Martin Peřina: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Moti Asayag: Looks good to me, but someone else must approve



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Icc4ed0b241b0961c49daa9ec29a905a0a15487f2
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Moti Asayag <masayag at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: Roy Golan <rgolan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list