naming "zkgraph" is not very descriptive - ppl may think it graphs zk namespace? How about "logvisualizer" or "loggrapher" (something indicating that the contrib visualizes the log). Although even this is potentially misleading (txn log vs log4j log). You have a great description at the top of the readme which is good though. (perhaps indicate that this is log4j log and not txn log?)
lib dir missing from patch (build fails, I created by hand and it was fine, good to get into patch though)
bin scripts are not executable - perhaps update patch, also committer should chmod the scripts to add "x" if missing
add apache license to zkgraph.css
LogFormatter.java has license twice
For some reason the app itself does not work for me. I click on the links (I tried both chrome and firefox) but nothing happens. "add log" for example, I click that and nothing at all happens. Here's the console that's running the zkgraph.sh script (this is the entire content of the console):
MergedLogSource(size=0, start=0, end=0)
2010-06-01 09:52:35.695:INFO::Logging to StdErrLog::DEBUG=false via org.eclipse.jetty.util.log.StdErrLog
2010-06-01 09:52:35.969:INFO::Started SelectChannelConnector@0.0.0.0:8182
log4j:WARN No appenders could be found for logger (org.apache.zookeeper.graph.servlets.NumEvents).
log4j:WARN Please initialize the log4j system properly.