Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 337872 - [TMF] Progress monitor feedback for indexing and constructor trigger
Summary: [TMF] Progress monitor feedback for indexing and constructor trigger
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: LinuxTools (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Francois Chouinard CLA
QA Contact: Francois Chouinard CLA
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-22 13:07 EST by Patrick Tasse CLA
Modified: 2022-01-13 14:53 EST (History)
0 users

See Also:


Attachments
Proposed patch in org.eclipse.linuxtools.tmf (5.84 KB, patch)
2011-02-22 13:14 EST, Patrick Tasse CLA
fchouinard: iplog+
Details | Diff
Updated patch (6.07 KB, patch)
2011-03-10 13:32 EST, Francois Chouinard CLA
fchouinard: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Patrick Tasse CLA 2011-02-22 13:07:20 EST
Build Identifier: 

A job is created to provide feedback to the user about the progress of the indexing of a trace or experiment.

Also, automatic indexing of a TmfTrace at creation can be controlled with a constructor boolean parameter.

Reproducible: Always
Comment 1 Patrick Tasse CLA 2011-02-22 13:14:11 EST
Created attachment 189522 [details]
Proposed patch in org.eclipse.linuxtools.tmf

Legal Message: I, Patrick Tasse, 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.
Comment 2 Francois Chouinard CLA 2011-03-10 13:32:48 EST
Created attachment 190896 [details]
Updated patch

The original patch works fine but for the unit tests that fail because of an ambiguity on the TmfTrace constructors indexing flag.

This patch augments the original one in order to set the flag correctly in the unit tests.
Comment 3 Francois Chouinard CLA 2011-03-29 18:28:05 EDT
Patch committed. Thanks Patrick.
Comment 4 Francois Chouinard CLA 2011-07-22 15:01:28 EDT
Delivered with 0.8