You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by re...@apache.org on 2020/02/28 10:57:59 UTC
[tomcat] branch master updated: Remove useless calls to setStartTime
This is an automated email from the ASF dual-hosted git repository.
remm pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/master by this push:
new 16a6094 Remove useless calls to setStartTime
16a6094 is described below
commit 16a6094dbe5994c0b5214d6f647e53b736748afe
Author: remm <re...@apache.org>
AuthorDate: Fri Feb 28 11:57:48 2020 +0100
Remove useless calls to setStartTime
---
java/org/apache/coyote/AbstractProcessor.java | 1 -
java/org/apache/coyote/ajp/AjpProcessor.java | 1 -
java/org/apache/coyote/http2/Stream.java | 3 ---
3 files changed, 5 deletions(-)
diff --git a/java/org/apache/coyote/AbstractProcessor.java b/java/org/apache/coyote/AbstractProcessor.java
index 5af3710..662e133 100644
--- a/java/org/apache/coyote/AbstractProcessor.java
+++ b/java/org/apache/coyote/AbstractProcessor.java
@@ -977,7 +977,6 @@ public abstract class AbstractProcessor extends AbstractProcessorLight implement
// information (e.g. client IP)
setSocketWrapper(socketWrapper);
// Setup the minimal request information
- request.setStartTime(System.currentTimeMillis());
request.setStartTimeNanos(System.nanoTime());
// Setup the minimal response information
response.setStatus(400);
diff --git a/java/org/apache/coyote/ajp/AjpProcessor.java b/java/org/apache/coyote/ajp/AjpProcessor.java
index 562fc67..f1c5d1f 100644
--- a/java/org/apache/coyote/ajp/AjpProcessor.java
+++ b/java/org/apache/coyote/ajp/AjpProcessor.java
@@ -394,7 +394,6 @@ public class AjpProcessor extends AbstractProcessor {
setErrorState(ErrorState.CLOSE_CONNECTION_NOW, null);
break;
}
- request.setStartTime(System.currentTimeMillis());
request.setStartTimeNanos(System.nanoTime());
} catch (IOException e) {
setErrorState(ErrorState.CLOSE_CONNECTION_NOW, e);
diff --git a/java/org/apache/coyote/http2/Stream.java b/java/org/apache/coyote/http2/Stream.java
index 647ddf6..e25103f 100644
--- a/java/org/apache/coyote/http2/Stream.java
+++ b/java/org/apache/coyote/http2/Stream.java
@@ -129,9 +129,6 @@ class Stream extends AbstractStream implements HeaderEmitter {
this.coyoteResponse.setOutputBuffer(http2OutputBuffer);
this.coyoteRequest.setResponse(coyoteResponse);
this.coyoteRequest.protocol().setString("HTTP/2.0");
- if (this.coyoteRequest.getStartTime() < 0) {
- this.coyoteRequest.setStartTime(System.currentTimeMillis());
- }
if (this.coyoteRequest.getStartTimeNanos() < 0) {
this.coyoteRequest.setStartTimeNanos(System.nanoTime());
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org