kekingcn/spring-boot-klock-starter

很优秀的开源分布式锁，请问还维护吗

einsitang opened this issue · 7 comments

einsitang commented

hello 作者，
我们项目需要用到分布式锁，觉得你这个项目非常棒，但很多依赖库都非常旧了，我们是基于Spring-boot 2.1.8 开发的应用，各种不兼容，需要拉去代码升级里面的spring库重新编译才可以使用。请问现在还维护吗，我给你提一个PR然后重新发布一个高版本的包

klboke commented

可以提个 pr 过来，我们看看兼容性

einsitang commented

不好意思，最近项目比较急，迟点修正一下然后提PR给你们

z5614036 commented

等你的pr了，老哥

tangshd commented
einsitang commented

不好意思，忙着项目给忘了

chriswuchn commented

你好，作者。看了你们的项目源码，非常棒，有一个小疑问，想请教一下。
`public Object around(ProceedingJoinPoint joinPoint, Klock klock) throws Throwable {
LockInfo lockInfo = this.lockInfoProvider.get(joinPoint, klock);
String curentLock = this.getCurrentLockId(joinPoint, klock);
this.currentThreadLock.put(curentLock, new KlockAspectHandler.LockRes(lockInfo, false));
Lock lock = this.lockFactory.getLock(lockInfo);
boolean lockRes = lock.acquire();
if (!lockRes) {
if (logger.isWarnEnabled()) {
logger.warn("Timeout while acquiring Lock({})", lockInfo.getName());
}
if (!StringUtils.isEmpty(klock.customLockTimeoutStrategy())) {
return this.handleCustomLockTimeout(klock.customLockTimeoutStrategy(), joinPoint);
}
//执行失败默认策略
klock.lockTimeoutStrategy().handle(lockInfo, lock, joinPoint);
}

((KlockAspectHandler.LockRes)this.currentThreadLock.get(curentLock)).setLock(lock);    //(1)
((KlockAspectHandler.LockRes)this.currentThreadLock.get(curentLock)).setRes(true);    //(2)
return joinPoint.proceed();  //(3)

}`

当获取锁失败时候，执行默认策略，默认策略如下：
NO_OPERATION { public void handle(LockInfo lockInfo, Lock lock, JoinPoint joinPoint) { } }
如果是这种情况的话，(1),(2),(3)语句还是执行，会不会有问题？是不是应该直接返回？

klboke commented

@chriswuchn 不是很清楚你的问题，你贴的代码可以在格式化下么