You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@flume.apache.org by "Sid Askary (Created) (JIRA)" <ji...@apache.org> on 2011/12/17 01:42:30 UTC

[jira] [Created] (FLUME-894) Add log4j as part of the build

Add log4j as part of the build
------------------------------

                 Key: FLUME-894
                 URL: https://issues.apache.org/jira/browse/FLUME-894
             Project: Flume
          Issue Type: Bug
          Components: Build
    Affects Versions: NG alpha 2
         Environment: ubuntu 10.10 vm
            Reporter: Sid Askary
            Assignee: E. Sammer
            Priority: Minor


log4j needs to added as part of the build - as a concrete implementation of sfl4j.

--
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] (FLUME-894) Add log4j as part of the build

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

jiraposter@reviews.apache.org commented on FLUME-894:
-----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3295/#review4086
-----------------------------------------------------------

Ship it!


+1

Don't we need to also add the log4j bindings for slf4j dependency?

- Arvind


On 2011-12-22 20:45:50, Eric Sammer wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/3295/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2011-12-22 20:45:50)
bq.  
bq.  
bq.  Review request for Flume.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Made log4j a dep so it's included in the resultant artifact.
bq.  
bq.  
bq.  This addresses bug FLUME-894.
bq.      https://issues.apache.org/jira/browse/FLUME-894
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    i/flume-ng-node/pom.xml a70cf26 
bq.  
bq.  Diff: https://reviews.apache.org/r/3295/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Eric
bq.  
bq.


                
> Add log4j as part of the build
> ------------------------------
>
>                 Key: FLUME-894
>                 URL: https://issues.apache.org/jira/browse/FLUME-894
>             Project: Flume
>          Issue Type: Bug
>          Components: Build
>    Affects Versions: NG alpha 2
>         Environment: ubuntu 10.10 vm
>            Reporter: Sid Askary
>            Assignee: E. Sammer
>            Priority: Minor
>             Fix For: NG alpha 3
>
>
> log4j needs to added as part of the build - as a concrete implementation of sfl4j.

--
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] (FLUME-894) Add log4j as part of the build

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

Hudson commented on FLUME-894:
------------------------------

Integrated in flume-728 #82 (See [https://builds.apache.org/job/flume-728/82/])
    FLUME-894: Add log4j as part of the build

esammer : http://svn.apache.org/viewvc/?view=rev&rev=1222466
Files : 
* /incubator/flume/branches/flume-728/flume-ng-node/pom.xml

                
> Add log4j as part of the build
> ------------------------------
>
>                 Key: FLUME-894
>                 URL: https://issues.apache.org/jira/browse/FLUME-894
>             Project: Flume
>          Issue Type: Bug
>          Components: Build
>    Affects Versions: NG alpha 2
>         Environment: ubuntu 10.10 vm
>            Reporter: Sid Askary
>            Assignee: E. Sammer
>            Priority: Minor
>             Fix For: NG alpha 3
>
>
> log4j needs to added as part of the build - as a concrete implementation of sfl4j.

--
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] (FLUME-894) Add log4j as part of the build

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

E. Sammer updated FLUME-894:
----------------------------

    Fix Version/s: NG alpha 3
    
> Add log4j as part of the build
> ------------------------------
>
>                 Key: FLUME-894
>                 URL: https://issues.apache.org/jira/browse/FLUME-894
>             Project: Flume
>          Issue Type: Bug
>          Components: Build
>    Affects Versions: NG alpha 2
>         Environment: ubuntu 10.10 vm
>            Reporter: Sid Askary
>            Assignee: E. Sammer
>            Priority: Minor
>             Fix For: NG alpha 3
>
>
> log4j needs to added as part of the build - as a concrete implementation of sfl4j.

--
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] (FLUME-894) Add log4j as part of the build

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

E. Sammer updated FLUME-894:
----------------------------

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

Committed to branch flume-728.
                
> Add log4j as part of the build
> ------------------------------
>
>                 Key: FLUME-894
>                 URL: https://issues.apache.org/jira/browse/FLUME-894
>             Project: Flume
>          Issue Type: Bug
>          Components: Build
>    Affects Versions: NG alpha 2
>         Environment: ubuntu 10.10 vm
>            Reporter: Sid Askary
>            Assignee: E. Sammer
>            Priority: Minor
>             Fix For: NG alpha 3
>
>
> log4j needs to added as part of the build - as a concrete implementation of sfl4j.

