midnight2104 commented on a change in pull request #3094:
URL: https://github.com/apache/incubator-shenyu/pull/3094#discussion_r834207212



##########
File path: shenyu-admin/src/main/resources/mappers/shenyu-dict-sqlmap.xml
##########
@@ -32,65 +32,89 @@
     </resultMap>
 
     <sql id="Base_Column_List">
-        id, `type`, dict_code, dict_name, dict_value, `desc`, sort, enabled, 
date_created, date_updated
+        id,
+        `type`,
+        dict_code,
+        dict_name,
+        dict_value,
+        `desc`,
+        sort,
+        enabled,
+        date_created,
+        date_updated
     </sql>
 
     <select id="selectById" parameterType="java.lang.String" 
resultMap="BaseResultMap">
-        select
-        <include refid="Base_Column_List"/>
-        from shenyu_dict
-        where id = #{id,jdbcType=VARCHAR}
+        SELECT
+              <include refid="Base_Column_List"/>
+         FROM shenyu_dict
+        WHERE id = #{id, jdbcType=VARCHAR}
     </select>
 
     <select id="selectByDictCodeAndDictName" resultMap="BaseResultMap">
-        select
-        <include refid="Base_Column_List"/>
-        from shenyu_dict
-        where dict_code = #{dictCode,jdbcType=VARCHAR} and dict_name = 
#{dictName,jdbcType=VARCHAR}
+        SELECT
+              <include refid="Base_Column_List"/>
+         FROM shenyu_dict
+        WHERE dict_code = #{dictCode, jdbcType=VARCHAR}
+          AND dict_name = #{dictName, jdbcType=VARCHAR}
     </select>
 
     <select id="findByType" parameterType="java.lang.String" 
resultMap="BaseResultMap">
-        select
-        <include refid="Base_Column_List"/>
-        from shenyu_dict where `type` = #{type,jdbcType=VARCHAR}
+        SELECT
+              <include refid="Base_Column_List"/>
+         FROM shenyu_dict
+        WHERE `type` = #{type, jdbcType=VARCHAR}
     </select>
 
     <select id="findByTypeBatch" resultMap="BaseResultMap">
-        select
-        <include refid="Base_Column_List"/>
-        from shenyu_dict where `type` in
-        <foreach collection="typeSet" item="type"
-                 open="(" separator="," close=")">
-            #{type,jdbcType=VARCHAR}
+        SELECT
+              <include refid="Base_Column_List"/>
+         FROM shenyu_dict
+        WHERE `type` IN
+        <foreach collection="typeSet" item="type" open="(" separator="," 
close=")">
+              #{type,jdbcType=VARCHAR}
         </foreach>
     </select>
 
     <delete id="delete" parameterType="java.lang.String">
-        delete from shenyu_dict
-        where id = #{id,jdbcType=VARCHAR}
+        DELETE FROM shenyu_dict
+        WHERE id = #{id,jdbcType=VARCHAR}
     </delete>
 
     <delete id="deleteByIdList">
-        delete from shenyu_dict
-        where id in
-        <foreach collection="idList" item="id" index="index"
-                 open="(" separator="," close=")">
+        DELETE FROM shenyu_dict
+        WHERE id IN
+        <foreach collection="idList" item="id" index="index" open="(" 
separator="," close=")">
             #{id,jdbcType=VARCHAR}
         </foreach>
     </delete>
 
     <insert id="insert" 
parameterType="org.apache.shenyu.admin.model.entity.ShenyuDictDO">
-        insert into shenyu_dict (id, `type`, dict_code, dict_name, dict_value, 
`desc`, sort, enabled, date_created,
-        date_updated
-        )
-        values (#{id,jdbcType=VARCHAR}, #{type,jdbcType=VARCHAR}, 
#{dictCode,jdbcType=VARCHAR},
-        #{dictName,jdbcType=VARCHAR}, #{dictValue,jdbcType=VARCHAR}, 
#{desc,jdbcType=VARCHAR}, #{sort,jdbcType=INTEGER},
-        #{enabled,jdbcType=TINYINT}, #{dateCreated,jdbcType=TIMESTAMP}, 
#{dateUpdated,jdbcType=TIMESTAMP}
-        )
+        INSERT INTO shenyu_dict
+                    (id,
+                    `type`,
+                    dict_code,
+                    dict_name,
+                    dict_value,
+                    `desc`,
+                    sort,
+                    enabled,
+                    date_created,
+                    date_updated)
+             VALUES (#{id,jdbcType=VARCHAR},
+                    #{type,jdbcType=VARCHAR},
+                    #{dictCode,jdbcType=VARCHAR},
+                    #{dictName,jbcType=VARCHAR},

Review comment:
       pls modify `jbcType` ->  `jdbcType`




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to