[Zookeeper-developer] [ zookeeper-Bugs-1958195 ] log4j is not present in zookeeper.jar

[SourceForge.net <no...@so...>]

Bugs item #1958195, was opened at 2008-05-05 19:37
Message generated for change (Comment added) made by mahadevkonar
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1008544&aid=1958195&group_id=209147

Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
>Resolution: Fixed
Priority: 5
Private: No
Submitted By: Erik Hetzner (egh)
Assigned to: Mahadev Konar (mahadevkonar)
Summary: log4j is not present in zookeeper.jar

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

----------------------------------------------------------------------

>Comment By: Mahadev Konar (mahadevkonar)
Date: 2008-05-15 22:07

Message:
Logged In: YES 
user_id=1926680
Originator: NO

fixed in trunk.

----------------------------------------------------------------------

Comment By: Mahadev Konar (mahadevkonar)
Date: 2008-05-06 19:51

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

----------------------------------------------------------------------

Comment By: Mahadev Konar (mahadevkonar)
Date: 2008-05-06 19:05

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

----------------------------------------------------------------------

Comment By: Patrick Hunt (phunt)
Date: 2008-05-05 22:52

Message:
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 )

----------------------------------------------------------------------

Comment By: Patrick Hunt (phunt)
Date: 2008-05-05 22:42

Message:
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.

----------------------------------------------------------------------

Comment By: Erik Hetzner (egh)
Date: 2008-05-05 22:15

Message:
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.



----------------------------------------------------------------------

Comment By: Benjamin Reed (breed)
Date: 2008-05-05 21:20

Message:
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.

----------------------------------------------------------------------

Comment By: Erik Hetzner (egh)
Date: 2008-05-05 19:44

Message:
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.

----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=1008544&aid=1958195&group_id=209147