You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-issues@hadoop.apache.org by "Patrick Hunt (Created) (JIRA)" <ji...@apache.org> on 2012/03/27 00:22:26 UTC

[jira] [Created] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

address log4j.properties inconsistencies btw main and template dirs
-------------------------------------------------------------------

                 Key: HADOOP-8216
                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
             Project: Hadoop Common
          Issue Type: Improvement
          Components: conf
            Reporter: Patrick Hunt
            Assignee: Patrick Hunt
            Priority: Minor


DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)

lesser issues but I noticed:

I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 

afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.

Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

Posted by "Eli Collins (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-8216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13238917#comment-13238917 ] 

Eli Collins commented on HADOOP-8216:
-------------------------------------

I think we should just use the values from templates/conf/log4j.properties in main/conf/log4j.properties, the former look the most up2date, and they're mostly the same (ie just different order in some places, and removing things that are commented out). 
                
> address log4j.properties inconsistencies btw main and template dirs
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8216
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>
> DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)
> lesser issues but I noticed:
> I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 
> afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.
> Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

Posted by "Hudson (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-8216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13239623#comment-13239623 ] 

Hudson commented on HADOOP-8216:
--------------------------------

Integrated in Hadoop-Common-trunk-Commit #1936 (See [https://builds.apache.org/job/Hadoop-Common-trunk-Commit/1936/])
    HADOOP-8216. Address log4j.properties inconsistencies btw main and template dirs. Contributed by Patrick Hunt (Revision 1305891)

     Result = SUCCESS
eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1305891
Files : 
* /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
* /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/conf/log4j.properties

                
> address log4j.properties inconsistencies btw main and template dirs
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8216
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>    Affects Versions: 0.23.0
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>         Attachments: HADOOP-8216.patch
>
>
> DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)
> lesser issues but I noticed:
> I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 
> afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.
> Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

Posted by "Hudson (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-8216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13239615#comment-13239615 ] 

Hudson commented on HADOOP-8216:
--------------------------------

Integrated in Hadoop-Hdfs-trunk-Commit #2011 (See [https://builds.apache.org/job/Hadoop-Hdfs-trunk-Commit/2011/])
    HADOOP-8216. Address log4j.properties inconsistencies btw main and template dirs. Contributed by Patrick Hunt (Revision 1305891)

     Result = SUCCESS
eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1305891
Files : 
* /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
* /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/conf/log4j.properties

                
> address log4j.properties inconsistencies btw main and template dirs
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8216
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>    Affects Versions: 0.23.0
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>         Attachments: HADOOP-8216.patch
>
>
> DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)
> lesser issues but I noticed:
> I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 
> afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.
> Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

Posted by "Hudson (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-8216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13240413#comment-13240413 ] 

Hudson commented on HADOOP-8216:
--------------------------------

Integrated in Hadoop-Mapreduce-trunk #1033 (See [https://builds.apache.org/job/Hadoop-Mapreduce-trunk/1033/])
    HADOOP-8216. Address log4j.properties inconsistencies btw main and template dirs. Contributed by Patrick Hunt (Revision 1305891)

     Result = FAILURE
eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1305891
Files : 
* /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
* /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/conf/log4j.properties

                
> address log4j.properties inconsistencies btw main and template dirs
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8216
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>    Affects Versions: 0.23.0
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>         Attachments: HADOOP-8216.patch
>
>
> DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)
> lesser issues but I noticed:
> I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 
> afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.
> Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

Posted by "Hudson (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-8216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13239668#comment-13239668 ] 

Hudson commented on HADOOP-8216:
--------------------------------

Integrated in Hadoop-Mapreduce-0.23-Commit #744 (See [https://builds.apache.org/job/Hadoop-Mapreduce-0.23-Commit/744/])
    HADOOP-8216. svn merge -c 1305891 from trunk (Revision 1305892)

     Result = ABORTED
eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1305892
Files : 
* /hadoop/common/branches/branch-0.23/hadoop-common-project
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-auth
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/conf/log4j.properties
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/docs
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/test/core

                
> address log4j.properties inconsistencies btw main and template dirs
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8216
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>    Affects Versions: 0.23.0
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>         Attachments: HADOOP-8216.patch
>
>
> DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)
> lesser issues but I noticed:
> I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 
> afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.
> Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

