>>>>> On Wed, 28 Sep 2005 16:38:45 +0100, Dave Shield <[EMAIL PROTECTED]> said:

Dave> This implementation isn't fully complete - it only includes
Dave> Existence and Boolean tests at the moment, and doesn't yet
Dave> cover Delta samples (as well as missing some necessary
Dave> config directives).  But it should be sufficiently operational
Dave> to be able to compare it sensibly against the existing code.

Dave> Comments?

I certainly don't mind it replacing, eventually, the existing code as
long as:

1) it's backwards compatible if it's reasonable to do so (and if not,
   it should be a new option probably and not a replacement).  Mostly
   this should effect just the monitor token.
2) It truly does implement everything that is there now, which I don't
   think it does.  You said it does, but I'm pretty sure that the
   delta support was there before.  At least I remember implementing
   it.  The threshold table is also currently implemented by the old
   code, and I think you said it's not by the new code?  That also
   concerns me from a replacement point of view till it's complete.

Also, one final comment: the disman tables are not true augments and
thus you probably shouldn't declare them that way.  True augments have
an exact 1 to 1 mapping, and those tables have identical indexes but
do not use the AUGMENT keyword anywhere since entries in them are
conditional.  It's N main rows to M parallel table rows where N >= M.

 

-- 
Wes Hardaker
Sparta, Inc.


-------------------------------------------------------
This SF.Net email is sponsored by:
Power Architecture Resource Center: Free content, downloads, discussions,
and more. http://solutions.newsforge.com/ibmarch.tmpl
_______________________________________________
Net-snmp-coders mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/net-snmp-coders

Reply via email to