tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
Subject DO NOT REPLY [Bug 49395] manager.findLeaks : display the date when the leaking context was started
Date Wed, 09 Jun 2010 21:02:18 GMT changed:

           What    |Removed                     |Added
  Attachment #25535|0                           |1
        is obsolete|                            |
  Attachment #25536|0                           |1
        is obsolete|                            |

--- Comment #6 from 2010-06-09 17:02:11 EDT ---
Created an attachment (id=25570)
 --> (
patch for tomcat 7, v2

I'm proposing a new patch that addresses the points you noticed. There remains
the question of JMX

> 1) Host.findReloadedContextMemoryLeaks is callable through JMX.
> I need to test it, but I suppose that ContextLeakInfo cannot be rendered by
> JConsole.
In the new patch I kept the String[] for the return value, separating context
path, start date and stop dates by a tab.

> 2) It looks like an API change for me.
If we keep String[], the only thing that change is the content of the Strings.
Is it that important ? This type of information is probably rather for humans.

> 3) I would like to see not only when the context was started, but also when it
> was stopped.

> 4) needs the Apache license header.

> 5) i18n.
I just modified the text in the HTMLmanager translation. Dates are is ISO 8601
format, no need for translation.

> 6) As a further enhancement I think that the result returned by
> StandardHost.findReloadedContextMemoryLeaks() might be sorted by contextPath
> and then by startDate.

> 7) The documentation has to be updated: manager-howto.xml,
> html-manager-howto.xml

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

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message