rdhabalia commented on a change in pull request #2373:
URL: https://github.com/apache/bookkeeper/pull/2373#discussion_r453934567
##########
File path:
bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/EntryLogger.java
##########
@@ -1011,6 +1011,11 @@ public void scanEntryLog(long entryLogId,
EntryLogScanner scanner) throws IOExce
// read the entry
data.clear();
+ if (entrySize < 0) {
Review comment:
as we don't know root cause, does it make sense to add a flag to skip
entry-log on such failure? It can be helpful to users who don't want to skip
and validate actual corruption.?
----------------------------------------------------------------
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]
