ThreadLocal和@Aspect注释

时间:2013-05-29 15:08:45

标签: java spring spring-aop

我正在使用@Aspect来实现数据库陈旧连接问题的重试逻辑(max_retries = 5)。在这个建议中,我有一个ThreadLocal对象,它跟踪逻辑重试多少次以获得连接,并且每次都会增加它无法获得连接,以避免过时连接问题的无限重试,最大重试次数为5(常量)。

但我遇到的问题是,在这个@Aspect java类中,ThreadLocal永远不会增加,这会导致代码中的endlees循环,当然不应该在最大重试次数后重试,但是永远不会达到那个数量而不是打破了循环。

如果有任何人遇到@Aspect和ThreadLcal对象的问题,请告诉我。

我很乐意分享这些代码。

private static ThreadLocal<Integer> retryCounter= new ThreadLocal<Integer>() {};
private static final String STALE_CONNECTION_EXCEPTION  = "com.ibm.websphere.ce.cm.StaleConnectionException";

@Around("service") 
public Object retryConnection(ProceedingJoinPoint pjp) throws Throwable {
        if (staleConnectionException == null) {
        return pjp.proceed();
    }
    Throwable exception = null;
    retryCounter.set(new Integer(0));
    while ( retryCounter.get() < MAX_TRIES) {
        try {
            return pjp.proceed();
        }
        catch (AppDataException he) {
            exception = retry(he.getCause());
        }
        catch (NestedRuntimeException e) {
            exception = retry(e);
        }
    }
    if (exception != null) {
        Logs.error("Stale connection exception occurred, no more   retries left", this.getClass(), null);
        logException(pjp, exception);
        throw new AppDataException(exception);
    }
    return null;
}

private Throwable retry(Throwable e) throws Throwable {
    if (e instanceof NestedRuntimeException && ((NestedRuntimeException)e).contains(staleConnectionException)) {
        retryCounter.set(retryCounter.get()+1);
        LogUtils.log("Stale connection exception occurred, retrying " +  retryCounter.get() + " of " + MAX_TRIES, this.getClass());
        return e;
    }
    else {
        throw e;
    }
}

2 个答案:

答案 0 :(得分:1)

正如评论中所提到的,不确定为什么你使用本地线程...但鉴于你是,可能导致无限循环的是递归使用这个方面。通过调试器运行它或对其进行配置,以查看您是否以嵌套方式使用相同的方面。

答案 1 :(得分:0)

老实说,看看你的代码,我认为你最好不要这样做,而只是在连接池中配置连接测试(假设你使用的是):http://pic.dhe.ibm.com/infocenter/wasinfo/v6r1/index.jsp?topic=/com.ibm.websphere.nd.multiplatform.doc/info/ae/ae/tdat_pretestconn.html