You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by mg...@apache.org on 2014/03/14 10:34:02 UTC
git commit: WICKET-5241 RequestLogger's server duration does not
include 'detach duration'
Repository: wicket
Updated Branches:
refs/heads/5241-request-logger-duration 1d9ef1107 -> 48eaae68f
WICKET-5241 RequestLogger's server duration does not include 'detach duration'
Remove detaching of the pages in RequestAdapter#commitRequest().
The page has been detached few calls ago:
In org.apache.wicket.request.cycle.RequestCycle#onDetach:
try
{
requestHandlerExecutor.detach(); // detaches pages #1
}
finally
{
listeners.onDetach(this); // detaches pages #2 (no more)
}
Project: http://git-wip-us.apache.org/repos/asf/wicket/repo
Commit: http://git-wip-us.apache.org/repos/asf/wicket/commit/48eaae68
Tree: http://git-wip-us.apache.org/repos/asf/wicket/tree/48eaae68
Diff: http://git-wip-us.apache.org/repos/asf/wicket/diff/48eaae68
Branch: refs/heads/5241-request-logger-duration
Commit: 48eaae68fe0327b421cb2ffb56912cd180bf6728
Parents: 1d9ef11
Author: Martin Tzvetanov Grigorov <mg...@apache.org>
Authored: Fri Mar 14 11:28:56 2014 +0200
Committer: Martin Tzvetanov Grigorov <mg...@apache.org>
Committed: Fri Mar 14 11:28:56 2014 +0200
----------------------------------------------------------------------
.../main/java/org/apache/wicket/page/RequestAdapter.java | 11 +----------
1 file changed, 1 insertion(+), 10 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/wicket/blob/48eaae68/wicket-core/src/main/java/org/apache/wicket/page/RequestAdapter.java
----------------------------------------------------------------------
diff --git a/wicket-core/src/main/java/org/apache/wicket/page/RequestAdapter.java b/wicket-core/src/main/java/org/apache/wicket/page/RequestAdapter.java
index 23b1d30..01f1392 100644
--- a/wicket-core/src/main/java/org/apache/wicket/page/RequestAdapter.java
+++ b/wicket-core/src/main/java/org/apache/wicket/page/RequestAdapter.java
@@ -151,15 +151,6 @@ public abstract class RequestAdapter
touchedPages.size());
for (IManageablePage page : touchedPages)
{
- try
- {
- page.detach();
- }
- catch (Exception e)
- {
- log.error("Error detaching page", e);
- }
-
boolean isPageStateless;
try
{
@@ -182,4 +173,4 @@ public abstract class RequestAdapter
}
}
}
-}
\ No newline at end of file
+}