Remove incorrect per-request-type synchronized and layer of indirection associated with it
diff --git a/src/main/java/org/apache/datasketches/server/BaseSketchesQueryHandler.java b/src/main/java/org/apache/datasketches/server/BaseSketchesQueryHandler.java
index 651c527..211733a 100644
--- a/src/main/java/org/apache/datasketches/server/BaseSketchesQueryHandler.java
+++ b/src/main/java/org/apache/datasketches/server/BaseSketchesQueryHandler.java
@@ -103,15 +103,6 @@
*/
protected abstract JsonObject processQuery(JsonObject query);
- /**
- * Internal method to synchronize calls to subclasses
- * @param query A JSON query to process
- * @return A JSON response
- */
- final JsonObject callProcessQuery(final JsonObject query) {
- return processQuery(query);
- }
-
@Override
public void handle(final String target,
final Request baseRequest,
@@ -130,10 +121,10 @@
try {
if (query == null) {
- result = callProcessQuery(null);
+ result = processQuery(null);
} else if (query.isJsonArray()) {
for (final JsonElement subQuery : query.getAsJsonArray()) {
- final JsonObject subResult = callProcessQuery(subQuery.getAsJsonObject());
+ final JsonObject subResult = processQuery(subQuery.getAsJsonObject());
if (subResult != null) {
// lazy initialization to avoid possibly empty array
if (result == null) {
@@ -143,7 +134,8 @@
}
}
} else {
- result = callProcessQuery((JsonObject) query);
+ //result = callProcessQuery((JsonObject) query);
+ result = processQuery((JsonObject) query);
}
if (result != null) {