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 2020/10/08 18:52:41 UTC
[GitHub] [trafficserver] traeak opened a new pull request #7257: Default to throttling, fixes to manufactured 416 responses.
traeak opened a new pull request #7257:
URL: https://github.com/apache/trafficserver/pull/7257
The slice plugin has a problem when consecutive slice blocks are held in ram cache in that they shoved into the slice plugin without doing any throttling (which is done for other cases like TCP_HIT, etc). This PR makes throttling the default and restores the data handoff to using TSIOBufferReaderAvail and TSIOBufferCopy instead of iterating through the TSIOBuffer blocks.
Also a bug was found with slice's 416 response creator, a bug exposed by the addition of slice self healing.
closes #7256
resolves #7179
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] zwoop commented on pull request #7257: slice: default to throttling, fixes to manufactured 416 responses.
Posted by GitBox <gi...@apache.org>.
zwoop commented on pull request #7257:
URL: https://github.com/apache/trafficserver/pull/7257#issuecomment-763749288
Backported to 9.0.1 in #7435
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] traeak merged pull request #7257: slice: default to throttling, fixes to manufactured 416 responses.
Posted by GitBox <gi...@apache.org>.
traeak merged pull request #7257:
URL: https://github.com/apache/trafficserver/pull/7257
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [trafficserver] traeak merged pull request #7257: slice: default to throttling, fixes to manufactured 416 responses.
Posted by GitBox <gi...@apache.org>.
traeak merged pull request #7257:
URL: https://github.com/apache/trafficserver/pull/7257
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org