Patrick Hunt updated ZOOKEEPER-82:

    Status: Open  (was: Patch Available)

I'm still unable to apply this patch, cancelling:

1) I get the following error during the patch

patch -p0 < /home/phunt/Desktop/ZOOKEEPER-82-b.patch 
patching file src/java/main/org/apache/zookeeper/ZooKeeper.java
Reversed (or previously applied) patch detected!  Assume -R? [n] 

I forced the patch, then fixed up the rejected patch issue by hand but there 
are still problems, 

2) ObservableNIOServerCnxn is referenced by the patch file, but not in the 
mainline SVN nor in the patch file itself.

> Make the ZooKeeperServer more DI friendly
> -----------------------------------------
>                 Key: ZOOKEEPER-82
>                 URL: https://issues.apache.org/jira/browse/ZOOKEEPER-82
>             Project: Zookeeper
>          Issue Type: Improvement
>          Components: server
>            Reporter: Hiram Chirino
>            Assignee: Hiram Chirino
>         Attachments: ZOOKEEPER-82-b.patch
> Proposed changes were discussed in [this mailing list 
> thread|http://mail-archives.apache.org/mod_mbox/hadoop-zookeeper-dev/200807.mbox/[EMAIL
> Basic goals are: 
> * Decouple the current configuration system from the public API.  I
> see stuff like ZooKeeperServer being coupled to ServerConfig a bit.
> * Allow the use of setter injection in addition to constructor
> injection. This is the most important thing needed to let spring more
> easily configure the objects.
> * Move the main() methods out of the ZooKeeperServer class.

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

Reply via email to