diff --git a/src/main/java/com/nis/domain/configuration/BaseStringCfg.java b/src/main/java/com/nis/domain/configuration/BaseStringCfg.java
index 56322a1a2..c6cb594a7 100644
--- a/src/main/java/com/nis/domain/configuration/BaseStringCfg.java
+++ b/src/main/java/com/nis/domain/configuration/BaseStringCfg.java
@@ -45,7 +45,7 @@ public class BaseStringCfg extends BaseCfg<BaseStringCfg> {
 	/**
 	 * 编译id
 	 */
-	protected Integer compile_id ;
+	protected Long compileId ;
 	/**
 	 * 表达式类型
 	 */
@@ -109,5 +109,19 @@ public class BaseStringCfg extends BaseCfg<BaseStringCfg> {
 		super.initDefaultValue();
 		this.isHexbin = 0;
 	}
+	/**
+	 * compile_id
+	 * @return  compile_id
+	*/
+	
+	public Long getCompileId() {
+		return compileId;
+	}
+	/**
+	 * @param compile_id the compile_id to set
+	 */
+	public void setCompileId(Long compileId) {
+		this.compileId = compileId;
+	}
 	
 }
diff --git a/src/main/java/com/nis/web/dao/configuration/IpCfgDao.xml b/src/main/java/com/nis/web/dao/configuration/IpCfgDao.xml
index d2cd15561..08dfd203f 100644
--- a/src/main/java/com/nis/web/dao/configuration/IpCfgDao.xml
+++ b/src/main/java/com/nis/web/dao/configuration/IpCfgDao.xml
@@ -399,96 +399,98 @@
   <update id="updateByPrimaryKeySelective" parameterType="com.nis.domain.configuration.BaseIpCfg" >
     update ${tableName}
     <set >
