Github user simonellistonball commented on a diff in the pull request:
https://github.com/apache/metron/pull/662#discussion_r129241629
--- Diff:
metron-interface/metron-rest/src/main/java/org/apache/metron/rest/controller/SearchController.java
---
@@ -45,4 +49,18 @@
ResponseEntity<SearchResponse> search(final @ApiParam(name =
"searchRequest", value = "Search request", required = true) @RequestBody
SearchRequest searchRequest) throws RestException {
return new ResponseEntity<>(searchService.search(searchRequest),
HttpStatus.OK);
}
+
--- End diff --
Feels like any asynchronicity should be at the service layer anyway for
other things to potentially benefit from, of course that would bubble up here,
but do we really expect the REST server to have to worry about high load?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
