Hi Canh, I should have Acked the document in my previous answer. Just fix any minor things and push.
Thanks Lennart From: Canh Van Truong <canh.v.tru...@dektech.com.au> Sent: den 11 januari 2019 12:01 To: Lennart Lund <lennart.l...@ericsson.com>; Minh Hon Chau <minh.c...@dektech.com.au> Cc: opensaf-devel@lists.sourceforge.net Subject: RE: Review request ntf: update document for limit of logger buffer [#2994] Hi Lennart Yes, I add new column for "Default value". Regards Canh From: Lennart Lund <lennart.l...@ericsson.com> Sent: Friday, January 11, 2019 5:33 PM To: Canh Van Truong <canh.v.tru...@dektech.com.au>; Minh Hon Chau <minh.c...@dektech.com.au> Cc: opensaf-devel@lists.sourceforge.net Subject: RE: Review request ntf: update document for limit of logger buffer [#2994] Hi Canh, I have one minor comment in the attached document. Also, it may be better if the table has three columns "Environment Variable, Default value and Comment" Thanks Lennart From: Canh Van Truong <canh.v.tru...@dektech.com.au<mailto:canh.v.tru...@dektech.com.au>> Sent: den 11 januari 2019 10:18 To: Lennart Lund <lennart.l...@ericsson.com<mailto:lennart.l...@ericsson.com>>; Minh Hon Chau <minh.c...@dektech.com.au<mailto:minh.c...@dektech.com.au>> Cc: opensaf-devel@lists.sourceforge.net<mailto:opensaf-devel@lists.sourceforge.net> Subject: RE: Review request ntf: update document for limit of logger buffer [#2994] Thanks Lennart, I have updated with your comments. Please give the comments if there is something need to be updated. Regards Canh From: Lennart Lund <lennart.l...@ericsson.com<mailto:lennart.l...@ericsson.com>> Sent: Thursday, January 10, 2019 9:07 PM To: Canh Van Truong <canh.v.tru...@dektech.com.au<mailto:canh.v.tru...@dektech.com.au>>; Minh Hon Chau <minh.c...@dektech.com.au<mailto:minh.c...@dektech.com.au>> Cc: opensaf-devel@lists.sourceforge.net<mailto:opensaf-devel@lists.sourceforge.net> Subject: RE: Review request ntf: update document for limit of logger buffer [#2994] Hi Canh, The following text describing the NTFSV_LOGGER_BUFFER_CAPACITY environment variable should be improved/simplified (hard to understand) Original text from document: "The limit of logger buffer size in NTFD. The logger buffer is used to store the notification if writing notification to log file fail. The limit should be set with relevant value to avoid congestion in NTFD. Because if this value is set too big while writing notification is fail for long time, NTFD has to write a big number of notifications whenever handling sending notification request and that will delay to handle other requests come to NTFD. The value of variable is from 10 to 5000." Suggestion: Note: my native language is not English so I suggest that this text is checked by someone who knows English better than I do. However, the following information is what I think is needed. "Notification log buffer size. Valid values are 10 to 5000 stored notifications. Default is 10. Some notifications are logged using the OpenSAF log service. NTF has a buffer to store notifications to be logged later in case the log service returns TRY AGAIN (may happen if the log service is temporary unavailable) when NTF writes the log record. When the log service is available again (returns OK) all notifications in the buffer will be written before the NTF service can service any new notification requests. If the buffer is big this may take some time and may cause the NTF client to timeout. If the buffer is full and the log service answers TRY AGAIN NTF will return TRY AGAIN when Notification send is called." Thanks Lennart From: Canh Van Truong <canh.v.tru...@dektech.com.au<mailto:canh.v.tru...@dektech.com.au>> Sent: den 10 januari 2019 12:52 To: Minh Hon Chau <minh.c...@dektech.com.au<mailto:minh.c...@dektech.com.au>>; Lennart Lund <lennart.l...@ericsson.com<mailto:lennart.l...@ericsson.com>> Cc: opensaf-devel@lists.sourceforge.net<mailto:opensaf-devel@lists.sourceforge.net> Subject: Review request ntf: update document for limit of logger buffer [#2994] Update the document because of the ticket #2961 ntf: Limit the logger buffer [#2961] Document with recorded changes attached. Activate "Show Changes" to see them [Edit/Track Changes/Show Changes] Thanks Canh _______________________________________________ Opensaf-devel mailing list Opensaf-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/opensaf-devel
