[ 
https://issues.apache.org/jira/browse/METRON-1700?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16564017#comment-16564017
 ] 

ASF GitHub Bot commented on METRON-1700:
----------------------------------------

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

    https://github.com/apache/metron/pull/1135#discussion_r206607449
  
    --- Diff: 
metron-interface/metron-rest/src/main/java/org/apache/metron/rest/service/impl/PcapServiceImpl.java
 ---
    @@ -199,6 +208,37 @@ public InputStream getRawPcap(String username, String 
jobId, Integer page) throw
         return inputStream;
       }
     
    +  @Override
    +  public Map<String, Object> getConfiguration(String username, String 
jobId) throws RestException {
    +    Map<String, Object> configuration = new HashMap<>();
    +    try {
    +      Map<String, Object> jobConfiguration = jobManager.getJob(username, 
jobId).getConfiguration();
    --- End diff --
    
    Will `jobManager.getJob(username, jobId)` never return null?  I see null 
checks in other places where similar calls are made.


> Create REST endpoint to get job configuration
> ---------------------------------------------
>
>                 Key: METRON-1700
>                 URL: https://issues.apache.org/jira/browse/METRON-1700
>             Project: Metron
>          Issue Type: Sub-task
>            Reporter: Ryan Merriman
>            Priority: Major
>
> We need a REST endpoint that will allow us to retrieve the job configuration 
> properties.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to