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 2021/07/13 16:12:40 UTC
[GitHub] [trafficserver] shinrich opened a new pull request #8058: Replace assert with warning in error event processing
shinrich opened a new pull request #8058:
URL: https://github.com/apache/trafficserver/pull/8058
Fixes an issue introduced in PR #7809.
Replaced the ink_release_assert with a warning so we can gain information about this situation.
--
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
[GitHub] [trafficserver] shinrich merged pull request #8058: Replace assert with warning in error event processing
Posted by GitBox <gi...@apache.org>.
shinrich merged pull request #8058:
URL: https://github.com/apache/trafficserver/pull/8058
--
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
[GitHub] [trafficserver] zwoop commented on pull request #8058: Replace assert with warning in error event processing
Posted by GitBox <gi...@apache.org>.
zwoop commented on pull request #8058:
URL: https://github.com/apache/trafficserver/pull/8058#issuecomment-883738487
Cherry-picked to v9.1.x branch.
--
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
[GitHub] [trafficserver] shinrich commented on pull request #8058: Replace assert with warning in error event processing
Posted by GitBox <gi...@apache.org>.
shinrich commented on pull request #8058:
URL: https://github.com/apache/trafficserver/pull/8058#issuecomment-879272250
We have been running this in our production. Vinith looked up the messages we received. 50 over our footprint in a 4 hour period. Most of them are just EPOLLERR (0x8). A least one also had EPOLLHUP (0x18). May be ok to just ignore them. Or to pass them along via a write event as this PR does.
--
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
[GitHub] [trafficserver] shinrich merged pull request #8058: Replace assert with warning in error event processing
Posted by GitBox <gi...@apache.org>.
shinrich merged pull request #8058:
URL: https://github.com/apache/trafficserver/pull/8058
--
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
[GitHub] [trafficserver] shinrich merged pull request #8058: Replace assert with warning in error event processing
Posted by GitBox <gi...@apache.org>.
shinrich merged pull request #8058:
URL: https://github.com/apache/trafficserver/pull/8058
--
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