[ https://issues.apache.org/jira/browse/WICKET-7110?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17845100#comment-17845100 ]
ASF GitHub Bot commented on WICKET-7110: ---------------------------------------- martin-g commented on code in PR #861: URL: https://github.com/apache/wicket/pull/861#discussion_r1595848322 ########## wicket-core/src/main/java/org/apache/wicket/page/DefaultPageLockManager.java: ########## @@ -222,3 +227,12 @@ Supplier<ConcurrentMap<Integer, PageAccessSynchronizer.PageLock>> getLocks() return locks; } } + +static class PageLockedException extends Exception +{ Review Comment: > this is outside the surrounding class who does this ?! :-) why not create its own class ? because you will need to add a license at the top ? :-) > Add stack trace of lock holding thread in CouldNotLockPageException > ------------------------------------------------------------------- > > Key: WICKET-7110 > URL: https://issues.apache.org/jira/browse/WICKET-7110 > Project: Wicket > Issue Type: Improvement > Reporter: Martijn Dashorst > Assignee: Martijn Dashorst > Priority: Minor > > Currently we only log the stack trace of the thread that keeps the lock on > the page when we can't claim a lock. It is beneficial to set the cause of the > CouldNotLockPageException to an exception that shows the stack trace elements > of the thread holding the lock. > This way you can see in the monitoring what was the cause instead of having > to go look for the holding (e.g. slow) page in the logs. -- This message was sent by Atlassian Jira (v8.20.10#820010)