This is an automated email from the ASF dual-hosted git repository. mmerli pushed a commit to branch branch-4.14 in repository https://gitbox.apache.org/repos/asf/bookkeeper.git
commit a80d0438c1beeabe1f477836c423d1d1981457b9 Author: Andras Beni <[email protected]> AuthorDate: Mon Mar 21 16:06:01 2022 +0100 Log NoLedgerException on debug level (#3117) NoLedgerException does not signify an error in the Bookie that needs to be fixed. Instead it is - at most - a user error that the user is notified about via the status code ENOLEDGER. Logging this problem at error level introduces an odd difference between the behavior of readLac using v2 versus v3 protocol version. In the former case ReadEntryProcessor logs the same problem at debug level. As a result changing protocol version appers to be introducing an error. --- .../src/main/java/org/apache/bookkeeper/proto/ReadLacProcessorV3.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/ReadLacProcessorV3.java b/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/ReadLacProcessorV3.java index a3bc311..c7757bb 100644 --- a/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/ReadLacProcessorV3.java +++ b/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/ReadLacProcessorV3.java @@ -74,7 +74,7 @@ class ReadLacProcessorV3 extends PacketProcessorBaseV3 implements Runnable { } } catch (Bookie.NoLedgerException e) { status = StatusCode.ENOLEDGER; - logger.error("No ledger found while performing readLac from ledger: {}", ledgerId, e); + logger.debug("No ledger found while performing readLac from ledger: {}", ledgerId, e); } catch (IOException e) { status = StatusCode.EIO; logger.error("IOException while performing readLac from ledger: {}", ledgerId); @@ -89,7 +89,7 @@ class ReadLacProcessorV3 extends PacketProcessorBaseV3 implements Runnable { } } catch (Bookie.NoLedgerException e) { status = StatusCode.ENOLEDGER; - logger.error("No ledger found while trying to read last entry: {}", ledgerId, e); + logger.debug("No ledger found while trying to read last entry: {}", ledgerId, e); } catch (IOException e) { status = StatusCode.EIO; logger.error("IOException while trying to read last entry: {}", ledgerId, e);
