You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@jmeter.apache.org by bu...@apache.org on 2016/03/03 13:27:52 UTC

[Bug 59107] New: HTTPSamplerBase.CleanerThread bug?

https://bz.apache.org/bugzilla/show_bug.cgi?id=59107

            Bug ID: 59107
           Summary: HTTPSamplerBase.CleanerThread bug?
           Product: JMeter
           Version: 2.13
          Hardware: PC
                OS: Mac OS X 10.1
            Status: NEW
          Severity: normal
          Priority: P2
         Component: Main
          Assignee: issues@jmeter.apache.org
          Reporter: sebb@apache.org

The method registerSamplerForEndNotification(HTTPSamplerBase) from the type
HTTPSamplerBase.CleanerThread is never used locally.

There does not seem to be any code that adds items to the List samplersToNotify
so AFAICT the CleanerThread class does not actually do anything.

This suggests there may be a bug - surely the code should add some entries to
the list?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 59107] HTTPSamplerBase.CleanerThread bug?

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59107

Philippe Mouawad <p....@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #4 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
Fixed within Bug 52073:
http://svn.apache.org/viewvc?rev=1734228&view=rev

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 59107] HTTPSamplerBase.CleanerThread bug?

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59107

--- Comment #3 from benoit.wiart@gmail.com ---
Should be closed the cleaner thread was removed.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 59107] HTTPSamplerBase.CleanerThread bug?

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59107

Philippe Mouawad <p....@ubik-ingenierie.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |p.mouawad@ubik-ingenierie.c
                   |                            |om

--- Comment #1 from Philippe Mouawad <p....@ubik-ingenierie.com> ---
I think it's just a removal forgotten when fixing Bug 59034.
The call to it was removed within that bug.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Bug 59107] HTTPSamplerBase.CleanerThread bug?

Posted by bu...@apache.org.
https://bz.apache.org/bugzilla/show_bug.cgi?id=59107

--- Comment #2 from Sebb <se...@apache.org> ---
I see.

In which case there's some more work to do on that bug.

-- 
You are receiving this mail because:
You are the assignee for the bug.