Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 59557 - Optimized Heap Dump directory specification in piAgent
Summary: Optimized Heap Dump directory specification in piAgent
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Hyades (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Piyush Agarwal CLA
QA Contact:
URL:
Whiteboard: closed460
Keywords:
Depends on:
Blocks:
 
Reported: 2004-04-21 19:10 EDT by Allan Pratt CLA
Modified: 2012-02-15 13:48 EST (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Allan Pratt CLA 2004-04-21 19:10:11 EDT
This bug reflects decisions by me, Anandi, and Richard. It is targeted at 
Piyush.

We decided:

1. There should be just one way to set the directory name that opt heap dumps 
will be written into.

2. That way should be in a <variable> element in ServiceConfig.xml, not an 
<option> in the agent's pluginconfig.xml. This way it's visible to all agents, 
not just the piAgent.

3. The variable name should not be TEMP, it should be LOCAL_AGENT_TEMP_DIR.

4. The default value for LOCAL_AGENT_TEMP_DIR, written into serviceconfig.xml 
by SetConfig, should be %TMP%

Here are the OptHeapDump-related action items that come out of those agreements:

* Please change the Leakbot code in the agent so it uses 
ra_getEnvironmentVariable("LOCAL_AGENT_TEMP_DIR") (and not an option query for 
OPTHEAP_TEMP_DIR) to find out where to write optheap dump files.

* If LOCAL_AGENT_TEMP_DIR is not set, please use ra_getEnvironmentVariable
("TMP") as a backup. It should always be set, on all platforms. But if it's not 
set, give it whatever fallback behavior it has now, write to the current 
directory or something.
Comment 1 Piyush Agarwal CLA 2004-04-29 04:45:17 EDT
That way should be in a <variable> element in ServiceConfig.xml, not an 
<option> in the agent's pluginconfig.xml. This way it's visible to all agents, 
not just the piAgent.
The variable name is LOCAL_AGENT_TEMP_DIR.
The default value for LOCAL_AGENT_TEMP_DIR, written into serviceconfig.xml by 
SetConfig is %TMP%
Comment 2 Harm Sluiman CLA 2004-11-03 11:21:24 EST
house keeping
Comment 3 Paul Slauenwhite CLA 2009-06-30 12:27:59 EDT
As of TPTP 4.6.0, TPTP is in maintenance mode and focusing on improving quality by resolving relevant enhancements/defects and increasing test coverage through test creation, automation, Build Verification Tests (BVTs), and expanded run-time execution. As part of the TPTP Bugzilla housecleaning process (see http://wiki.eclipse.org/Bugzilla_Housecleaning_Processes), this enhancement/defect is verified/closed by the Project Lead since this enhancement/defect has been resolved and unverified for more than 1 year and considered to be fixed. If this enhancement/defect is still unresolved and reproducible in the latest TPTP release (http://www.eclipse.org/tptp/home/downloads/), please re-open.