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)


Copy link to this message
-
Re: Review Request: ZOOKEEPER-1346: Handle 4lws and monitoring on separate port (creating jetty server)


> On Nov. 28, 2012, 6:19 p.m., Patrick Hunt wrote:
> > src/java/main/org/apache/zookeeper/server/ZooKeeperServerMain.java, line 111
> > <https://reviews.apache.org/r/8094/diff/1/?file=190885#file190885line111>
> >
> >     This needs to be configurable from the get go.
>
> Skye Wanderman-Milne wrote:
>     I'll add configuration options for the base URL, port, anything else I can think of.

Added system properties for port, base URL, and commands URL.
- Skye
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/8094/#review13824
-----------------------------------------------------------
On Nov. 30, 2012, 9:01 p.m., Skye Wanderman-Milne wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8094/
> -----------------------------------------------------------
>
> (Updated Nov. 30, 2012, 9:01 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/Leader.java 4d09b43
>   src/java/main/org/apache/zookeeper/server/quorum/Learner.java e8d548b
>   src/java/main/org/apache/zookeeper/server/quorum/LearnerSessionTracker.java 3182419
>   src/java/main/org/apache/zookeeper/server/quorum/QuorumPeer.java 4e3a87d
>   src/java/main/org/apache/zookeeper/server/quorum/QuorumPeerMain.java deae926
>   src/java/main/org/apache/zookeeper/server/quorum/ReadOnlyZooKeeperServer.java d3f1492
>   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: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
+
Camille Fournier 2012-11-18, 16:10
+
Skye Wanderman-Milne 2012-11-30, 21:17
+
Skye Wanderman-Milne 2012-11-19, 23:55
+
Skye Wanderman-Milne 2013-01-15, 23:53