Close client before session...

... avoids the problem of session.Close() blocking and causing failures.
gluster · Apr 12, 2017 · 09ceb28 · 09ceb28
1 parent 2620948
commit 09ceb28
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/store/store.go b/store/store.go
@@ -48,10 +48,10 @@ func New() *GDStore {
 
 // Close closes the store connections
 func (s *GDStore) Close() {
-	if e := s.Session.Close(); e != nil {
-		log.WithError(e).Warn("failed to close etcd session")
-	}
 	if e := s.Client.Close(); e != nil {
 		log.WithError(e).Warn("failed to close etcd client connection")
 	}
+	if e := s.Session.Close(); e != nil {
+		log.WithError(e).Warn("failed to close etcd session")
+	}
 }