[GitHub] metron pull request #1135: METRON-1700: Create REST endpoint to get job conf...

[nickwallen]

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

    https://github.com/apache/metron/pull/1135#discussion_r206609288
  
    --- Diff: 
metron-interface/metron-rest/src/main/java/org/apache/metron/rest/service/PcapService.java
 ---
    @@ -43,5 +44,5 @@
     
       InputStream getRawPcap(String username, String jobId, Integer page) 
throws RestException;
     
    -
    +  Map<String, Object> getConfiguration(String username, String jobId) 
throws RestException;
    --- End diff --
    
    Would be nice to see some javadocs here, but that doesn't seem unique to 
this PR.  Maybe in another PR we can get some javadocs in here.


---