[
https://issues.apache.org/jira/browse/BOOKKEEPER-1057?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16006571#comment-16006571
]
ASF GitHub Bot commented on BOOKKEEPER-1057:
--------------------------------------------
Github user jvrao commented on the issue:
https://github.com/apache/bookkeeper/pull/145
We (plan to) use error messages to do a lot of post processing. We would
like to see all errors logged as it has much more information than pure log
message.
On Wed, May 10, 2017 at 11:55 PM, Enrico Olivelli <notificati...@github.com>
wrote:
> @jvrao <https://github.com/jvrao> @merlimat <https://github.com/merlimat>
> IHMO In case of a failed bookie it is not good to spam logs with errors.
> It is enough that the method returns an error code.
>
> I am OK if you prefer to distinguish the pointed case of "recovery" from
> the other cases.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/bookkeeper/pull/145#issuecomment-300699854>,
> or mute the thread
>
<https://github.com/notifications/unsubscribe-auth/AAChrnYyIYzjZAaTTALDvpBzBJOxi0WVks5r4rD4gaJpZM4NXBv9>
> .
>
--
Jvrao
---
First they ignore you, then they laugh at you, then they fight you, then
you win. - Mahatma Gandhi
> Do not log error message after read failure in PendingReadOp
> ------------------------------------------------------------
>
> Key: BOOKKEEPER-1057
> URL: https://issues.apache.org/jira/browse/BOOKKEEPER-1057
> Project: Bookkeeper
> Issue Type: Task
> Reporter: Matteo Merli
> Assignee: Matteo Merli
> Priority: Trivial
> Fix For: 4.5.0
>
>
> In {{PendingReadOp}}, there is an error message that is printed each time a
> read on a specific bookie is failing:
> {noformat}
> LOG.error("Read of ledger entry failed: L{} E{}-E{}, Heard from {}. First
> unread entry is {}",
> new Object[] { lh.getId(), startEntryId, endEntryId, heardFromHosts,
> firstUnread });
> {noformat}
> This message is getting printed each time a ledger is recovered and there is
> no error, since the ledger recovery logic is to keep reading and incrementing
> the entryId until a NoEntry error is received.
> This log message should be set at debug level.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
