[incubator-kyuubi] branch master updated: [KYUUBI #2255]The engine state of Spark's EngineEvent is hardcoded with 0

[ya...@apache.org]

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

yao 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 6bd9edf  [KYUUBI #2255]The engine state of Spark's EngineEvent is hardcoded with 0
6bd9edf is described below

commit 6bd9edf27f07d6c229bfcb0266046daa8771a1e4
Author: Nick Song <ch...@163.com>
AuthorDate: Fri Apr 1 22:12:24 2022 +0800

    [KYUUBI #2255]The engine state of Spark's EngineEvent is hardcoded with 0
    
    ### _Why are the changes needed?_
    
     resolve this bug #2255
    ### _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
    
    - [x] [Run test](https://kyuubi.apache.org/docs/latest/develop_tools/testing.html#running-tests) locally before make a pull request
    
    Closes #2264 from Nick-0723/fix_engine_state.
    
    Closes #2255
    
    dcc7b49c [Nick Song] remove unnecessary codes
    426af108 [Nick Song] fix engine state
    
    Authored-by: Nick Song <ch...@163.com>
    Signed-off-by: Kent Yao <ya...@apache.org>
---
 .../main/scala/org/apache/kyuubi/engine/spark/events/EngineEvent.scala  | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/externals/kyuubi-spark-sql-engine/src/main/scala/org/apache/kyuubi/engine/spark/events/EngineEvent.scala b/externals/kyuubi-spark-sql-engine/src/main/scala/org/apache/kyuubi/engine/spark/events/EngineEvent.scala
index da8faa3..32f7858 100644
--- a/externals/kyuubi-spark-sql-engine/src/main/scala/org/apache/kyuubi/engine/spark/events/EngineEvent.scala
+++ b/externals/kyuubi-spark-sql-engine/src/main/scala/org/apache/kyuubi/engine/spark/events/EngineEvent.scala
@@ -115,7 +115,7 @@ object EngineEvent {
       webUrl,
       sc.startTime,
       endTime = -1L,
-      state = 0,
+      state = engine.getServiceState.id,
       diagnostic = "",
       sc.getConf.getAll.toMap ++ engine.getConf.getAll)
   }