metron-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From nickwallen <...@git.apache.org>
Subject [GitHub] metron pull request #731: METRON-1146 Refactor for ParserExtensionConfig and...
Date Wed, 06 Sep 2017 16:29:15 GMT
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)


---

Mime
View raw message