Skip to content
This repository has been archived by the owner on Oct 30, 2020. It is now read-only.

set connect timeout to a more reasonable value than 100ms #11

Closed
wants to merge 2 commits into from
Closed

set connect timeout to a more reasonable value than 100ms #11

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
837a382
set connect timeout to a more reasonable value (this fails for us in …
ezbz Nov 26, 2012
81d8d35
added a parameter to the CLI to control zookeeper connection timeout
ezbz Nov 28, 2012
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 5 additions & 3 deletions org.linkedin.zookeeper-cli-impl/src/main/groovy/org/linkedin/zookeeper/cli/ClientMain.groovy
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,9 +60,10 @@ public class ClientMain

def start()
{
def connectString = config.zkConnectionString ?: "localhost:2181"
def connectString = config.zkConnectionString ?: "localhost:2181"
def connectTimeout = config.zkConnectionTimeout ?: "100"

client = new ZKClient(connectString, Timespan.parse('100'), null)
client = new ZKClient(connectString, Timespan.parse(connectTimeout), null)

client.start()
log.debug "Talking to zookeeper on ${connectString}"
Expand All @@ -78,9 +79,10 @@ public class ClientMain

static int mainNoExit(Object args)
{
def cli = new CliBuilder(usage: './bin/zk.sh [-h] [-s <zkConnectionString>] command')
def cli = new CliBuilder(usage: './bin/zk.sh [-h] [-s <zkConnectionString>] [-t <zkConnectionTimeout>] command')
cli.h(longOpt: 'help', 'display help')
cli.s(longOpt: 'zkConnectionString', 'the zookeeper connection string (host:port)', args: 1, required: false)
cli.t(longOpt: 'zkConnectionTimeout', 'the zookeeper connection timeout (e.g. 10s)', args: 2, required: false)

// this is splitting the main cmdline args from the command and its args
boolean isMainArg = true
Expand Down