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 f3b6609  [KYUUBI #1141] Remove redundant unit
f3b6609 is described below

commit f3b6609e5da48216ff6121c4d19e8ad15429416b
Author: timothy65535 <[email protected]>
AuthorDate: Thu Sep 23 19:32:15 2021 +0800

    [KYUUBI #1141] Remove redundant unit
    
    <!--
    Thanks for sending a pull request!
    
    Here are some tips for you:
      1. If this is your first time, please read our contributor guidelines: 
https://kyuubi.readthedocs.io/en/latest/community/contributions.html
      2. If the PR is related to an issue in 
https://github.com/apache/incubator-kyuubi/issues, add '[KYUUBI #XXXX]' in your 
PR title, e.g., '[KYUUBI #XXXX] Your PR title ...'.
      3. If the PR is unfinished, add '[WIP]' in your PR title, e.g., 
'[WIP][KYUUBI #XXXX] Your PR title ...'.
    -->
    
    ### _Why are the changes needed?_
    <!--
    Please clarify why the changes are needed. For instance,
      1. If you add a feature, you can talk about the use case of it.
      2. If you fix a bug, you can clarify why it is a bug.
    -->
    
    Remove redundant unit. `MB`
    
    ```
    2021-09-23 08:41:08.889 INFO spark.SparkSQLEngine:
        Spark application name: 
kyuubi_USER_root_6f67c12c-f362-4a07-87d0-7aa7e5f43171
              application ID:  spark-461363e9cc934d1986f7ae4191dc0410
              application web UI: 
http://spark-740a727c11ce3da7-driver-svc.default.svc:44692
              master: k8s://https://10.10.9.35:6443
              version: 3.1.2
              driver: [cpu: 1, mem: 1G]
              executor: [cpu: 2, mem: 2G MB, maxNum: 18]
        Start time: Thu Sep 23 08:40:39 UTC 2021
    
        User: root (shared mode: USER)
        State: LATENT
    ```
    ### _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.readthedocs.io/en/latest/develop_tools/testing.html#running-tests)
 locally before make a pull request
    
    Closes #1141 from timothy65535/remove.
    
    Closes #1141
    
    af1827e9 [timothy65535] Remove redundant unit
    
    Authored-by: timothy65535 <[email protected]>
    Signed-off-by: ulysses-you <[email protected]>
---
 .../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 77d6c54..b90b873 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
@@ -82,7 +82,7 @@ case class EngineEvent(
        |          master: $master
        |          version: $sparkVersion
        |          driver: [cpu: $driverCores, mem: $driverMemory]
-       |          executor: [cpu: $executorCore, mem: $executorMemory MB, 
maxNum: $maxExecutors]
+       |          executor: [cpu: $executorCore, mem: $executorMemory, maxNum: 
$maxExecutors]
        |    Start time: ${new Date(startTime)}
        |    ${if (endTime != -1L) "End time: " + new Date(endTime) else ""}
        |    User: $owner (shared mode: $shareLevel)

Reply via email to