Where ApacheSolr logs messages from exceptions, it would be very helpful to use watchdog_exception instead of the plain watchdog so that more context can be provided with the error. Patch attached :)

CommentFileSizeAuthor
7.x-apachesolr-watchdog_exception-upgrade.patch7.45 KBjosh waihi

Comments

pwolanin’s picture

pwolanin commented

Hmm, maybe - but we'd lose the multi-line formatting of the java stack trace.

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Needs work » Needs review

Could we maybe get a screenshot with annotations of the improvements? + please prepare a valid patch file :-)

Status: Active » Needs work

The last submitted patch, 7.x-apachesolr-watchdog_exception-upgrade.patch, failed testing.

nick_vh’s picture

nick_vh
he/him
Location Ghent
commented
Status: Needs review » Postponed