sumitagrawl commented on code in PR #5034:
URL: https://github.com/apache/ozone/pull/5034#discussion_r1259286671
##########
hadoop-hdds/framework/src/main/java/org/apache/hadoop/hdds/utils/db/RDBStore.java:
##########
@@ -347,13 +347,13 @@ public Collection<ColumnFamily> getColumnFamilies() {
@Override
public DBUpdatesWrapper getUpdatesSince(long sequenceNumber)
- throws SequenceNumberNotFoundException {
+ throws IOException {
return getUpdatesSince(sequenceNumber, Long.MAX_VALUE);
}
@Override
public DBUpdatesWrapper getUpdatesSince(long sequenceNumber, long limitCount)
- throws SequenceNumberNotFoundException {
+ throws IOException {
Review Comment:
dbUpdatesWrapper.setLatestSequenceNumber(db.getLatestSequenceNumber());
<-- this is as used down should be moved within try block, so that dbwrapper
status and log can be added.
--
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]
