[ZOOKEEPER-114] cleanup ugly event messages in zookeeper client

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Fix Version/s: 3.0.0
• Component/s: java client
• Hadoop Flags: Reviewed
• Assignee: Jakob Homan
• Reporter: Patrick D. Hunt
• Created: 06/Aug/08 23:31
• Updated: 26/Oct/08 01:10
• Resolved: 01/Oct/08 02:49

Description

Cleanup the event messages output by the java command line client:

The java client ZooKeeper.java has a process method in MyWatcher that prints raw event information to the console. Many new users find this confusing (esp since state changes have a null path).

public void process(WatcherEvent event)

{ System.err.println(event.getPath() + ": " + event.getState() + "-" + event.getType()); }

If this is a state change we should print that, if it's an event we should print that (state change doesn't have a path). Somewhere in the message we should also indicate that the client received a watch notification from the server.

Issue Links

• is blocked by: ZOOKEEPER-144 add tostring support for watcher event, and enums for event type/state
• is part of: ZOOKEEPER-144 add tostring support for watcher event, and enums for event type/state