You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@metron.apache.org by nickwallen <gi...@git.apache.org> on 2018/04/11 17:41:07 UTC

[GitHub] metron pull request #975: METRON-1503: Alerts are not getting populated in a...

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

    https://github.com/apache/metron/pull/975#discussion_r180833486
  
    --- Diff: metron-platform/metron-indexing/src/main/java/org/apache/metron/indexing/dao/search/SearchResponse.java ---
    @@ -61,4 +61,20 @@ public void setResults(List<SearchResult> results) {
       public void setFacetCounts(Map<String, Map<String, Long>> facetCounts) {
         this.facetCounts = facetCounts;
       }
    +
    +  @Override
    +  public boolean equals(Object o) {
    --- End diff --
    
    Would it make sense to throw in a matching `hashCode(..)` just to avoid any potential future headaches?


---