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

    https://github.com/apache/metron/pull/970#discussion_r181186669
  
    --- Diff: 
metron-platform/metron-solr/src/main/java/org/apache/metron/solr/dao/SolrSearchDao.java
 ---
    @@ -238,19 +204,19 @@ protected SearchResponse buildSearchResponse(
         return searchResponse;
       }
     
    -  protected SearchResult getSearchResult(SolrDocument solrDocument, 
Optional<List<String>> fields) {
    -    SearchResult searchResult = new SearchResult();
    -    searchResult.setId((String) 
solrDocument.getFieldValue(Constants.GUID));
    -    Map<String, Object> source;
    -    if (fields.isPresent()) {
    -      source = new HashMap<>();
    -      fields.get().forEach(field -> source.put(field, 
solrDocument.getFieldValue(field)));
    -    } else {
    -      source = solrDocument.getFieldValueMap();
    -    }
    -    searchResult.setSource(source);
    -    return searchResult;
    -  }
    +//  protected SearchResult getSearchResult(SolrDocument solrDocument, 
Optional<List<String>> fields) {
    --- End diff --
    
    My bad.  The merge from the feature branch was messy and I didn't clean up 
after myself.


---

Reply via email to