-      <if test="cfgDesc != null" >
-        cfg_desc = #{cfgDesc,jdbcType=VARCHAR},
-      </if>
-      <if test="ipType != null" >
-        ip_type = #{ipType,jdbcType=INTEGER},
-      </if>
-      <if test="srcIp != null" >
-        src_ip = #{srcIp,jdbcType=VARCHAR},
-      </if>
-      <if test="srcIpMask != null" >
-        src_ip_mask = #{srcIpMask,jdbcType=VARCHAR},
-      </if>
-      <if test="srcPort != null" >
-        src_port = #{srcPort,jdbcType=VARCHAR},
-      </if>
-      <if test="srcPortMask != null" >
-        src_port_mask = #{srcPortMask,jdbcType=VARCHAR},
-      </if>
-      <if test="dstIp != null" >
-        dst_ip = #{dstIp,jdbcType=VARCHAR},
-      </if>
-      <if test="dstIpMask != null" >
-        dst_ip_mask = #{dstIpMask,jdbcType=VARCHAR},
-      </if>
-      <if test="dstPort != null" >
-        dst_port = #{dstPort,jdbcType=VARCHAR},
-      </if>
-      <if test="srcPortMask != null" >
-        dst_port_mask = #{srcPortMask,jdbcType=VARCHAR},
-      </if>
-      <if test="direction != null" >
-        direction = #{direction,jdbcType=INTEGER},
-      </if>
-      <if test="protocol != null" >
-        protocol = #{protocol,jdbcType=INTEGER},
-      </if>
-      <if test="protocolId != null" >
-        protocol_id = #{protocolId,jdbcType=INTEGER},
-      </if>
-      <if test="action != null" >
-        action = #{action,jdbcType=INTEGER},
-      </if>
-      <if test="isValid != null" >
-        is_valid = #{isValid,jdbcType=INTEGER},
-      </if>
-      <if test="isAudit != null" >
-        is_audit = #{isAudit,jdbcType=INTEGER},
-      </if>
-      <if test="creatorId != null" >
-        creator_id = #{creatorId,jdbcType=INTEGER},
-      </if>
-      <if test="createTime != null" >
-        create_time = #{createTime,jdbcType=TIMESTAMP},
-      </if>
-      <if test="editorId != null" >
-        editor_id = #{editorId,jdbcType=INTEGER},
-      </if>
-      <if test="editTime != null" >
-        edit_time = #{editTime,jdbcType=TIMESTAMP},
-      </if>
-      <if test="auditorId != null" >
-        auditor_id = #{auditorId,jdbcType=INTEGER},
-      </if>
-      <if test="auditTime != null" >
-        audit_time = #{auditTime,jdbcType=TIMESTAMP},
-      </if>
-      <if test="serviceId != null" >
-        service_id = #{serviceId,jdbcType=INTEGER},
-      </if>
-      <if test="requestId != null" >
-        request_id = #{requestId,jdbcType=INTEGER},
-      </if>
-      <if test="compileId != null" >
-        compile_id = #{compileId,jdbcType=INTEGER},
-      </if>
-      <if test="isAreaEffective != null" >
-        is_area_effective = #{isAreaEffective,jdbcType=INTEGER},
-      </if>
-      <if test="classify != null" >
-        classify = #{classify,jdbcType=VARCHAR},
-      </if>
-      <if test="attribute != null" >
-        attribute = #{attribute,jdbcType=VARCHAR},
-      </if>
-      <if test="lable != null" >
-        lable = #{lable,jdbcType=VARCHAR},
-      </if>
-      <if test="areaEffectiveIds != null" >
-        area_effective_ids =  #{areaEffectiveIds,jdbcType=VARCHAR}
-      </if>
+   	  <trim  suffixOverrides=",">
+	      <if test="cfgDesc != null" >
+	        cfg_desc = #{cfgDesc,jdbcType=VARCHAR},
+	      </if>
+	      <if test="ipType != null" >
+	        ip_type = #{ipType,jdbcType=INTEGER},
+	      </if>
+	      <if test="srcIp != null" >
+	        src_ip = #{srcIp,jdbcType=VARCHAR},
+	      </if>
+	      <if test="srcIpMask != null" >
+	        src_ip_mask = #{srcIpMask,jdbcType=VARCHAR},
+	      </if>
+	      <if test="srcPort != null" >
+	        src_port = #{srcPort,jdbcType=VARCHAR},
+	      </if>
+	      <if test="srcPortMask != null" >
+	        src_port_mask = #{srcPortMask,jdbcType=VARCHAR},
+	      </if>
+	      <if test="dstIp != null" >
+	        dst_ip = #{dstIp,jdbcType=VARCHAR},
+	      </if>
+	      <if test="dstIpMask != null" >
+	        dst_ip_mask = #{dstIpMask,jdbcType=VARCHAR},
+	      </if>
+	      <if test="dstPort != null" >
+	        dst_port = #{dstPort,jdbcType=VARCHAR},
+	      </if>
+	      <if test="srcPortMask != null" >
+	        dst_port_mask = #{srcPortMask,jdbcType=VARCHAR},
+	      </if>
+	      <if test="direction != null" >
+	        direction = #{direction,jdbcType=INTEGER},
+	      </if>
+	      <if test="protocol != null" >
+	        protocol = #{protocol,jdbcType=INTEGER},
+	      </if>
+	      <if test="protocolId != null" >
+	        protocol_id = #{protocolId,jdbcType=INTEGER},
+	      </if>
+	      <if test="action != null" >
+	        action = #{action,jdbcType=INTEGER},
+	      </if>
+	      <if test="isValid != null" >
+	        is_valid = #{isValid,jdbcType=INTEGER},
+	      </if>
+	      <if test="isAudit != null" >
+	        is_audit = #{isAudit,jdbcType=INTEGER},
+	      </if>
+	      <if test="creatorId != null" >
+	        creator_id = #{creatorId,jdbcType=INTEGER},
+	      </if>
+	      <if test="createTime != null" >
+	        create_time = #{createTime,jdbcType=TIMESTAMP},
+	      </if>
+	      <if test="editorId != null" >
+	        editor_id = #{editorId,jdbcType=INTEGER},
+	      </if>
+	      <if test="editTime != null" >
+	        edit_time = #{editTime,jdbcType=TIMESTAMP},
+	      </if>
+	      <if test="auditorId != null" >
+	        auditor_id = #{auditorId,jdbcType=INTEGER},
+	      </if>
+	      <if test="auditTime != null" >
+	        audit_time = #{auditTime,jdbcType=TIMESTAMP},
+	      </if>
+	      <if test="serviceId != null" >
+	        service_id = #{serviceId,jdbcType=INTEGER},
+	      </if>
+	      <if test="requestId != null" >
+	        request_id = #{requestId,jdbcType=INTEGER},
+	      </if>
+	      <if test="compileId != null" >
+	        compile_id = #{compileId,jdbcType=INTEGER},
+	      </if>
+	      <if test="isAreaEffective != null" >
+	        is_area_effective = #{isAreaEffective,jdbcType=INTEGER},
+	      </if>
+	      <if test="classify != null" >
+	        classify = #{classify,jdbcType=VARCHAR},
+	      </if>
+	      <if test="attribute != null" >
+	        attribute = #{attribute,jdbcType=VARCHAR},
+	      </if>
+	      <if test="lable != null" >
+	        lable = #{lable,jdbcType=VARCHAR},
+	      </if>
+	      <if test="areaEffectiveIds != null" >
+	        area_effective_ids =  #{areaEffectiveIds,jdbcType=VARCHAR}
+	      </if>
+      </trim>
     </set>
     where cfg_id = #{cfgId,jdbcType=BIGINT}
   </update>