[incubator-kyuubi] branch branch-1.3 updated: Revert "[KYUUBI #1318] Increase CONN_OPEN iff openSession is successful"

[chengpan]

This is an automated email from the ASF dual-hosted git repository.

chengpan pushed a commit to branch branch-1.3
in repository https://gitbox.apache.org/repos/asf/incubator-kyuubi.git


The following commit(s) were added to refs/heads/branch-1.3 by this push:
     new 603977c  Revert "[KYUUBI #1318] Increase CONN_OPEN iff openSession is 
successful"
603977c is described below

commit 603977c7b2a31a75cdbb39898f883ac48e241403
Author: Cheng Pan <cheng...@apache.org>
AuthorDate: Tue Nov 2 21:37:55 2021 +0800

    Revert "[KYUUBI #1318] Increase CONN_OPEN iff openSession is successful"
    
    This reverts commit ff98d49049c61e11e69a1d42b1848352f4a75df4.
---
 .../src/main/scala/org/apache/kyuubi/session/KyuubiSessionImpl.scala  | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git 
a/kyuubi-server/src/main/scala/org/apache/kyuubi/session/KyuubiSessionImpl.scala
 
b/kyuubi-server/src/main/scala/org/apache/kyuubi/session/KyuubiSessionImpl.scala
index 4fb7790..536464a 100644
--- 
a/kyuubi-server/src/main/scala/org/apache/kyuubi/session/KyuubiSessionImpl.scala
+++ 
b/kyuubi-server/src/main/scala/org/apache/kyuubi/session/KyuubiSessionImpl.scala
@@ -56,14 +56,12 @@ class KyuubiSessionImpl(
   override def open(): Unit = {
     MetricsSystem.tracing { ms =>
       ms.incCount(CONN_TOTAL)
+      ms.incCount(MetricRegistry.name(CONN_OPEN, user))
     }
     super.open()
     withZkClient(sessionConf) { zkClient =>
       val (host, port) = engine.getOrCreate(zkClient)
       openSession(host, port)
-      MetricsSystem.tracing { ms =>
-        ms.incCount(MetricRegistry.name(CONN_OPEN, user))
-      }
     }
   }