You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by "Sorin Suciu (JIRA)" <ji...@apache.org> on 2010/10/22 08:47:15 UTC

[jira] Created: (QPID-2911) Remove the class printing from the log4j layout as the (%F:%L) provides sufficient debug information

Remove the class printing from the log4j layout as the  (%F:%L) provides sufficient debug information
-----------------------------------------------------------------------------------------------------

                 Key: QPID-2911
                 URL: https://issues.apache.org/jira/browse/QPID-2911
             Project: Qpid
          Issue Type: Improvement
          Components: Java Broker
    Affects Versions: 0.7
            Reporter: Sorin Suciu
            Assignee: Sorin Suciu
            Priority: Minor
             Fix For: 0.7


Currently we are printing the class name along with filename:line_number in the logs. The class name is redundant in this scenario as the file name would be sufficient for debug purposes. This reduces the log size which might be a sensitive thing for heavy load cases. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org


[jira] Resolved: (QPID-2911) Remove the class printing from the log4j layout as the (%F:%L) provides sufficient debug information

Posted by "Andrew Kennedy (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/QPID-2911?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andrew Kennedy resolved QPID-2911.
----------------------------------

    Resolution: Fixed

Reviewed and committed patch from Sorin S <ss...@gmail.com>

> Remove the class printing from the log4j layout as the  (%F:%L) provides sufficient debug information
> -----------------------------------------------------------------------------------------------------
>
>                 Key: QPID-2911
>                 URL: https://issues.apache.org/jira/browse/QPID-2911
>             Project: Qpid
>          Issue Type: Improvement
>          Components: Java Broker
>    Affects Versions: 0.7
>            Reporter: Sorin Suciu
>            Assignee: Andrew Kennedy
>            Priority: Minor
>             Fix For: 0.7
>
>         Attachments: qpid-2911.patch
>
>
> Currently we are printing the class name along with filename:line_number in the logs. The class name is redundant in this scenario as the file name would be sufficient for debug purposes. This reduces the log size which might be a sensitive thing for heavy load cases. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org


[jira] Updated: (QPID-2911) Remove the class printing from the log4j layout as the (%F:%L) provides sufficient debug information

Posted by "Sorin Suciu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/QPID-2911?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sorin Suciu updated QPID-2911:
------------------------------

    Attachment: qpid-2911.patch

> Remove the class printing from the log4j layout as the  (%F:%L) provides sufficient debug information
> -----------------------------------------------------------------------------------------------------
>
>                 Key: QPID-2911
>                 URL: https://issues.apache.org/jira/browse/QPID-2911
>             Project: Qpid
>          Issue Type: Improvement
>          Components: Java Broker
>    Affects Versions: 0.7
>            Reporter: Sorin Suciu
>            Assignee: Sorin Suciu
>            Priority: Minor
>             Fix For: 0.7
>
>         Attachments: qpid-2911.patch
>
>
> Currently we are printing the class name along with filename:line_number in the logs. The class name is redundant in this scenario as the file name would be sufficient for debug purposes. This reduces the log size which might be a sensitive thing for heavy load cases. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org


[jira] Updated: (QPID-2911) Remove the class printing from the log4j layout as the (%F:%L) provides sufficient debug information

Posted by "Andrew Kennedy (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/QPID-2911?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Andrew Kennedy updated QPID-2911:
---------------------------------

    Status: Ready To Review  (was: In Progress)

> Remove the class printing from the log4j layout as the  (%F:%L) provides sufficient debug information
> -----------------------------------------------------------------------------------------------------
>
>                 Key: QPID-2911
>                 URL: https://issues.apache.org/jira/browse/QPID-2911
>             Project: Qpid
>          Issue Type: Improvement
>          Components: Java Broker
>    Affects Versions: 0.7
>            Reporter: Sorin Suciu
>            Assignee: Andrew Kennedy
>            Priority: Minor
>             Fix For: 0.7
>
>         Attachments: qpid-2911.patch
>
>
> Currently we are printing the class name along with filename:line_number in the logs. The class name is redundant in this scenario as the file name would be sufficient for debug purposes. This reduces the log size which might be a sensitive thing for heavy load cases. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org


[jira] Assigned: (QPID-2911) Remove the class printing from the log4j layout as the (%F:%L) provides sufficient debug information

Posted by "Sorin Suciu (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/QPID-2911?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Sorin Suciu reassigned QPID-2911:
---------------------------------

    Assignee: Andrew Kennedy  (was: Sorin Suciu)

Andrew, can you review please?

> Remove the class printing from the log4j layout as the  (%F:%L) provides sufficient debug information
> -----------------------------------------------------------------------------------------------------
>
>                 Key: QPID-2911
>                 URL: https://issues.apache.org/jira/browse/QPID-2911
>             Project: Qpid
>          Issue Type: Improvement
>          Components: Java Broker
>    Affects Versions: 0.7
>            Reporter: Sorin Suciu
>            Assignee: Andrew Kennedy
>            Priority: Minor
>             Fix For: 0.7
>
>         Attachments: qpid-2911.patch
>
>
> Currently we are printing the class name along with filename:line_number in the logs. The class name is redundant in this scenario as the file name would be sufficient for debug purposes. This reduces the log size which might be a sensitive thing for heavy load cases. 

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
Apache Qpid - AMQP Messaging Implementation
Project:      http://qpid.apache.org
Use/Interact: mailto:dev-subscribe@qpid.apache.org