jt2594838 commented on a change in pull request #1319:
URL: https://github.com/apache/incubator-iotdb/pull/1319#discussion_r436427656
##########
File path:
cluster/src/main/java/org/apache/iotdb/cluster/log/catchup/CatchUpTask.java
##########
@@ -55,28 +55,32 @@ public CatchUpTask(Node node, Peer peer, RaftMember
raftMember) {
}
/**
- *
* @return true if the matched index exceed the memory log bound so a
snapshot is necessary, or
* false if the catch up can be done using memory logs.
* @throws TException
* @throws InterruptedException
*/
boolean checkMatchIndex() throws TException, InterruptedException,
LeaderUnknownException {
+ boolean isLogDebug = logger.isDebugEnabled();
+ long lo = -1, hi;
Review comment:
I think it is fine to put multiple variables in a single line, but this
will add Sonar code smells, so it is better to put them into separate lines.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
