[jboss-jira] [JBoss JIRA] Commented: (JBAS-6411) Show Java runtime information in boot.log

[Galder Zamarreno (JIRA)]

    [ https://jira.jboss.org/jira/browse/JBAS-6411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12449394#action_12449394 ] 

Galder Zamarreno commented on JBAS-6411:
----------------------------------------

Re: Before doing changes,

I did the changes last night.

Re: isn't boot.log logging *all* system properties already? 

It is indeed.

However, we could use your same argument to remove the following two lines which also come from system properties.:

00:40:36,112 INFO [ServerInfo] Java version: 1.6.0_0,Sun Microsystems Inc. 
00:40:36,112 INFO [ServerInfo] Java VM: OpenJDK Server VM 10.0-b19,Sun Microsystems Inc. 

The idea here is to visually have the same feedback as you get when doing java -version.
It's a matter of all or nothing with regards to printing Java/VM version information. I'm happy 
with either as long as we're consistent.

To show information partially, like it was before I did these changes can be confusing.

> Show Java runtime information in boot.log
> -----------------------------------------
>
>                 Key: JBAS-6411
>                 URL: https://jira.jboss.org/jira/browse/JBAS-6411
>             Project: JBoss Application Server
>          Issue Type: Task
>      Security Level: Public(Everyone can see) 
>          Components: System service
>            Reporter: Galder Zamarreno
>            Assignee: Galder Zamarreno
>             Fix For: JBossAS-5.0.1.GA, JBossAS-5.1.0.Beta1, JBossAS-6.0.0.Alpha1
>
>
> With different flavours of JDKs becoming more common, 
> it's important to show Java runtime properties in log/boot.log:
> System.getProperty("java.runtime.version");
> System.getProperty("java.runtime.name");

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira