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

[nickwallen]

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?


---