Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Zookeeper, mail # dev - Re: Review Request: ZOOKEEPER-1346: Handle 4lws and monitoring on separate port (creating jetty server)


+
Skye Wanderman-Milne 2012-11-30, 21:14
+
Skye Wanderman-Milne 2012-11-30, 21:13
+
Skye Wanderman-Milne 2012-11-16, 23:25
+
Skye Wanderman-Milne 2012-11-30, 21:01
+
Henry Robinson 2012-12-11, 01:25
+
Patrick Hunt 2012-12-14, 17:39
+
Patrick Hunt 2012-12-14, 22:39
+
Skye Wanderman-Milne 2012-12-14, 22:28
+
Skye Wanderman-Milne 2012-12-18, 08:15
+
Skye Wanderman-Milne 2013-01-04, 02:17
+
Camille Fournier 2012-11-18, 16:24
+
Camille Fournier 2012-11-18, 16:21
+
Skye Wanderman-Milne 2012-11-19, 23:55
Copy link to this message
-
Re: Review Request: ZOOKEEPER-1346: Handle 4lws and monitoring on separate port (creating jetty server)
Camille Fournier 2012-11-18, 16:10

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8094/#review13547
-----------------------------------------------------------

src/java/main/org/apache/zookeeper/server/NIOServerCnxnFactory.java
<https://reviews.apache.org/r/8094/#comment29108>

    Why make this a more restrictive type?

src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java
<https://reviews.apache.org/r/8094/#comment29109>

    Maybe you should rewrite the dumpSessions that uses the PrintWriter to print the output of this method instead of duplicating some of the logic

src/java/main/org/apache/zookeeper/server/WatchManager.java
<https://reviews.apache.org/r/8094/#comment29110>

    Ditto on dumpWatches with the PrintWriter. Let's have one place for the logic for building up what should be written, and use the built object to print so if we decide to change the logic we only have to change it once.

src/java/main/org/apache/zookeeper/server/ZooKeeperServerMain.java
<https://reviews.apache.org/r/8094/#comment29111>

    What's with the TODO?
- Camille Fournier
On Nov. 16, 2012, 11:25 p.m., Skye Wanderman-Milne wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8094/
> -----------------------------------------------------------
>
> (Updated Nov. 16, 2012, 11:25 p.m.)
>
>
> Review request for zookeeper, Patrick Hunt, Camille Fournier, and Henry Robinson.
>
>
> Description
> -------
>
> See my comment in ZOOKEEPER-1346.
>
>
> This addresses bug ZOOKEEPER-1346.
>     https://issues.apache.org/jira/browse/ZOOKEEPER-1346
>
>
> Diffs
> -----
>
>   ivy.xml fadf4f4
>   src/java/main/org/apache/zookeeper/server/DataTree.java 0bb2317
>   src/java/main/org/apache/zookeeper/server/NIOServerCnxnFactory.java 8b4c46b
>   src/java/main/org/apache/zookeeper/server/NettyServerCnxnFactory.java eec2f2a
>   src/java/main/org/apache/zookeeper/server/ServerCnxn.java 6dd509b
>   src/java/main/org/apache/zookeeper/server/ServerCnxnFactory.java e5c6565
>   src/java/main/org/apache/zookeeper/server/ServerStats.java aa0d93f
>   src/java/main/org/apache/zookeeper/server/SessionTracker.java 3535e1b
>   src/java/main/org/apache/zookeeper/server/SessionTrackerImpl.java 31f2785
>   src/java/main/org/apache/zookeeper/server/WatchManager.java 0e7c815
>   src/java/main/org/apache/zookeeper/server/ZKDatabase.java d6c0c05
>   src/java/main/org/apache/zookeeper/server/ZooKeeperServer.java 7bb7b2f
>   src/java/main/org/apache/zookeeper/server/ZooKeeperServerMain.java 369e621
>   src/java/main/org/apache/zookeeper/server/ZooTrace.java ac14fe2
>   src/java/main/org/apache/zookeeper/server/admin/AdminServer.java PRE-CREATION
>   src/java/main/org/apache/zookeeper/server/admin/Command.java PRE-CREATION
>   src/java/main/org/apache/zookeeper/server/admin/CommandBase.java PRE-CREATION
>   src/java/main/org/apache/zookeeper/server/admin/CommandOutputter.java PRE-CREATION
>   src/java/main/org/apache/zookeeper/server/admin/Commands.java PRE-CREATION
>   src/java/main/org/apache/zookeeper/server/admin/JsonOutputter.java PRE-CREATION
>   src/java/main/org/apache/zookeeper/server/quorum/LearnerSessionTracker.java 3182419
>   src/java/test/org/apache/zookeeper/server/PrepRequestProcessorTest.java 8665bac
>   src/java/test/org/apache/zookeeper/server/quorum/Zab1_0Test.java 42207e1
>
> Diff: https://reviews.apache.org/r/8094/diff/
>
>
> Testing
> -------
>
> unit tests
>
> Ran in standalone mode (only option right now) and manually tried out all the commands/links
>
>
> Thanks,
>
> Skye Wanderman-Milne
>
>

+
Skye Wanderman-Milne 2012-11-30, 21:17
+
Skye Wanderman-Milne 2012-11-19, 23:55
+
Skye Wanderman-Milne 2013-01-15, 23:53