Ack with the following comments: - Publication month is April 2016 - Add (atleast) a one liner that describes what this new attribute is about.
Thanks, Mathi. ----- [email protected] wrote: > Hi Vu > > > > Ack. Please notify me when acked by all reviewers and I will push > > > > Thanks > > Lennart > > > > > > > From: Vu Minh Nguyen [mailto:[email protected]] > Sent: den 11 april 2016 08:44 > To: Lennart Lund; 'Mathivanan Naickan Palanivelu' > Cc: [email protected]; Anders Widell; Jorge Pacheco > Garcia > Subject: RE: Review Request for log: update PR document for node > originator [#1688] > > > > Hi Lennart, > > > > I am sorry. I considered adding the global OpenSAF configuration as an > extension, > > so putting to OpenSAF_extensions.doc PR would be appropriate. > > > > Attached file contains the updated points. Please have a look. > > (Adding a sub-chapter 3.7.2) > > > > > Regards, Vu. > > > > > > > From: Lennart Lund [ mailto:[email protected] ] > Sent: Friday, April 08, 2016 8:38 PM > To: Vu Minh Nguyen; 'Mathivanan Naickan Palanivelu' > Cc: [email protected] ; Lennart Lund; Anders Widell; > Jorge Pacheco Garcia > Subject: RE: Review Request for log: update PR document for node > originator [#1688] > > > > Hi Vu > > > > Wrong document! > > I don’t think the information about the new global configuration > possibilities in OpenSAF should be described in the extension > document. It is better to update the overview document. It can be > placed as a sub section in the “Tunable OpenSAF configuration > parameters” section. > > > > There is also some suggested changes and comments in the attached > document > > > > Thanks > > Lennart > > > > > > > From: Vu Minh Nguyen [ mailto:[email protected] ] > Sent: den 7 april 2016 09:03 > To: 'Mathivanan Naickan Palanivelu'; Lennart Lund > Cc: [email protected] > Subject: RE: Review Request for log: update PR document for node > originator [#1688] > > > > Hi all, > > > > Thanks for your comments. Please see the update for both docs. > > > > Regarding OpenSAF_extensions.doc: > > 1) Add new chapter “5 Others” > > 2) Update the cover & Content tables > > > > If you have any more comments, please let me know. > > Otherwise, Lennart could help to push the docs. > > > > > Regards, Vu. > > > > > > > From: Mathivanan Naickan Palanivelu [ mailto:[email protected] > ] > Sent: Thursday, April 07, 2016 1:06 PM > To: Lennart Lund; Vu Minh Nguyen > Cc: [email protected] > Subject: RE: Review Request for log: update PR document for node > originator [#1688] > > > > Hi Vu, > > > > I have my suggestions in the attached doc. > > And I agree with Lennart , but in the OpenSAF_extensions.doc and > update the doc as suggested by him. > > > > Thanks, > > Mathi. > > > > > > > From: Lennart Lund [ mailto:[email protected] ] > Sent: Monday, April 04, 2016 5:07 PM > To: Vu Minh Nguyen; Mathivanan Naickan Palanivelu > Cc: [email protected] ; Lennart Lund > Subject: RE: Review Request for log: update PR document for node > originator [#1688] > > > > Hi Vu, > > > > Ack for this document (log). > > > > Comment: > > A description of the new OpenSAF configuration object is needed. > However it is correct to not have this description in the log > document. > > I suggest that the “OpenSAF_Overview_PR” document is updated with this > information. > > Information to add: > > · Description of the class e.g. create a table describing the name of > the class, describe that a default object of this class is created and > the name of this object, all attributes and their default value (so > far we have only one attribute but others may be added in the future) > > > > Thanks > > Lennart > > > > > > > From: Vu Minh Nguyen [ mailto:[email protected] ] > Sent: den 29 mars 2016 10:47 > To: [email protected] ; Lennart Lund > Cc: [email protected] > Subject: Review Request for log: update PR document for node > originator [#1688] > > > > Summary: log: update PR document for node originator [#1688] > > Trac Ticket(s): #1688 > > Peer Reviewer(s): Lennart, Mathi > > > > -------------------------------- > > Impacted area Impact y/n > > -------------------------------- > > Docs y > > Build system n > > RPM/packaging n > > Configuration files n > > Startup scripts n > > SAF services n > > OpenSAF services n > > Core libraries n > > Samples n > > Tests n > > Other n > > > > > > Comments (indicate scope for each "y" above): > > --------------------------------------------- > > Update Programmers Reference with the changes introduced in [#1480] > > > > Marked document is attached. > > > > Regards, Vu. ------------------------------------------------------------------------------ Find and fix application performance issues faster with Applications Manager Applications Manager provides deep performance insights into multiple tiers of your business applications. It resolves application problems quickly and reduces your MTTR. Get your free trial! http://pubads.g.doubleclick.net/ gampad/clk?id=1444514301&iu=/ca-pub-7940484522588532 _______________________________________________ Opensaf-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/opensaf-devel