--
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] (FLUME-894) Add log4j as part of the build

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

jiraposter@reviews.apache.org commented on FLUME-894:
-----------------------------------------------------



bq.  On 2011-12-22 21:31:15, Arvind Prabhakar wrote:
bq.  > +1
bq.  > 
bq.  > Don't we need to also add the log4j bindings for slf4j dependency?

Ignore my comment - I see it is already there.


- Arvind


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3295/#review4086
-----------------------------------------------------------


On 2011-12-22 20:45:50, Eric Sammer wrote:
bq.  
bq.  -----------------------------------------------------------
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/3295/
bq.  -----------------------------------------------------------
bq.  
bq.  (Updated 2011-12-22 20:45:50)
bq.  
bq.  
bq.  Review request for Flume.
bq.  
bq.  
bq.  Summary
bq.  -------
bq.  
bq.  Made log4j a dep so it's included in the resultant artifact.
bq.  
bq.  
bq.  This addresses bug FLUME-894.
bq.      https://issues.apache.org/jira/browse/FLUME-894
bq.  
bq.  
bq.  Diffs
bq.  -----
bq.  
bq.    i/flume-ng-node/pom.xml a70cf26 
bq.  
bq.  Diff: https://reviews.apache.org/r/3295/diff
bq.  
bq.  
bq.  Testing
bq.  -------
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Eric
bq.  
bq.


                
> Add log4j as part of the build
> ------------------------------
>
>                 Key: FLUME-894
>                 URL: https://issues.apache.org/jira/browse/FLUME-894
>             Project: Flume
>          Issue Type: Bug
>          Components: Build
>    Affects Versions: NG alpha 2
>         Environment: ubuntu 10.10 vm
>            Reporter: Sid Askary
>            Assignee: E. Sammer
>            Priority: Minor
>             Fix For: NG alpha 3
>
>
> log4j needs to added as part of the build - as a concrete implementation of sfl4j.

--
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] [Closed] (FLUME-894) Add log4j as part of the build

Posted by "E. Sammer (Closed) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/FLUME-894?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

E. Sammer closed FLUME-894.
---------------------------

    
> Add log4j as part of the build
> ------------------------------
>
>                 Key: FLUME-894
>                 URL: https://issues.apache.org/jira/browse/FLUME-894
>             Project: Flume
>          Issue Type: Bug
>          Components: Build
>    Affects Versions: NG alpha 2
>         Environment: ubuntu 10.10 vm
>            Reporter: Sid Askary
>            Assignee: E. Sammer
>            Priority: Minor
>             Fix For: NG alpha 3
>
>
> log4j needs to added as part of the build - as a concrete implementation of sfl4j.

--
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] (FLUME-894) Add log4j as part of the build

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

E. Sammer updated FLUME-894:
----------------------------

    Status: Patch Available  (was: Open)
    
> Add log4j as part of the build
> ------------------------------
>
>                 Key: FLUME-894
>                 URL: https://issues.apache.org/jira/browse/FLUME-894
>             Project: Flume
>          Issue Type: Bug
>          Components: Build
>    Affects Versions: NG alpha 2
>         Environment: ubuntu 10.10 vm
>            Reporter: Sid Askary
>            Assignee: E. Sammer
>            Priority: Minor
>
> log4j needs to added as part of the build - as a concrete implementation of sfl4j.

--
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] (FLUME-894) Add log4j as part of the build

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

jiraposter@reviews.apache.org commented on FLUME-894:
-----------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3295/
-----------------------------------------------------------

Review request for Flume.


Summary
-------

Made log4j a dep so it's included in the resultant artifact.


This addresses bug FLUME-894.
    https://issues.apache.org/jira/browse/FLUME-894


Diffs
-----

  i/flume-ng-node/pom.xml a70cf26 

Diff: https://reviews.apache.org/r/3295/diff


Testing
-------


Thanks,

Eric


                
> Add log4j as part of the build
> ------------------------------
>
>                 Key: FLUME-894
>                 URL: https://issues.apache.org/jira/browse/FLUME-894
>             Project: Flume
>          Issue Type: Bug
>          Components: Build
>    Affects Versions: NG alpha 2
>         Environment: ubuntu 10.10 vm
>            Reporter: Sid Askary
>            Assignee: E. Sammer
>            Priority: Minor
>             Fix For: NG alpha 3
>
>
> log4j needs to added as part of the build - as a concrete implementation of sfl4j.

--
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