>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