You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kyuubi.apache.org by ch...@apache.org on 2022/12/03 19:50:20 UTC

[incubator-kyuubi] branch master updated: [KYUUBI #3882] Add eventTime in KyuubiSessionEvent

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 1b825b9cd [KYUUBI #3882] Add eventTime in KyuubiSessionEvent
1b825b9cd is described below

commit 1b825b9cd6147a915c82fa32628c307935bd704c
Author: wForget <64...@qq.com>
AuthorDate: Sun Dec 4 03:50:10 2022 +0800

    [KYUUBI #3882] Add eventTime in KyuubiSessionEvent
    
    ### _Why are the changes needed?_
    
    close #3882
    
    ### _How was this patch tested?_
    - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible
    
    - [ ] Add screenshots for manual tests if appropriate
    
    - [ ] [Run test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests) locally before make a pull request
    
    Closes #3883 from wForget/KYUUBI-3882.
    
    Closes #3882
    
    aeff5ebb [wForget] [KYUUBI-3882] Add eventTime in KyuubiSessionEvent
    
    Authored-by: wForget <64...@qq.com>
    Signed-off-by: Cheng Pan <ch...@apache.org>
---
 .../src/main/scala/org/apache/kyuubi/events/KyuubiSessionEvent.scala   | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/kyuubi-server/src/main/scala/org/apache/kyuubi/events/KyuubiSessionEvent.scala b/kyuubi-server/src/main/scala/org/apache/kyuubi/events/KyuubiSessionEvent.scala
index 12629e8c7..821017b29 100644
--- a/kyuubi-server/src/main/scala/org/apache/kyuubi/events/KyuubiSessionEvent.scala
+++ b/kyuubi-server/src/main/scala/org/apache/kyuubi/events/KyuubiSessionEvent.scala
@@ -30,6 +30,7 @@ import org.apache.kyuubi.session.KyuubiSession
  * @param serverIP A unique Kyuubi server id, e.g. kyuubi server ip address and port,
  *                 it is useful if has multi-instance Kyuubi Server
  * @param conf session config
+ * @param eventTime the time when the event made
  * @param startTime session create time
  * @param remoteSessionId remote engine session id
  * @param engineId engine id. For engine on yarn, it is applicationId.
@@ -47,6 +48,7 @@ case class KyuubiSessionEvent(
     clientIP: String,
     serverIP: String,
     conf: Map[String, String],
+    eventTime: Long,
     startTime: Long,
     var remoteSessionId: String = "",
     var engineId: String = "",
@@ -69,6 +71,7 @@ object KyuubiSessionEvent {
       session.ipAddress,
       session.connectionUrl,
       session.conf,
+      System.currentTimeMillis(),
       session.createTime)
   }
 }