Hi Ginnie,

This looks really good. A few comments/questions...

Section 4:
Distro Constructor is able to set up remote logging service-why is this an assumption? I can see that the application should be able to set this up but why DC specifically?

As for my thoughts on which option to use for adding handlers:

-I would prefer to have an addHandler method that abstracts out formatting from the consumer.

However, I just want to be sure I am reading your proposal correct... in option 1,the formatting is set by the InstallLogger, so the consumer doesn't have anything to set with this regard, correct?

In option 3, does the consumer have a choice to either add their own handlers, or use the ones built in to the InstallLogger, using the dictionary approach?

Section 9.2: Which option have you chosen? I assume Option 3 based on the UML diagrams, but you should likely state the choice in this section.

thanks,
sarah
****











On 05/13/10 03:17 PM, Ginnie Wray wrote:
Hi -

As a result of some additional research, I've updated sections of the
design document for the install logger. Sections 1 through 11 of the
document below have been updated. I have not revised any of the material
that pertain to remote logging.

http://hub.opensolaris.org/bin/view/Project+caiman/Logging

The current version is v1.3. I've also uploaded a UML diagram that is
included in the document. It's a little small in the document, so you
can take a look with larger font using the online version. It is the UML
diagram v1.3.

I would like to have feedback by COB on May 24th, but I know we're all
busy, so anything beyond that deadline is appreciated as well.

Thanks,
ginnie

_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

_______________________________________________
caiman-discuss mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/caiman-discuss

Reply via email to