[Sblim-issues] [ sblim-Bugs-3101157 ] Failed to enable provider timing analysis with SfcbLocal

SourceForge Headquarters 225 Broadway Suite 1600 San Diego, CA 92101 +1 (858) 454-5900

Bugs item #3101157, was opened at 2010-11-01 17:45
Message generated for change (Comment added) made by nsharoff
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=712784&aid=3101157&group_id=128809

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: sfcb
Group: Performance
Status: Open
Resolution: None
Priority: 4
Private: No
Submitted By: Chris Poblete (cpoblete)
Assigned to: Chris Buccella (buccella)
Summary: Failed to enable provider timing analysis with SfcbLocal

Initial Comment:
Using the local interface connection in Sfcc, we could not enable the provider timing that is used with debug logging. In code inspection, we find that the check for sessionId is always failing since it is always 0 for local interface. 

The attached patch file (for sfcb 1.3.9) provides the proposed resolution:

The proposal is to exclude the sessionId in the check whether to continue the timing process when requested.  An added change is to remove the duplicate verification of the trace mask.

This is a performance fix to allow timing analysis with the SfcbLocal interface. The proposed solution has been tested to work in our environment. Please integrate a solution so that Sfcb can continue to work in our environment with future releases. Thanks.

----------------------------------------------------------------------

>Comment By: Narasimha Sharoff (nsharoff)
Date: 2011-07-21 09:56

Message:
Verification of trace mask is not be required in TIMING_STOP if the uset
that is set during TIMING_START is reset at the completion of TIMING_STOP.
This check ensures TIMING_STOP must match a preceding TIMING_START.

If uset is not reset, then TIMING_STOP may be called without a preceding
TIMING_START and checking the trace mask becomes necessary.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=712784&aid=3101157&group_id=128809