Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/853#discussion_r168809125 --- Diff: metron-platform/metron-indexing/src/main/java/org/apache/metron/indexing/dao/search/SearchRequest.java --- @@ -101,16 +99,16 @@ public void setSort(List<SortField> sort) { this.sort = sort; } - public Optional<List<String>> getFields() { - return fields == null || fields.size() == 0 ? Optional.empty() : Optional.of(fields); + public List<String> getFields() { + return fields; } public void setFields(List<String> fields) { this.fields = fields; } - public Optional<List<String>> getFacetFields() { - return facetFields == null || facetFields.size() == 0 ? Optional.empty() : Optional.of(facetFields); + public List<String> getFacetFields() { + return facetFields; --- End diff -- If we're going to change the interface, what about ``` return facetFields == null ? new ArrayList() : facetFields; ``` so it's not null.
---