[GitHub] metron pull request #1040: METRON-1589: '/api/v1/search/search' fails when '...

2018-06-05 Thread justinleet
Github user justinleet closed the pull request at: https://github.com/apache/metron/pull/1040 ---

[GitHub] metron pull request #1040: METRON-1589: '/api/v1/search/search' fails when '...

2018-06-04 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1040#discussion_r192870707 --- Diff: metron-platform/metron-solr/src/main/java/org/apache/metron/solr/writer/SolrWriter.java --- @@ -17,47 +17,46 @@ */ package

[GitHub] metron pull request #1040: METRON-1589: '/api/v1/search/search' fails when '...

2018-05-31 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/1040#discussion_r192221092 --- Diff: metron-platform/metron-solr/src/main/java/org/apache/metron/solr/dao/SolrDao.java --- @@ -134,13 +134,30 @@ public void patch(RetrieveLatestDao

[GitHub] metron pull request #1040: METRON-1589: '/api/v1/search/search' fails when '...

2018-05-31 Thread justinleet
GitHub user justinleet opened a pull request: https://github.com/apache/metron/pull/1040 METRON-1589: '/api/v1/search/search' fails when 'Solr Zookeeper Urls' has comma separated multiple zookeeper urls ## Contributor Comments Refactored `SolrDao` to kill `getZkHost` and made