[GitHub] metron pull request #702: METRON-1114: Add group by capabilities to search R...

2017-09-11 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/metron/pull/702 ---

[GitHub] metron pull request #702: METRON-1114: Add group by capabilities to search R...

2017-09-08 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/702#discussion_r137893211 --- Diff: metron-interface/metron-rest/src/main/java/org/apache/metron/rest/config/IndexConfig.java --- @@ -56,8 +52,10 @@ public IndexDao indexDao() {

[GitHub] metron pull request #702: METRON-1114: Add group by capabilities to search R...

2017-09-08 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/702#discussion_r137893133 --- Diff: metron-platform/metron-indexing/src/main/java/org/apache/metron/indexing/dao/search/GroupOrderType.java --- @@ -0,0 +1,39 @@ +/** + *

[GitHub] metron pull request #702: METRON-1114: Add group by capabilities to search R...

2017-09-08 Thread merrimanr
Github user merrimanr commented on a diff in the pull request: https://github.com/apache/metron/pull/702#discussion_r137893095 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/dao/ElasticsearchDao.java --- @@ -330,43 +334,112 @@ public

[GitHub] metron pull request #702: METRON-1114: Add group by capabilities to search R...

2017-09-08 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/702#discussion_r137762973 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/dao/ElasticsearchDao.java --- @@ -330,43 +334,112 @@ public

[GitHub] metron pull request #702: METRON-1114: Add group by capabilities to search R...

2017-08-17 Thread merrimanr
GitHub user merrimanr opened a pull request: https://github.com/apache/metron/pull/702 METRON-1114: Add group by capabilities to search REST endpoint ## Contributor Comments This PR adds a group by feature to the search REST endpoint. This PR is fairly complex so I don't think