JosiahWI opened a new pull request, #10827: URL: https://github.com/apache/trafficserver/pull/10827
I could not find a way to do this without moving any logic around. I changed the logic as minimally as possible and split it into 3 commits as follows: - The first commit reorders some of the logic to split it apart and prepare part of it to be extracted to a new method. - The second commit does the extraction. - The third commit cleans up to remove duplication and get rid of `Stripe::add_agg_todo`. The logic change was quite limited and straightforward, so I don't think this one is very risky, but please review carefully anyway. One oddity here is `CACHE_AGG_FAIL_RATE`, which would be defined in `P_CacheInternal.h` if it weren't commented out there. -- 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-unsubscr...@trafficserver.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
