记一次调用service实现类Synchronized同步锁失效的坑

我的同步方法是这样写的

@Override
@Transactional(rollbackFor = Exception.class)
public synchronized SyTaskChildDO initTaskChild(SyTaskChildQueryParam param) {
    try {
        List<SyTaskChildDO> taskCidList = syTaskChildDAO.listByParam(param);
        if(taskCidList.isEmpty()){
            return null;
        }
        SyTaskChildDO childDO = taskCidList.get(0);
        childDO.setState(TaskStateEnum.IN_EXECUTE.getCode());
        //修改任务状态为执行中
        Long childNum = syTaskChildDAO.update(childDO);
        if(childNum<1){
            return null;
        }
        return childDO;
    } catch (Exception e) {
        LOGGER.error("SyTaskChildManageServiceImpl initTaskChild Exception",e);
        throw e;
    }
}

在同步方法上面添加了事务，这样会导致同步方法有时候会失效。

原因在于事务的范围比synchronized范围大，当锁释放了才会提交事务，在这期间就有可能别的线程读取到任务去执行