You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@trafficserver.apache.org by GitBox <gi...@apache.org> on 2022/04/16 13:18:36 UTC

[GitHub] [trafficserver] bryancall commented on a diff in pull request #8747: Add http2.default_buffer_water_mark config to tune latency

bryancall commented on code in PR #8747:
URL: https://github.com/apache/trafficserver/pull/8747#discussion_r851627057


##########
proxy/http2/Http2ConnectionState.cc:
##########
@@ -1679,6 +1673,12 @@ Http2ConnectionState::send_a_data_frame(Http2Stream *stream, size_t &payload_len
     payload_length = 0;
   }
 
+  if (payload_length > 0 && this->session->is_write_high_water()) {

Review Comment:
   You are not going to send data to the client unless there is 32k to send?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org