rename plus one bug fix

codegen/templates/http_client.tmpl

@@ -238,17 +238,19 @@ func (c *{{$clientName}}) {{$methodName}}(
 	if (c.circuitBreakerDisabled) {
 		res, err = req.Do()
 	} else {
-		var realErr error
+		// We want hystrix ckt-breaker to count errors only for system issues
+		var clientErr error
 		err = hystrix.DoC(ctx, "{{$clientID}}", func(ctx context.Context) error {
-			res, realErr = req.Do()
+			res, clientErr = req.Do()
 			if res.StatusCode < 500 {
+				// This is not a system error/issue
 				return nil
 			}
-			return realErr
+			return clientErr
 		}, nil)
 		if err == nil {
-			// Bad request or equivalent error, bubble it up
-			err = realErr
+			// ckt-breaker was ok, bubble up client error if set
+			err = clientErr
 		}
 	}
 	if err != nil {

codegen/templates/tchannel_client.tmpl

@@ -264,23 +264,24 @@ type {{$clientName}} struct {
 				ctx, "{{$svc.Name}}", "{{.Name}}", reqHeaders, args, &result,
 			)
 		} else {
-			var realErr error
+			// We want hystrix ckt-breaker to count errors only for system issues
+			var clientErr error
 			err = hystrix.DoC(ctx, "{{$clientID}}", func(ctx context.Context) error {
-				success, respHeaders, err = c.client.Call(
+				success, respHeaders, clientErr = c.client.Call(
 					ctx, "{{$svc.Name}}", "{{.Name}}", reqHeaders, args, &result,
 				)
-				if _, isSysErr := err.(tchannel.SystemError); !isSysErr {
+				if _, isSysErr := clientErr.(tchannel.SystemError); !isSysErr {
+					// Declare ok if it is not a system-error
 					return nil
 				}
-				return err
+				return clientErr
 			}, nil)
 			if err == nil {
-				// Bad request or equivalent error, bubble it up
-				err = realErr
+				// ckt-breaker was ok, bubble up client error if set
+				err = clientErr
 			}
 		}
 
-
 		if err == nil && !success {
 			switch {
 				{{range .Exceptions -}}