Maor Lipchuk has posted comments on this change.
Change subject: core: Add log when task could not end action.
......................................................................
Patch Set 4: (3 inline comments)
....................................................
File
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/EntityMultiAsyncTasks.java
Line 205: if (task.getParameters() != null
Line 206: &&
task.getParameters().getDbAsyncTask() != null
Line 207: &&
task.getParameters().getDbAsyncTask().getaction_type() == getActionType()
Line 208: && (task.getState() ==
AsyncTaskState.Initializing || task.getState() == AsyncTaskState.WaitForPoll)) {
Line 209: log.debugFormat("Task id {0} Changed its
task state from {1} to polling.",
regarding the polling state, I don't think it is that relevant, but I don't
mind changing it.
will change the syntax issues
Line 210: task.getTaskID(),
Line 211: task.getState());
Line 212: task.setState(AsyncTaskState.Polling);
Line 213: }
Line 223: for (EntityAsyncTask task : _listTasks.values()) {
Line 224: if (task.getParameters() != null &&
task.getParameters().getDbAsyncTask() != null
Line 225: &&
task.getParameters().getDbAsyncTask().getaction_type() == currTaskActionType
Line 226: && task.getState() ==
AsyncTaskState.Initializing) {
Line 227: log.debugFormat("Task id {0} Changed its
task state from {1} to wait for poll.",
done
Line 228: task.getTaskID(),
Line 229: task.getState());
Line 230: task.setState(AsyncTaskState.WaitForPoll);
Line 231: }
Line 242: public boolean getAllCleared() {
Line 243: synchronized (_listTasks) {
Line 244: for (EntityAsyncTask task : _listTasks.values()) {
Line 245: if (!taskWasCleared(task)) {
Line 246: log.infoFormat("[within thread]: Some of the
entity {0} were not cleared yet (Task {1} is in state {2}).",
done
Line 247: getContainerId(),
Line 248: task.getTaskID(),
Line 249: task.getState());
Line 250: return false;
--
To view, visit http://gerrit.ovirt.org/14433
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia1e4700676622b23c79a388b6aa3b92ae1aa15e9
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <mlipc...@redhat.com>
Gerrit-Reviewer: Liron Ar <lara...@redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com>
Gerrit-Reviewer: Michael Kublin <mkub...@redhat.com>
Gerrit-Reviewer: Vered Volansky <vvola...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
