#18 log4j is not present in zookeeper.jar

closed-fixed
None
5
2008-05-15
2008-05-05
No

The attached patch includes log4j classes in the zookeeper.jar file.

Discussion

  • Erik Hetzner

    Erik Hetzner - 2008-05-05

    patch to include log4j classes in zookeeper.jar

     
  • Erik Hetzner

    Erik Hetzner - 2008-05-05

    Logged In: YES
    user_id=67250
    Originator: YES

    Perhaps this is intentional?

    It means that the binary server jar distributed via sf.net is not stand alone, however.

     
  • Benjamin Reed

    Benjamin Reed - 2008-05-05

    Logged In: YES
    user_id=154690
    Originator: NO

    It is intentional. The fat jar proponent was out voted.

    We will be doing a new release shortly that distributes the java component as a tar.gz with scripts to start things up.

     
  • Erik Hetzner

    Erik Hetzner - 2008-05-05

    Logged In: YES
    user_id=67250
    Originator: YES

    Ok, I modified the wiki tutorial to remove information stating that the .jar file can be used as a standalone jar.

    You might also want to remove the .jar file distributed on sf.net, because it is impossible to run this file using -jar.

     
  • Patrick Hunt

    Patrick Hunt - 2008-05-05

    Logged In: YES
    user_id=12853
    Originator: NO

    Thanks egh -- I further updated the page to reference the zookeeper logging page. The packaging issues are being worked on and should be addressed shortly. Regards.

     
  • Patrick Hunt

    Patrick Hunt - 2008-05-05
    • assigned_to: nobody --> mahadevkonar
     
  • Patrick Hunt

    Patrick Hunt - 2008-05-05

    Logged In: YES
    user_id=12853
    Originator: NO

    Assigned to Mahadev who is working on log4j/zk.jar changes (Ben is looking at the startup script in a separate patch 1951806 )

     
  • Mahadev Konar

    Mahadev Konar - 2008-05-06

    Logged In: YES
    user_id=1926680
    Originator: NO

    hi erik,
    thanks for your patch. For now we will not be including log4j in zookeeper.jar. I am uploading a patch that builds log4j.properties in zookeeper.jar. So all we need in the classpath is log4.jar and zookeeper.jar. Does that seem good enough?

    File Added: log4j_build_1.patch

     
  • Mahadev Konar

    Mahadev Konar - 2008-05-06

    Logged In: YES
    user_id=1926680
    Originator: NO

    here is another version of the patch that will allow scripts to change the log4 settings.
    File Added: log4j_build_2.patch

     
  • Mahadev Konar

    Mahadev Konar - 2008-05-15
    • status: open --> closed-fixed
     
  • Mahadev Konar

    Mahadev Konar - 2008-05-15

    Logged In: YES
    user_id=1926680
    Originator: NO

    fixed in trunk.

     

Log in to post a comment.

Get latest updates about Open Source Projects, Conferences and News.

Sign up for the SourceForge newsletter:

JavaScript is required for this form.





No, thanks