Posted by "Hudson (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-8216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13240383#comment-13240383 ] 

Hudson commented on HADOOP-8216:
--------------------------------

Integrated in Hadoop-Hdfs-trunk #998 (See [https://builds.apache.org/job/Hadoop-Hdfs-trunk/998/])
    HADOOP-8216. Address log4j.properties inconsistencies btw main and template dirs. Contributed by Patrick Hunt (Revision 1305891)

     Result = FAILURE
eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1305891
Files : 
* /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
* /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/conf/log4j.properties

                
> address log4j.properties inconsistencies btw main and template dirs
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8216
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>    Affects Versions: 0.23.0
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>         Attachments: HADOOP-8216.patch
>
>
> DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)
> lesser issues but I noticed:
> I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 
> afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.
> Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

Posted by "Hudson (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-8216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13239617#comment-13239617 ] 

Hudson commented on HADOOP-8216:
--------------------------------

Integrated in Hadoop-Common-0.23-Commit #734 (See [https://builds.apache.org/job/Hadoop-Common-0.23-Commit/734/])
    HADOOP-8216. svn merge -c 1305891 from trunk (Revision 1305892)

     Result = SUCCESS
eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1305892
Files : 
* /hadoop/common/branches/branch-0.23/hadoop-common-project
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-auth
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/conf/log4j.properties
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/docs
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/test/core

                
> address log4j.properties inconsistencies btw main and template dirs
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8216
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>    Affects Versions: 0.23.0
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>         Attachments: HADOOP-8216.patch
>
>
> DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)
> lesser issues but I noticed:
> I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 
> afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.
> Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

Posted by "Hadoop QA (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-8216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13239094#comment-13239094 ] 

Hadoop QA commented on HADOOP-8216:
-----------------------------------

-1 overall.  Here are the results of testing the latest attachment 
  http://issues.apache.org/jira/secure/attachment/12520037/HADOOP-8216.patch
  against trunk revision .

    +1 @author.  The patch does not contain any @author tags.

    -1 tests included.  The patch doesn't appear to include any new or modified tests.
                        Please justify why no new tests are needed for this patch.
                        Also please list what manual steps were performed to verify this patch.

    +1 javadoc.  The javadoc tool did not generate any warning messages.

    +1 javac.  The applied patch does not increase the total number of javac compiler warnings.

    +1 eclipse:eclipse.  The patch built with eclipse:eclipse.

    +1 findbugs.  The patch does not introduce any new Findbugs (version 1.3.9) warnings.

    +1 release audit.  The applied patch does not increase the total number of release audit warnings.

    -1 core tests.  The patch failed these unit tests:
                  org.apache.hadoop.fs.viewfs.TestViewFsTrash

    +1 contrib tests.  The patch passed contrib unit tests.

Test results: https://builds.apache.org/job/PreCommit-HADOOP-Build/787//testReport/
Console output: https://builds.apache.org/job/PreCommit-HADOOP-Build/787//console

This message is automatically generated.
                
> address log4j.properties inconsistencies btw main and template dirs
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8216
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>         Attachments: HADOOP-8216.patch
>
>
> DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)
> lesser issues but I noticed:
> I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 
> afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.
> Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

Posted by "Hudson (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-8216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13239616#comment-13239616 ] 

Hudson commented on HADOOP-8216:
--------------------------------

Integrated in Hadoop-Hdfs-0.23-Commit #724 (See [https://builds.apache.org/job/Hadoop-Hdfs-0.23-Commit/724/])
    HADOOP-8216. svn merge -c 1305891 from trunk (Revision 1305892)

     Result = SUCCESS
eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1305892
Files : 
* /hadoop/common/branches/branch-0.23/hadoop-common-project
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-auth
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/CHANGES.txt
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/conf/log4j.properties
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/docs
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/main/java
* /hadoop/common/branches/branch-0.23/hadoop-common-project/hadoop-common/src/test/core

                
> address log4j.properties inconsistencies btw main and template dirs
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8216
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>    Affects Versions: 0.23.0
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>         Attachments: HADOOP-8216.patch
>
>
> DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)
> lesser issues but I noticed:
> I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 
> afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.
> Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

