Skip to content

Conversation

@icing
Copy link
Contributor

@icing icing commented Oct 28, 2025

Timer stats improvements

  • move the TIMER_POSTQUEUE to the time a connection is chosen, so that TIMER_NAMELOOKUP always happens afterwards
  • client writer: do not trigger TIMER_STARTTRANSFER on CLIENTWRITE_INFO as ftp and other pingpong protocols write that before starting anything that is the tranfer itself
  • Elimnating debug trancing of "closed stream/connection - bailing" as confusing, as connection is not really closed on most cases.
  • Setting 'data->req.upload_done` correctly, so that no "abort upload" is happening at the end of a perfectly fine download.
  • Adding test cases with up-/download of 0-length files.
  • pytest: add a "timeline" of timer value checks to Resulst in curl.py, so that this can be used in several test cases, replacing the local stuff in test_16
  • add timeline checks to ftp test cases

@github-actions github-actions bot added the tests label Oct 28, 2025
@icing icing marked this pull request as ready for review October 28, 2025 15:52
@icing icing requested a review from bagder October 28, 2025 15:52
@icing icing force-pushed the timer-stats-improvements branch 2 times, most recently from f0ade0a to e9dba4b Compare November 25, 2025 09:02
* move the TIMER_POSTQUEUE to the time a connection is chosen,
  so that TIMER_NAMELOOKUP always happens afterwards
* client writer: do not trigger TIMER_STARTTRANSFER on CLIENTWRITE_INFO
  as ftp and other pingpong protocols write that before starting anything
  that is the tranfer itself
* Elimnating debug trancing of "closed stream/connection - bailing"
  as confusing, as connection is not really closed on most cases.
* Setting 'data->req.upload_done` correctly, so that no "abort upload"
  is happening at the end of a perfectly fine download.
* Adding test cases with up-/download of 0-length files.
* pytest: add a "timeline" of timer value checks to Resulst in curl.py,
  so that this can be used in several test cases, replacing the local
  stuff in test_16
* add timeline checks to ftp test cases
@icing icing force-pushed the timer-stats-improvements branch from e9dba4b to 57c881d Compare November 25, 2025 09:54
@icing icing requested a review from bagder November 25, 2025 10:31
@bagder bagder closed this in 208a6ae Nov 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants