[GitHub] [trafficserver] lzx404243 opened a new pull request, #9655: rename attempts to retry_attempts

["lzx404243 (via GitHub)" <gi...@apache.org>]

lzx404243 opened a new pull request, #9655:
URL: https://github.com/apache/trafficserver/pull/9655

   From the name, the `HttpTransact::_CurrentInfo.attempts` seems to represent the total connection attempt, but it's currently really the retry attempt:
   - it's initialized to `0`
   - it's incremented only in the OS or parent retry code path
   
   This PR renames this to `HttpTransact::_CurrentInfo.retry_attempts` to avoid confusion. Note that there is no functionality changes. Changes are to just the naming and some related log messages(plus one assertion).


-- 
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] bneradt merged pull request #9655: rename attempts to retry_attempts

["bneradt (via GitHub)" <gi...@apache.org>]

bneradt merged PR #9655:
URL: https://github.com/apache/trafficserver/pull/9655


-- 
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