[PATCH] displaying thread id in mod_status ExtendedStatus table

[Jeff Trawick <tr...@gmail.com>]

On 6/11/05, Jeff Trawick <tr...@gmail.com> wrote:
> On 6/11/05, William A. Rowe, Jr. <wr...@rowe-clan.net> wrote:
> > At 12:34 PM 6/11/2005, Jeff Trawick wrote:
> > >> I like the idea of toggling columns, I'd make it more like
> > >> columns=pid,tid,ss,m,req to allow for abbreviated displays.
> > >
> > >columns= with column names like that works for me
> > >
> > >columns=all displays everything (may change layout in the future)
> > >default list for columns is whatever we display now
> >
> > I like columns=* personally :)  Less ambiguous if we decree all columns
> > can be toggled (especially if we allow modules to extend them).
> >
> > Perhaps +/- syntax?  Painful in code, but easier for the user
> > (?columns=+tid).  +/- wouldn't reorder the columns from their
> > default, while explicit lists would?

No reordering of columns is possible with this patch.
The Srv column is always implicitly enabled.
'*' = all columns (need to escape it on shell command-line)
Bogus values for columns parameter are not reported.
No changes implemented for the no-table flavor of output (yet).

Something that is a bit painful is that APR doesn't have a way to
format a thread id in hex, and that is a useful format for correlation
with some third party logs.  That should be resolved eventually so
that the thread id in hex (and perhaps the process id in hex too) can
be requested.

Step in the right direction?
Not usable since somebody is going to implement configurable field order soon?
Broken?
etc.

Thanks for any comments!