Eli Mesika has posted comments on this change.

Change subject: engine: Schedule a job for monitoring host's updates
......................................................................


Patch Set 2:

(1 comment)

https://gerrit.ovirt.org/#/c/40460/2/backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java
File 
backend/manager/modules/vdsbroker/src/main/java/org/ovirt/engine/core/vdsbroker/VdsManager.java:

Line 154:                         refreshRate,
Line 155:                         TimeUnit.MILLISECONDS);
Line 156: 
Line 157:         double availableUpdatesRefreshRate = Config.<Double> 
getValue(ConfigValues.HostPackagesUpdateTimeInHours);
Line 158:         final int HOURS_TO_MINUTES = 60;
> If i'll convert the availableUpdatesRefreshRate which is double to long, i'
OK , thanks for the explanation
Line 159:         long rateInMinutes = Math.round(availableUpdatesRefreshRate * 
HOURS_TO_MINUTES);
Line 160: 
Line 161:         availableUpdatesJobId =
Line 162:                 sched.scheduleAFixedDelayJob(


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I2114d00e626c09cd7b8be1f9a4564a0f3c0ba700
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: Eli Mesika <emes...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Moti Asayag <masa...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to