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

    https://github.com/apache/metron/pull/843#discussion_r151799462
  
    --- Diff: 
metron-interface/metron-rest/src/main/java/org/apache/metron/rest/service/impl/SearchServiceImpl.java
 ---
    @@ -96,6 +96,11 @@ public GroupResponse group(GroupRequest groupRequest) 
throws RestException {
       @Override
       public Map<String, FieldType> getColumnMetadata(List<String> indices) 
throws RestException {
         try {
    +      if (indices == null || indices.isEmpty()) {
    +        indices = getDefaultIndices();
    --- End diff --
    
    If you're motivated, it would make sense to add a log statement whenever we 
muck with the indices behind the scenes.  I feel like I'll be chasing a bug 
someday and not realize that the service layer is doing this.


---

Reply via email to