From d64a91784a16206c036048a5006a103ed7fe0d89 Mon Sep 17 00:00:00 2001
From: Renan DelValle <renanidelvalle+noreply@gmail.com>
Date: Mon, 16 Apr 2018 17:54:41 -0700
Subject: [PATCH] Turn on logging by default.

---
 examples/client.go |  2 --
 realis.go          | 10 ++++------
 2 files changed, 4 insertions(+), 8 deletions(-)

diff --git a/examples/client.go b/examples/client.go
index 7162408..aaee92e 100644
--- a/examples/client.go
+++ b/examples/client.go
@@ -18,7 +18,6 @@ import (
 	"flag"
 	"fmt"
 	"io/ioutil"
-	"log"
 	"os"
 
 	"time"
@@ -90,7 +89,6 @@ func main() {
 			Factor:   2.0,
 			Jitter:   0.1,
 		}),
-		realis.SetLogger(log.New(os.Stdout, "realis-debug: ", log.Ldate)),
 		realis.Debug(),
 	}
 
diff --git a/realis.go b/realis.go
index a697a60..4fd90bb 100644
--- a/realis.go
+++ b/realis.go
@@ -27,9 +27,8 @@ import (
 	"os"
 	"path/filepath"
 	"strings"
-	"time"
-
 	"sync"
+	"time"
 
 	"git.apache.org/thrift.git/lib/go/thrift"
 	"github.com/paypal/gorealis/gen-go/apache/aurora"
@@ -250,6 +249,7 @@ func NewRealisClient(options ...ClientOption) (Realis, error) {
 	// Default configs
 	config.timeoutms = 10000
 	config.backoff = defaultBackoff
+	config.logger = log.New(os.Stdout, "realis: ", log.Ltime|log.Ldate|log.LUTC)
 
 	// Save options to recreate client if a connection error happens
 	config.options = options
@@ -259,10 +259,8 @@ func NewRealisClient(options ...ClientOption) (Realis, error) {
 		opt(config)
 	}
 
-	// Set a logger if debug has been set to true but no logger has been set, otherwise, set noop logger
-	if config.logger == nil && config.debug {
-		config.logger = log.New(os.Stdout, "realis: ", log.Ltime|log.Ldate|log.LUTC)
-	} else if config.logger == nil {
+	// Turn off all logging (including debug)
+	if config.logger == nil {
 		config.logger = LevelLogger{NoopLogger{}, false}
 	}