Community
Participate
Working Groups
This Bug is a reminder to implement the following couple of small tasks 1) SyntheticEventProvider.java, replace the current three sub-request of trace level time ranges, for one sub-request of coalesced time ranges. 2) in the StateExperimentManger.java, optimise the dispatching of events to its corresponding trace by a comparing the trace references vs String path+name, (note, comparing references may not be trivial if trace clones are implemented as part of the solution)
Created attachment 178090 [details] proposed patch This patch tries to fix both parts of this bug report: 1) It replaces the multiple (one per trace) requests in LttngSyntheticEventProvider.java by one request of coalesced time ranges. 2) Optimises the dispatching of events to its corresponding trace in StateExperimentManager.java by using the trace reference directly in the map as key. Legal Message: I, Marc Dumais, declare that I developed attached code from scratch, without referencing any 3rd party materials except material licensed under the EPL. {I am authorized by my employer to make this contribution under the EPL.}
The functionality is good and doing as expected, I have the following comments for the patch. 1) Remove unused import in LttngBaseEventRequest e.g. "ctrl-shift o" import org.eclipse.linuxtools.lttng.state.trace.IStateTraceManager; 2) LttngBaseEventRequest, the "traceToTraceState" Map is used in LttngBaseEventRequest but via an extended inner instance in LttngSyntheticEventProvider, so it should be moved to and built in the LttngSyntheticEventProvider loop replacing traces[] and traceModels[] arrays, this instance shall then be used by any subsequent requests and only updated once per experiment selection. 3) LtttngSyntheticEventProvider a) Map<IStateTraceManager, LttngBaseEventRequest> fEventProviderRequests The key is still in use but not the values i.e. single coalesced request, this shall be replaced by a List<IStateTraceManager> b) LttngBaseEventRequest subRequest The syntheticAckIndicator does not seem to be needed any longer private LttngSyntheticEvent syntheticAckIndicator = null;
Created attachment 179334 [details] Updated patch, following review comments Hi Alvaro, Thanks for the comments. Find attached an updated patch, taking your comments into consideration. Thanks, Marc
Created attachment 180760 [details] Updated patch, following rebase to latest of TRUNK as of 2010-10-13
Created attachment 180879 [details] Updated patch, following rebase to latest of TRUNK as of 2010-10-14 Let's try this again :)
Comment on attachment 180879 [details] Updated patch, following rebase to latest of TRUNK as of 2010-10-14 Patch committed in Helios and Indigo
Done.
Delivered with 0.7
