ascherbakoff commented on code in PR #4821:
URL: https://github.com/apache/ignite-3/pull/4821#discussion_r1891267978
##########
modules/raft/src/main/java/org/apache/ignite/raft/jraft/core/NodeImpl.java:
##########
@@ -889,7 +905,7 @@ public boolean bootstrap(final BootstrapOptions opts)
throws InterruptedExceptio
final long bootstrapLogTerm = opts.getLastLogIndex() > 0 ? 1 : 0;
final LogId bootstrapId = new LogId(opts.getLastLogIndex(),
bootstrapLogTerm);
this.options = opts.getNodeOptions() == null ? new NodeOptions() :
opts.getNodeOptions();
- this.clock = options.getClock();
+ this.clock = this.options.getClock() == null ? new HybridClockImpl() :
this.options.getClock();
Review Comment:
Currently clocks can't be null, because it's required for safe time
generation.
This logic is required for some tests, which are not setting clocks directly.
--
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
