>From Ali Alsuliman <[email protected]>:

Ali Alsuliman has uploaded this change for review. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18306 )


Change subject: [ASTERIXDB-3343][API] Make completed requests map synchronized
......................................................................

[ASTERIXDB-3343][API] Make completed requests map synchronized

- user model changes: no
- storage format changes: no
- interface changes: no

Details:
Completed requests collection has been changed to LinkedHashMap
recently. It needs to be a synchronized map.

Change-Id: Ic5a8c12e3feaddeea6882c34fba3c12dc22452c3
---
M 
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/utils/RequestTracker.java
1 file changed, 19 insertions(+), 2 deletions(-)



  git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb 
refs/changes/06/18306/1

diff --git 
a/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/utils/RequestTracker.java
 
b/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/utils/RequestTracker.java
index 9875651..99f9546 100644
--- 
a/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/utils/RequestTracker.java
+++ 
b/asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/utils/RequestTracker.java
@@ -49,14 +49,14 @@
     public RequestTracker(ICcApplicationContext ccAppCtx) {
         this.ccAppCtx = ccAppCtx;
         int archiveSize = 
ccAppCtx.getExternalProperties().getRequestsArchiveSize();
-        completedRequests = new LinkedHashMap<>(archiveSize) {
+        completedRequests = Collections.synchronizedMap(new 
LinkedHashMap<>(archiveSize) {
             private static final long serialVersionUID = 1L;

             @Override
             protected boolean removeEldestEntry(Map.Entry<String, 
IClientRequest> eldest) {
                 return size() > archiveSize;
             }
-        };
+        });
         numRequests = new AtomicLong(0);
         numOfFailedRequests = new AtomicLong(0);
     }

--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18306
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings

Gerrit-Project: asterixdb
Gerrit-Branch: trinity
Gerrit-Change-Id: Ic5a8c12e3feaddeea6882c34fba3c12dc22452c3
Gerrit-Change-Number: 18306
Gerrit-PatchSet: 1
Gerrit-Owner: Ali Alsuliman <[email protected]>
Gerrit-MessageType: newchange

Reply via email to