review comments: default time period for 30s, etc.

benchmark.js

@@ -41,7 +41,13 @@ function Benchmark(options) {
     self.cmdOptions = options.cmdOptions;
     self.requests = options.cmdOptions.requests || 0;
     self.requestsLimit = self.requests !== 0;
-    self.time = options.cmdOptions.time || 0;
+    self.time = options.cmdOptions.time;
+    if (self.requests === 0 && self.time === undefined) {
+        self.time = 30 * 1000;
+    } else {
+        self.time = 0;
+    }
+
     self.timeLimit = self.time !== 0;
     self.delay = options.cmdOptions.delay || 100;
     self.rate = options.cmdOptions.rate;

index.js

@@ -224,6 +224,13 @@ function parseArgs(argv) {
 }
 
 function TCurl(opts) {
+    if (!(this instanceof TCurl)) {
+        return new TCurl(opts);
+    }
+
+    var self = this;
+    self.subChannel = null;
+    self.client = null;
 }
 
 TCurl.prototype.parseJsonArgs = function parseJsonArgs(opts, delegate) {

man/tcurl.1

@@ -166,7 +166,8 @@ tcurl \-p localhost:8080 serviceName \-\-health \-\-rate 1000 \-\-delay 100
 .P
 \fB\-\-time value\fP
     Specify the time in milliseconds how long the benchmark should run\.
-    By default, there is no limit on the time length\.
+    When no request limit is set, the default value is 30 seconds\. Otherwise,
+    the default is unlimited\.
 .SH EXIT CODES
 .RS 0
 .IP \(bu 2

usage.md

@@ -129,7 +129,8 @@ a tchannel service. It supports thrift, JSON, and raw request format.
 
 `--time value`
     Specify the time in milliseconds how long the benchmark should run.
-    By default, there is no limit on the time length.
+    When no request limit is set, the default value is 30 seconds. Otherwise,
+    the default is unlimited.
 
 
 ## EXIT CODES