[kyuubi] branch master updated: [KYUUBI #4438] Render the batch info map in kyuubi-ctl

[fe...@apache.org]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new 39eac5a78 [KYUUBI #4438] Render the batch info map in kyuubi-ctl
39eac5a78 is described below

commit 39eac5a7805d4e9c14c2852cc00bec1c480723f9
Author: fwang12 <fw...@ebay.com>
AuthorDate: Tue Mar 7 14:55:07 2023 +0800

    [KYUUBI #4438] Render the batch info map in kyuubi-ctl
    
    ### _Why are the changes needed?_
    
    Close #4438
    Follow up of #4390
    ### _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/master/develop_tools/testing.html#running-tests) locally before make a pull request
    
    Closes #4463 from turboFei/render_batch.
    
    Closes #4438
    
    b5ae211bc [fwang12] render batch info
    
    Authored-by: fwang12 <fw...@ebay.com>
    Signed-off-by: fwang12 <fw...@ebay.com>
---
 kyuubi-ctl/src/main/scala/org/apache/kyuubi/ctl/util/Render.scala | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/kyuubi-ctl/src/main/scala/org/apache/kyuubi/ctl/util/Render.scala b/kyuubi-ctl/src/main/scala/org/apache/kyuubi/ctl/util/Render.scala
index aba6df35a..2d4879e42 100644
--- a/kyuubi-ctl/src/main/scala/org/apache/kyuubi/ctl/util/Render.scala
+++ b/kyuubi-ctl/src/main/scala/org/apache/kyuubi/ctl/util/Render.scala
@@ -111,6 +111,9 @@ private[ctl] object Render {
 
   private def buildBatchAppInfo(batch: Batch, showDiagnostic: Boolean = true): List[String] = {
     val batchAppInfo = ListBuffer[String]()
+    batch.getBatchInfo.asScala.foreach { case (key, value) =>
+      batchAppInfo += s"$key: $value"
+    }
     if (batch.getAppStartTime > 0) {
       batchAppInfo += s"App Start Time:" +
         s" ${millisToDateString(batch.getAppStartTime, "yyyy-MM-dd HH:mm:ss")}"