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

arshad pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/seatunnel-web.git


The following commit(s) were added to refs/heads/main by this push:
     new d9383ab9 [hotfix] Fix the problem that create_user and update_user are 
not displayed. (#252)
d9383ab9 is described below

commit d9383ab963020842f9f33b6f4b139aafa8f6f215
Author: Jast <[email protected]>
AuthorDate: Wed Jan 8 14:36:33 2025 +0800

    [hotfix] Fix the problem that create_user and update_user are not 
displayed. (#252)
---
 .../seatunnel/app/dal/dao/IJobDefinitionDao.java   |  4 +-
 .../app/dal/dao/impl/JobDefinitionDaoImpl.java     |  7 ++--
 .../apache/seatunnel/app/dal/mapper/JobMapper.java |  5 ++-
 .../app/service/impl/JobDefinitionServiceImpl.java | 31 +-------------
 .../apache/seatunnel/app/dal/mapper/JobMapper.xml  | 49 ++++++++++++++--------
 5 files changed, 43 insertions(+), 53 deletions(-)

diff --git 
a/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/dal/dao/IJobDefinitionDao.java
 
b/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/dal/dao/IJobDefinitionDao.java
index 647936a4..8ff36100 100644
--- 
a/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/dal/dao/IJobDefinitionDao.java
+++ 
b/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/dal/dao/IJobDefinitionDao.java
@@ -19,6 +19,7 @@ package org.apache.seatunnel.app.dal.dao;
 
 import org.apache.seatunnel.app.dal.entity.JobDefinition;
 import org.apache.seatunnel.app.domain.response.PageInfo;
+import org.apache.seatunnel.app.domain.response.job.JobDefinitionRes;
 
 import lombok.NonNull;
 
@@ -32,7 +33,8 @@ public interface IJobDefinitionDao {
 
     void updateJob(JobDefinition jobDefinition);
 
-    PageInfo<JobDefinition> getJob(String name, Integer pageNo, Integer 
pageSize, String jobMode);
+    PageInfo<JobDefinitionRes> getJob(
+            String name, Integer pageNo, Integer pageSize, String jobMode);
 
     List<JobDefinition> getJobList(@NonNull String name);
 
diff --git 
a/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/dal/dao/impl/JobDefinitionDaoImpl.java
 
b/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/dal/dao/impl/JobDefinitionDaoImpl.java
index 2d010cc8..1563b89a 100644
--- 
a/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/dal/dao/impl/JobDefinitionDaoImpl.java
+++ 
b/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/dal/dao/impl/JobDefinitionDaoImpl.java
@@ -21,6 +21,7 @@ import org.apache.seatunnel.app.dal.dao.IJobDefinitionDao;
 import org.apache.seatunnel.app.dal.entity.JobDefinition;
 import org.apache.seatunnel.app.dal.mapper.JobMapper;
 import org.apache.seatunnel.app.domain.response.PageInfo;
+import org.apache.seatunnel.app.domain.response.job.JobDefinitionRes;
 
 import org.apache.commons.lang3.StringUtils;
 
@@ -56,9 +57,9 @@ public class JobDefinitionDaoImpl implements 
IJobDefinitionDao {
     }
 
     @Override
-    public PageInfo<JobDefinition> getJob(
+    public PageInfo<JobDefinitionRes> getJob(
             String searchName, Integer pageNo, Integer pageSize, String 
jobMode) {
-        IPage<JobDefinition> jobDefinitionIPage;
+        IPage<JobDefinitionRes> jobDefinitionIPage;
         if (StringUtils.isEmpty(jobMode)) {
             jobDefinitionIPage =
                     jobMapper.queryJobListPaging(new Page<>(pageNo, pageSize), 
searchName);
@@ -67,7 +68,7 @@ public class JobDefinitionDaoImpl implements 
IJobDefinitionDao {
                     jobMapper.queryJobListPagingWithJobMode(
                             new Page<>(pageNo, pageSize), searchName, jobMode);
         }
-        PageInfo<JobDefinition> jobs = new PageInfo<>();
+        PageInfo<JobDefinitionRes> jobs = new PageInfo<>();
         jobs.setData(jobDefinitionIPage.getRecords());
         jobs.setPageSize(pageSize);
         jobs.setPageNo(pageNo);
diff --git 
a/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/dal/mapper/JobMapper.java
 
b/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/dal/mapper/JobMapper.java
index 4b07620b..b84894af 100644
--- 
a/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/dal/mapper/JobMapper.java
+++ 
b/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/dal/mapper/JobMapper.java
@@ -18,6 +18,7 @@
 package org.apache.seatunnel.app.dal.mapper;
 
 import org.apache.seatunnel.app.dal.entity.JobDefinition;
+import org.apache.seatunnel.app.domain.response.job.JobDefinitionRes;
 
 import org.apache.ibatis.annotations.Param;
 
@@ -28,10 +29,10 @@ import java.util.List;
 
 public interface JobMapper extends BaseMapper<JobDefinition> {
 
-    IPage<JobDefinition> queryJobListPaging(
+    IPage<JobDefinitionRes> queryJobListPaging(
             IPage<JobDefinition> page, @Param("searchName") String searchName);
 
-    IPage<JobDefinition> queryJobListPagingWithJobMode(
+    IPage<JobDefinitionRes> queryJobListPagingWithJobMode(
             IPage<JobDefinition> page,
             @Param("searchName") String searchName,
             @Param("jobMode") String jobMode);
diff --git 
a/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/service/impl/JobDefinitionServiceImpl.java
 
b/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/service/impl/JobDefinitionServiceImpl.java
index 38c172ec..b97a55ff 100644
--- 
a/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/service/impl/JobDefinitionServiceImpl.java
+++ 
b/seatunnel-server/seatunnel-app/src/main/java/org/apache/seatunnel/app/service/impl/JobDefinitionServiceImpl.java
@@ -119,36 +119,7 @@ public class JobDefinitionServiceImpl extends 
SeatunnelBaseServiceImpl
                         SeatunnelErrorEnum.ILLEGAL_STATE, "Unsupported 
JobMode");
             }
         }
-        PageInfo<JobDefinition> jobDefinitionPageInfo =
-                jobDefinitionDao.getJob(searchName, pageNo, pageSize, jobMode);
-        List<Integer> userIds =
-                jobDefinitionPageInfo.getData().stream()
-                        .map(JobDefinition::getCreateUserId)
-                        .collect(Collectors.toList());
-        userIds.addAll(
-                jobDefinitionPageInfo.getData().stream()
-                        .map(JobDefinition::getUpdateUserId)
-                        .collect(Collectors.toList()));
-        List<JobDefinitionRes> jobDefinitionResList = new ArrayList<>();
-        for (int i = 0; i < jobDefinitionPageInfo.getData().size(); i++) {
-            JobDefinition jobDefinition = 
jobDefinitionPageInfo.getData().get(i);
-            JobDefinitionRes jobDefinitionRes = new JobDefinitionRes();
-            jobDefinitionRes.setId(jobDefinition.getId());
-            jobDefinitionRes.setName(jobDefinition.getName());
-            jobDefinitionRes.setDescription(jobDefinition.getDescription());
-            jobDefinitionRes.setJobType(jobDefinition.getJobType());
-            jobDefinitionRes.setCreateUserId(jobDefinition.getCreateUserId());
-            
jobDefinitionRes.setUpdateUserId(jobDefinitionRes.getUpdateUserId());
-            jobDefinitionRes.setCreateTime(jobDefinition.getCreateTime());
-            jobDefinitionRes.setUpdateTime(jobDefinition.getUpdateTime());
-            jobDefinitionResList.add(jobDefinitionRes);
-        }
-        PageInfo<JobDefinitionRes> pageInfo = new PageInfo<>();
-        pageInfo.setPageNo(jobDefinitionPageInfo.getPageNo());
-        pageInfo.setPageSize(jobDefinitionPageInfo.getPageSize());
-        pageInfo.setTotalCount(jobDefinitionPageInfo.getTotalCount());
-        pageInfo.setData(jobDefinitionResList);
-        return pageInfo;
+        return jobDefinitionDao.getJob(searchName, pageNo, pageSize, jobMode);
     }
 
     @Override
diff --git 
a/seatunnel-server/seatunnel-app/src/main/resources/org/apache/seatunnel/app/dal/mapper/JobMapper.xml
 
b/seatunnel-server/seatunnel-app/src/main/resources/org/apache/seatunnel/app/dal/mapper/JobMapper.xml
index 064d5e5c..a8a2b06b 100644
--- 
a/seatunnel-server/seatunnel-app/src/main/resources/org/apache/seatunnel/app/dal/mapper/JobMapper.xml
+++ 
b/seatunnel-server/seatunnel-app/src/main/resources/org/apache/seatunnel/app/dal/mapper/JobMapper.xml
@@ -26,8 +26,10 @@
     <result column="update_time" jdbcType="TIMESTAMP" property="updateTime"/>
   </resultMap>
   <sql id="Base_Column_List">
-    id
-    , `name`, `description`,`job_type`, create_user_id, update_user_id
+    id, `name`, `description`,`job_type`, create_user_id, update_user_id
+  </sql>
+  <sql id="Query_Job_Column_List">
+    t.id, t.`name`, t.`description`, t.`job_type`, t.create_user_id, 
t.update_user_id, t.create_time, t.update_time
   </sql>
   <insert id="insert" 
parameterType="org.apache.seatunnel.app.dal.entity.JobDefinition">
     insert into `t_st_job_definition` (<include refid="Base_Column_List"/>)
@@ -35,28 +37,41 @@
     #{description,jdbcType=VARCHAR},#{jobType,jdbcType=VARCHAR},
     #{createUserId,jdbcType=BIGINT}, #{updateUserId,jdbcType=BIGINT})
   </insert>
-  <select id="queryJobListPaging" 
resultType="org.apache.seatunnel.app.dal.entity.JobDefinition">
+  <select id="queryJobListPaging" 
resultType="org.apache.seatunnel.app.domain.response.job.JobDefinitionRes">
     select
-    <include refid="Base_Column_List"/>, create_time, update_time
-    from t_st_job_definition
+      <include refid="Query_Job_Column_List"/> ,
+      cu.username as create_username,
+      uu.username as update_username
+    from
+      t_st_job_definition t
+      left join user cu on t.create_user_id = cu.id
+      left join user uu on t.update_user_id = uu.id
     <where>
-    <if test="searchName!=null and searchName != ''">
-      and name LIKE concat('%', #{searchName}, '%')
-    </if>
+      <if test="searchName!=null and searchName!= ''">
+        and t.name LIKE concat('%', #{searchName}, '%')
+      </if>
     </where>
-    order by update_time desc
+    order by
+      t.update_time desc
   </select>
 
-  <select id="queryJobListPagingWithJobMode" 
resultType="org.apache.seatunnel.app.dal.entity.JobDefinition">
-    select t.*
-    from t_st_job_definition t
-    join (select * from t_st_job_version where job_mode = #{jobMode}) v on 
t.id = v.job_id
+  <select id="queryJobListPagingWithJobMode" 
resultType="org.apache.seatunnel.app.domain.response.job.JobDefinitionRes">
+    select
+      <include refid="Query_Job_Column_List"/> ,
+      cu.username as create_username,
+      uu.username as update_username
+    from
+      t_st_job_definition t
+      join (select * from t_st_job_version where job_mode = #{jobMode}) v on 
t.id = v.job_id
+      left join user cu on t.create_user_id = cu.id
+      left join user uu on t.update_user_id = uu.id
     <where>
-    <if test="searchName!=null and searchName != ''">
-      and t.name LIKE concat('%', #{searchName}, '%')
-    </if>
+      <if test="searchName!=null and searchName != ''">
+        and t.name LIKE concat('%', #{searchName}, '%')
+      </if>
     </where>
-    order by update_time desc
+    order by
+      update_time desc
   </select>
 
   <select id="queryJobList" 
resultType="org.apache.seatunnel.app.dal.entity.JobDefinition">

Reply via email to