Bug 59205 - Connect time metric in "View Results Tree" is always equal zero for TCPSampler.
Summary: Connect time metric in "View Results Tree" is always equal zero for TCPSampler.
Status: VERIFIED FIXED
Alias: None
Product: JMeter - Now in Github
Classification: Unclassified
Component: Main (show other bugs)
Version: 2.13
Hardware: PC All
: P2 normal (vote)
Target Milestone: ---
Assignee: JMeter issues mailing list
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2016-03-21 12:33 UTC by kpishnik
Modified: 2016-03-23 14:18 UTC (History)
1 user (show)


Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description kpishnik 2016-03-21 12:33:36 UTC 
New feature connection time metric(added in 2.13) does not work for TCPSampler.

It's always equal zero.

I have tried for "View Results Tree".

Everything is OK for HTTP Request.

I use "LengthPrefixedBinaryTCPClientImpl".
Comment 1 Felix Schumacher 2016-03-21 18:18:42 UTC 
Could you test the next nightly, whether this fix is correct for you?

Date: Mon Mar 21 18:17:03 2016
New Revision: 1736072

URL: http://svn.apache.org/viewvc?rev=1736072&view=rev
Log:
TCP Sampler: Set connect time in sampler when connection is established.

Bugzilla Id: 59205

Modified:
    jmeter/trunk/src/protocol/tcp/org/apache/jmeter/protocol/tcp/sampler/TCPSampler.java
    jmeter/trunk/xdocs/changes.xml
Comment 2 kpishnik 2016-03-22 08:44:24 UTC 
Now, everything is OK. Thank you.
Comment 3 Philippe Mouawad 2016-03-23 14:18:54 UTC 
Author: pmouawad
Date: Wed Mar 23 14:17:05 2016
New Revision: 1736324

URL: http://svn.apache.org/viewvc?rev=1736324&view=rev
Log:
Bug 59205 TCP Sampler: Set connect time in sampler when connection is established
As per dev mailing list discussion, update documentation and handle to connection error
Bugzilla Id: 59205

Modified:
    jmeter/trunk/src/protocol/tcp/org/apache/jmeter/protocol/tcp/sampler/TCPSampler.java
    jmeter/trunk/xdocs/usermanual/glossary.xml
Comment 4 The ASF infrastructure team 2022-09-24 20:38:03 UTC 
This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/3913