|
From: Joe G. <joe...@us...> - 2003-12-02 18:50:09
|
Quoting Doug Fort <dou...@do...>:
> I have attached a proposed modification to the Brick configuration file. This
> declares the recorders to subscribe to InstrumentationRecorder. It may be a
> bit
> much to load the recorders dynamically, however it's a technique I'm
> interested
> in and it will allow us (or others) to add new recorders without changing the
> code.
>
I like the idea of "dynamically" adding recorders...
Just a couple (four really) of comments/questions:
1.- We probably do NOT need a console recorder right away. The instrumentation
data stream is huge...and not very useful on the screen. (Am I interperting the
function of a console recorder correctly?) And, useful info can be obtained
using tail/grep on the file recorder output.
2.- The end of the appsettings should be:
<!-- Non-command line configuration option(s), DO NOT DELETE any
of the following lines -->
<add key="connectthreadcount" value="6" />
<add key="vc_inventory_level" value="4" />
<add key="max_new_vc_queue_depth" value="100" />
<add key="max_run_queue_depth" value="0" />
<add key="max_process_queue_depth" value="0" />
<add key="max_wait_queue_depth" value="0" />
(I may not have checked in an up-to-date .config file for the brick.)
3.- Does this mean we can drop the config line:
<add key="instumentationfile" value="addanc.brick.instrumentation" />
and have the output handled by the fileRecorder?
4.- If yes, can we have the default name for the fileRecorder match the old
default, "addanc.brick.instrumentation"?
--
Joe Griffin
Addanc
Downright Software Collective
------------------------------------------------------------------------
This email and any files transmitted with it are confidential and
intended solely for the use of the individual or entity to whom they
are addressed. If you have received this email in error please notify
sec...@do....
------------------------------------------------------------------------
This mail sent through Downright Software's implementation of HORDE/IMP.
For info, see: http://horde.org/imp/
|