Github user nickwallen commented on a diff in the pull request:

    https://github.com/apache/metron/pull/731#discussion_r137320162
  
    --- Diff: 
metron-interface/metron-rest/src/main/java/org/apache/metron/rest/controller/ParserExtensionController.java
 ---
    @@ -90,8 +91,8 @@
       @ApiOperation(value = "Retrieves all ParserExtensionConfigs from 
Zookeeper")
       @ApiResponse(message = "Returns all ParserExtensionConfigs", code = 200)
       @RequestMapping(method = RequestMethod.GET)
    -  ResponseEntity<Map<String, ParserExtensionConfig>> findAll() throws 
RestException {
    -    return new ResponseEntity<Map<String, 
ParserExtensionConfig>>(extensionService.getAllParserExtensions(), 
HttpStatus.OK);
    +  ResponseEntity<List<ParserExtensionConfig>> findAll() throws 
RestException {
    --- End diff --
    
    Makes sense.  Its cleaner this way.  (Even if the real reason is that you 
aren't good enough with Angular.  Ha)


---

Reply via email to