Posted by "Patrick Hunt (Updated) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-8216?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Patrick Hunt updated HADOOP-8216:
---------------------------------

    Attachment: HADOOP-8216.patch

I checked over the template and it seemed a superset of main, so I've gone ahead and done as Eli suggested. I was able to start a cluster w/o issues and run a basic jobs. The generated logs looked fine to me.
                
> address log4j.properties inconsistencies btw main and template dirs
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8216
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>         Attachments: HADOOP-8216.patch
>
>
> DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)
> lesser issues but I noticed:
> I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 
> afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.
> Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

Posted by "Hudson (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/HADOOP-8216?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13239669#comment-13239669 ] 

Hudson commented on HADOOP-8216:
--------------------------------

Integrated in Hadoop-Mapreduce-trunk-Commit #1949 (See [https://builds.apache.org/job/Hadoop-Mapreduce-trunk-Commit/1949/])
    HADOOP-8216. Address log4j.properties inconsistencies btw main and template dirs. Contributed by Patrick Hunt (Revision 1305891)

     Result = ABORTED
eli : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1305891
Files : 
* /hadoop/common/trunk/hadoop-common-project/hadoop-common/CHANGES.txt
* /hadoop/common/trunk/hadoop-common-project/hadoop-common/src/main/conf/log4j.properties

                
> address log4j.properties inconsistencies btw main and template dirs
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8216
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>    Affects Versions: 0.23.0
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>         Attachments: HADOOP-8216.patch
>
>
> DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)
> lesser issues but I noticed:
> I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 
> afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.
> Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

Posted by "Eli Collins (Updated) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-8216?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Eli Collins updated HADOOP-8216:
--------------------------------

     Target Version/s: 0.23.3
    Affects Version/s: 0.23.0
         Hadoop Flags: Reviewed

+1  looks good
                
> address log4j.properties inconsistencies btw main and template dirs
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8216
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>    Affects Versions: 0.23.0
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>         Attachments: HADOOP-8216.patch
>
>
> DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)
> lesser issues but I noticed:
> I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 
> afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.
> Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

Posted by "Eli Collins (Updated) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-8216?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Eli Collins updated HADOOP-8216:
--------------------------------

    Resolution: Fixed
        Status: Resolved  (was: Patch Available)

Forgot to mention, the test failure is unrelated. I've committed and merged to branch-23. Thanks Pat!
                
> address log4j.properties inconsistencies btw main and template dirs
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8216
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>    Affects Versions: 0.23.0
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>         Attachments: HADOOP-8216.patch
>
>
> DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)
> lesser issues but I noticed:
> I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 
> afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.
> Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (HADOOP-8216) address log4j.properties inconsistencies btw main and template dirs

Posted by "Patrick Hunt (Updated) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/HADOOP-8216?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Patrick Hunt updated HADOOP-8216:
---------------------------------

    Status: Patch Available  (was: Open)
    
> address log4j.properties inconsistencies btw main and template dirs
> -------------------------------------------------------------------
>
>                 Key: HADOOP-8216
>                 URL: https://issues.apache.org/jira/browse/HADOOP-8216
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: conf
>            Reporter: Patrick Hunt
>            Assignee: Patrick Hunt
>            Priority: Minor
>         Attachments: HADOOP-8216.patch
>
>
> DRFAAUDIT is in ./hadoop-common-project/hadoop-common/src/main/packages/templates/conf/log4j.properties but not in ./hadoop-common-project/hadoop-common/src/main/conf/log4j.properties (although it's used in hadoop-env.sh)
> lesser issues but I noticed:
> I see MRAUDIT commented out of main, but it's uncommented in template. (the conversion pattern also differs). 
> afaict the other appenders (JSA, MRAUDIT, TLA, etc...) are all still being used and should not be removed.
> Should we be renaming some of these? JSA is being used by the history server for example.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira