moonchen commented on PR #13491: URL: https://github.com/apache/trafficserver/pull/13491#issuecomment-5218588444
> Approving, although I do not think it is wise to lock this in as a requirement. I don't think this should have an AuTest. This is a performance optimization and we should be able to change our minds and do something else without waiting for a major release. I would argue that adding the AuTest here is an unofficial guarantee that users can rely on this behavior. I understand your concern. I'm not trying to guarantee this behavior going forward with the autest. My goal with the test is to ensure that the code change does what it's supposed to. My understand about autest is that they're not set in stone, even within one major release, and they can be changed when the expected behavior of the code changes. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
