rnewson commented on code in PR #6096:
URL: https://github.com/apache/couchdb/pull/6096#discussion_r4094555959
##########
extra/nouveau/src/main/java/org/apache/couchdb/nouveau/core/IndexManager.java:
##########
@@ -417,7 +417,7 @@ private long getSeq(final IndexWriter writer, final String
key, final long defau
return Long.parseLong(entry.getValue());
}
}
- return 0L;
+ return defaultValue;
Review Comment:
defaultValue will be the purge seq from the erlang side. using 0 would be
wrong, which is what I'm fixing here (though I think line 420 is unreachable in
practice. either the commit data doesn't exist or it does and contains a
"purge_seq" entry.
--
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]
