ivanzlenko commented on code in PR #5101:
URL: https://github.com/apache/ignite-3/pull/5101#discussion_r1931492005
##########
modules/metastorage/src/main/java/org/apache/ignite/internal/metastorage/server/time/ClusterTimeImpl.java:
##########
@@ -91,13 +100,17 @@ public ClusterTimeImpl(String nodeName, IgniteSpinBusyLock
busyLock, HybridClock
*
* @param syncTimeAction Action that performs the time sync operation.
*/
- public void startSafeTimeScheduler(SyncTimeAction syncTimeAction,
MetaStorageConfiguration configuration) {
+ public void startSafeTimeScheduler(SyncTimeAction syncTimeAction,
MetaStorageConfiguration configuration, long term) {
if (!busyLock.enterBusy()) {
return;
}
try {
synchronized (this) {
+ if (lastSchedulerStoppedTerm > term) {
Review Comment:
Ah, okay, it's just for local variables. But yeah, it's better to have a
value.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscr...@ignite.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
