Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 346564

Summary: Add ability to disable SDEF cache during development
Product: z_Archived Reporter: Greg Amerson <gregory.amerson>
Component: SapphireAssignee: Konstantin Komissarchik <konstantin>
Status: CLOSED FIXED QA Contact:
Severity: enhancement    
Priority: P3 CC: kamesh.sampath, konstantin
Version: unspecifiedKeywords: plan
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
patch attempt 1 none

Description Greg Amerson CLA 2011-05-19 20:32:55 EDT
While working on SDEF files in the PDE it would be much faster if the SDEF cache could be disabled so that it would be re-read each time the UI was built.
Comment 1 Greg Amerson CLA 2011-05-19 20:34:31 EDT
Suggested method would be to introduce support for a framework wide debug flag:
-Dsapphire.debug=true

If its true, then the SDEF cache could check this flag and if debug is on, it would not cache the definition, thus allowing for rapid development.
Comment 2 Konstantin Komissarchik CLA 2011-05-19 21:03:43 EDT
Note that I would expect caching to be disabled only for sdef models (as opposed to disabling the overall caching system that is used for any kind of model).
Comment 3 Konstantin Komissarchik CLA 2011-09-05 14:51:27 EDT
Tagging as a possible work item for Kamesh.
Comment 4 Greg Amerson CLA 2011-10-15 10:22:46 EDT
I've getting back into SDEF development again and before I started I went ahead and made a patch for this.  I'll attach my current patch.
Comment 5 Greg Amerson CLA 2011-10-15 10:25:31 EDT
Created attachment 205256 [details]
patch attempt 1

Added a simple -D debug flag to look for to avoid cache check for SDEFs.
Comment 6 Konstantin Komissarchik CLA 2011-10-17 15:49:26 EDT
I generalized the system property by calling it "sapphire.dev.mode" and creating a public method to check its state. We will might want to attach other behaviors to dev mode besides disabling sdef cache.

Please verify.
Comment 7 Greg Amerson CLA 2011-10-21 03:26:35 EDT
Verified with latest CVS.
Comment 8 Konstantin Komissarchik CLA 2011-10-21 10:52:01 EDT
Closing. Thanks.