Author: remm
Date: Thu Nov 9 14:39:49 2017
New Revision: 1814733
URL: http://svn.apache.org/viewvc?rev=1814733&view=rev
Log:
Although I don't understand why yet, the test for BZ61740 demonstrated the
headers write need a sync. This has a performance cost.
Modified:
tomcat/trunk/java/org/apache/coyote/http2/Http2AsyncUpgradeHandler.java
Modified:
tomcat/trunk/java/org/apache/coyote/http2/Http2AsyncUpgradeHandler.java
URL:
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/coyote/http2/Http2AsyncUpgradeHandler.java?rev=1814733&r1=1814732&r2=1814733&view=diff
==============================================================================
--- tomcat/trunk/java/org/apache/coyote/http2/Http2AsyncUpgradeHandler.java
(original)
+++ tomcat/trunk/java/org/apache/coyote/http2/Http2AsyncUpgradeHandler.java Thu
Nov 9 14:39:49 2017
@@ -31,7 +31,6 @@ import java.util.concurrent.TimeUnit;
import org.apache.coyote.Adapter;
import org.apache.coyote.ProtocolException;
import org.apache.coyote.Request;
-import org.apache.tomcat.util.http.MimeHeaders;
import org.apache.tomcat.util.net.SendfileState;
import org.apache.tomcat.util.net.SocketWrapperBase;
import org.apache.tomcat.util.net.SocketWrapperBase.BlockingMode;
@@ -149,16 +148,6 @@ public class Http2AsyncUpgradeHandler ex
}
- @Override
- void writeHeaders(Stream stream, int pushedStreamId, MimeHeaders
mimeHeaders,
- boolean endOfStream, int payloadSize) throws IOException {
- doWriteHeaders(stream, pushedStreamId, mimeHeaders, endOfStream,
payloadSize);
- if (endOfStream) {
- stream.sentEndOfStream();
- }
- }
-
-
@Override
protected HeaderFrameBuffers getHeaderFrameBuffers(int initialPayloadSize)
{
return new AsyncHeaderFrameBuffers(initialPayloadSize);
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
For additional commands, e-mail: dev-h...@tomcat.apache.org
