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

    https://github.com/apache/metron/pull/832#discussion_r149471276
  
    --- Diff: 
metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/utils/ElasticsearchUtils.java
 ---
    @@ -179,4 +186,46 @@ else if(ipObj instanceof List) {
         }
         throw new IllegalStateException("Unable to read the elasticsearch ips, 
expected es.ip to be either a list of strings, a string hostname or a host:port 
string");
       }
    +
    +  /**
    +   * Converts an Elasticsearch SearchRequest to JSON.
    +   * @param esRequest The search request.
    +   * @return The JSON representation of the SearchRequest.
    +   */
    +  public static String 
toJSON(org.elasticsearch.action.search.SearchRequest esRequest) {
    +    String json = "null";
    --- End diff --
    
    Yes, definitely this is kind of weird.  This is taken care of in the latest 
commit.


---

Reply via email to