adoroszlai commented on PR #5034:
URL: https://github.com/apache/ozone/pull/5034#issuecomment-1628582722
> There is one more function getLatestSequenceNumber() which doesn't have
assertClose().
I left that one out because it is not new, was already there when HDDS-7284
added the `assertClose()` checks.
@sumitagrawl can you please confirm if `assertClose()` call was
intentionally omitted from `getLatestSequenceNumber()` in HDDS-7284?
```diff
public long getLatestSequenceNumber() {
- return db.get().getLatestSequenceNumber();
+ try {
+ counter.incrementAndGet();
+ return db.get().getLatestSequenceNumber();
+ } finally {
+ counter.decrementAndGet();
+ }
}
```
--
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]
