jmeter-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <>
Subject [GitHub] [jmeter] mjmadhu edited a comment on pull request #640: Fix to 65041
Date Fri, 12 Feb 2021 11:04:23 GMT

mjmadhu edited a comment on pull request #640:

   > Thanks for the PR.
   > In
Philippe has tried another way of implementing this, which was hindered by a small bug. I
think that bug should be fixed.
   > What do you think?
   I think there's a difference in approach. The intent here is also to support timeout values
provided via jmeter properties and hc.parameters files given by the end users. The final HTTP
request made from **** will pick a response timeout from the 3 options
in the following order of precedence (Response timeout in HTTP Sampler/ HTTP Request Defaults
> http.socket.timeout in hc.parameters > httpclient.timeout property in
   It's different from
   1. We're not trying to change/set default timeout value "only" via HTTP sampler, and it
still defaults to 0 if not provided 
   2. This PR is only for response timeout as of now, not including connect timeout.
   If we want to give specific default values for these timeouts, we can still have them in with this feature support.

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.

For queries about this service, please contact Infrastructure at:

View raw message