fix: 2.4.0
1. 修复新增组时描述为空新增异常问题 2. 修复定时任务参数为空时新增报错问题
This commit is contained in:
parent
5bcb156934
commit
a9a317b367
@ -8,7 +8,7 @@ CREATE TABLE `group_config`
|
|||||||
(
|
(
|
||||||
`id` bigint(20) unsigned NOT NULL AUTO_INCREMENT COMMENT '主键',
|
`id` bigint(20) unsigned NOT NULL AUTO_INCREMENT COMMENT '主键',
|
||||||
`group_name` varchar(64) NOT NULL DEFAULT '' COMMENT '组名称',
|
`group_name` varchar(64) NOT NULL DEFAULT '' COMMENT '组名称',
|
||||||
`description` varchar(256) NOT NULL COMMENT '组描述',
|
`description` varchar(256) NOT NULL DEFAULT '' COMMENT '组描述',
|
||||||
`group_status` tinyint(4) NOT NULL DEFAULT '0' COMMENT '组状态 0、未启用 1、启用',
|
`group_status` tinyint(4) NOT NULL DEFAULT '0' COMMENT '组状态 0、未启用 1、启用',
|
||||||
`version` int(11) NOT NULL COMMENT '版本号',
|
`version` int(11) NOT NULL COMMENT '版本号',
|
||||||
`group_partition` int(11) NOT NULL COMMENT '分区',
|
`group_partition` int(11) NOT NULL COMMENT '分区',
|
||||||
@ -220,7 +220,7 @@ CREATE TABLE `job` (
|
|||||||
`id` bigint(20) unsigned NOT NULL AUTO_INCREMENT COMMENT '主键',
|
`id` bigint(20) unsigned NOT NULL AUTO_INCREMENT COMMENT '主键',
|
||||||
`group_name` varchar(64) NOT NULL COMMENT '组名称',
|
`group_name` varchar(64) NOT NULL COMMENT '组名称',
|
||||||
`job_name` varchar(64) NOT NULL COMMENT '名称',
|
`job_name` varchar(64) NOT NULL COMMENT '名称',
|
||||||
`args_str` text NOT NULL COMMENT '执行方法参数',
|
`args_str` text DEFAULT NULL COMMENT '执行方法参数',
|
||||||
`args_type` tinyint(4) NOT NULL DEFAULT '1' COMMENT '参数类型 ',
|
`args_type` tinyint(4) NOT NULL DEFAULT '1' COMMENT '参数类型 ',
|
||||||
`next_trigger_at` datetime NOT NULL COMMENT '下次触发时间',
|
`next_trigger_at` datetime NOT NULL COMMENT '下次触发时间',
|
||||||
`job_status` tinyint(4) NOT NULL DEFAULT '1' COMMENT '重试状态 0、关闭、1、开启',
|
`job_status` tinyint(4) NOT NULL DEFAULT '1' COMMENT '重试状态 0、关闭、1、开启',
|
||||||
|
@ -97,14 +97,15 @@ public class GroupConfigServiceImpl implements GroupConfigService {
|
|||||||
public Boolean updateGroup(GroupConfigRequestVO groupConfigRequestVO) {
|
public Boolean updateGroup(GroupConfigRequestVO groupConfigRequestVO) {
|
||||||
|
|
||||||
ConfigAccess<GroupConfig> groupConfigAccess = accessTemplate.getGroupConfigAccess();
|
ConfigAccess<GroupConfig> groupConfigAccess = accessTemplate.getGroupConfigAccess();
|
||||||
GroupConfig groupConfig = groupConfigAccess.one(
|
long count = groupConfigAccess.count(
|
||||||
new LambdaQueryWrapper<GroupConfig>().eq(GroupConfig::getGroupName, groupConfigRequestVO.getGroupName()));
|
new LambdaQueryWrapper<GroupConfig>().eq(GroupConfig::getGroupName, groupConfigRequestVO.getGroupName()));
|
||||||
if (Objects.isNull(groupConfig)) {
|
if (count <= 0) {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
GroupConfig groupConfig = GroupConfigConverter.INSTANCE.convert(groupConfigRequestVO);
|
||||||
groupConfig.setVersion(groupConfig.getVersion() + 1);
|
groupConfig.setVersion(groupConfig.getVersion() + 1);
|
||||||
BeanUtils.copyProperties(groupConfigRequestVO, groupConfig);
|
groupConfig.setDescription(Optional.ofNullable(groupConfigRequestVO.getDescription()).orElse(StrUtil.EMPTY));
|
||||||
|
|
||||||
Assert.isTrue(systemProperties.getTotalPartition() > groupConfigRequestVO.getGroupPartition(), () -> new EasyRetryServerException("分区超过最大分区. [{}]", systemProperties.getTotalPartition() - 1));
|
Assert.isTrue(systemProperties.getTotalPartition() > groupConfigRequestVO.getGroupPartition(), () -> new EasyRetryServerException("分区超过最大分区. [{}]", systemProperties.getTotalPartition() - 1));
|
||||||
Assert.isTrue(groupConfigRequestVO.getGroupPartition() >= 0, () -> new EasyRetryServerException("分区不能是负数."));
|
Assert.isTrue(groupConfigRequestVO.getGroupPartition() >= 0, () -> new EasyRetryServerException("分区不能是负数."));
|
||||||
@ -176,6 +177,7 @@ public class GroupConfigServiceImpl implements GroupConfigService {
|
|||||||
groupConfig.setCreateDt(LocalDateTime.now());
|
groupConfig.setCreateDt(LocalDateTime.now());
|
||||||
groupConfig.setVersion(1);
|
groupConfig.setVersion(1);
|
||||||
groupConfig.setGroupName(groupConfigRequestVO.getGroupName());
|
groupConfig.setGroupName(groupConfigRequestVO.getGroupName());
|
||||||
|
groupConfig.setDescription(Optional.ofNullable(groupConfigRequestVO.getDescription()).orElse(StrUtil.EMPTY));
|
||||||
if (Objects.isNull(groupConfigRequestVO.getGroupPartition())) {
|
if (Objects.isNull(groupConfigRequestVO.getGroupPartition())) {
|
||||||
groupConfig.setGroupPartition(HashUtil.bkdrHash(groupConfigRequestVO.getGroupName()) % systemProperties.getTotalPartition());
|
groupConfig.setGroupPartition(HashUtil.bkdrHash(groupConfigRequestVO.getGroupName()) % systemProperties.getTotalPartition());
|
||||||
groupConfig.setBucketIndex(HashUtil.bkdrHash(groupConfigRequestVO.getGroupName()) % systemProperties.getBucketTotal());
|
groupConfig.setBucketIndex(HashUtil.bkdrHash(groupConfigRequestVO.getGroupName()) % systemProperties.getBucketTotal());
|
||||||
|
Loading…
Reference in New Issue
Block a user