mneethiraj commented on code in PR #339:
URL: https://github.com/apache/atlas/pull/339#discussion_r2063967183
##########
server-api/src/main/java/org/apache/atlas/RequestContext.java:
##########
@@ -44,8 +44,9 @@
public class RequestContext {
private static final Logger METRICS = LoggerFactory.getLogger("METRICS");
- private static final ThreadLocal<RequestContext> CURRENT_CONTEXT = new
ThreadLocal<>();
- private static final Set<RequestContext> ACTIVE_REQUESTS = new
HashSet<>();
+ private static final ThreadLocal<RequestContext> CURRENT_CONTEXT
= new ThreadLocal<>();
+ private static final Set<RequestContext> ACTIVE_REQUESTS
= new HashSet<>();
+ private static final ThreadLocal<Boolean> isUpdateNotification
= new ThreadLocal<>();
Review Comment:
Instead of having `isUpdateNotification` as a `static ThreadLocal<>`, have
it as a instance member of `RequestContext` (similar to other members like
`requestTime`). This will also make it unnecessary to reset the flag in
NotificationHookConsumer.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@atlas.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
