[
https://issues.apache.org/jira/browse/OAK-9846?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17571766#comment-17571766
]
Konrad Windszus commented on OAK-9846:
--------------------------------------
In addition it would probably make sense to also add logging in time (i.e. not
only deferred), as then the error is surrounded by these log statements:
https://github.com/apache/jackrabbit-oak/blob/e1ef7342234bd233009affa47c0dcdf44e4fed89/oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/checks/NodeStoreChecksService.java#L67-L71
> o.a.j.oak.composite.ErrorHolder should include the name of the checker for
> each report
> --------------------------------------------------------------------------------------
>
> Key: OAK-9846
> URL: https://issues.apache.org/jira/browse/OAK-9846
> Project: Jackrabbit Oak
> Issue Type: Improvement
> Reporter: Konrad Windszus
> Assignee: Konrad Windszus
> Priority: Minor
>
> Currently all {{MountedNodeStoreChecker}} reports are accumulated in
> {{ErrorHolder}} and then given out all together in
> https://github.com/apache/jackrabbit-oak/blob/c6ddcc55bee3de915459af01e91edad32d538f3d/oak-store-composite/src/main/java/org/apache/jackrabbit/oak/composite/checks/ErrorHolder.java#L46-L55.
> Unfortunately those reports do no longer indicate from which
> {{MountedNodeStoreChecker}} service they originate. The origin should be
> given out as well to ease debugging.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
