aratno commented on code in PR #4192:
URL: https://github.com/apache/cassandra/pull/4192#discussion_r2165035176
##########
src/java/org/apache/cassandra/replication/Shard.java:
##########
@@ -158,7 +158,25 @@ private CoordinatorLog getOrCreate(CoordinatorLogId logId)
private CoordinatorLog getOrCreate(long logId)
{
CoordinatorLog log = logs.get(logId);
- return log != null
- ? log : logs.computeIfAbsent(logId, ignore ->
CoordinatorLog.create(localHostId, new CoordinatorLogId(logId), participants));
+ if (log != null)
+ return log;
+ CoordinatorLog newLog = logs.computeIfAbsent(logId, ignore ->
CoordinatorLog.create(localHostId, new CoordinatorLogId(logId), participants));
Review Comment:
Yeah, I agree, that was changed from long to CoordinatorLogId alongside the
recent SRP changes.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
