[BZ 1069648] Persist JAVA_OPTS from connection settings for Standalone se... #42
Conversation
What will happen if standalone.conf already has JAVA_OPTS set a few lines down? The rhq-props are set at the top and the other occurrence will overwrite that setting? |
Based on Monday's review I moved the section to the bottom of the file and will just override everything else set before. So now, the users could update (although not recommended) the conf file and not directly affect the RHQ updates. |
@@ -28,10 +41,17 @@ | |||
*/ | |||
public class StandaloneASDiscovery extends BaseProcessDiscovery { | |||
|
|||
private static final Log log = LogFactory.getLog(BaseProcessDiscovery.class); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
getLog(StandaloneASDiscovery.class) ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed
Other than the comments I just added inline, the change looks good to me. |
…d is additive to existing settings. This is a safer approach.
Looks good, merging... |
[BZ 1069648] Persist JAVA_OPTS from connection settings for Standalone se...
...rvers.