Holding the timeThreshold in a private variable and changing its type to long would be better, reducing the cost of unboxing and hash searching.

thanks ， but i dont know “Holding the timeThreshold in a private variable ” means what？ thanks again

Just like this:

public class AccessLogFilter implements ContainerResponseFilter {
     private long timeThreshold =configProvider.get().get(ServerOptions.SLOW_QUERY_LOG_TIME_THRESHOLD);

.....
}

Of course, you need to check for null.

Just like this:

public class AccessLogFilter implements ContainerResponseFilter {
     private long timeThreshold =configProvider.get().get(ServerOptions.SLOW_QUERY_LOG_TIME_THRESHOLD);

.....
}

Of course, you need to check for null.

we cant get timeThreshold as private var from configProvider, because configProvider managed by @context; it will case NPE.