[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-09 Thread simonellistonball
Github user simonellistonball commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r173416554 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/metron_service.py ---

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-09 Thread wardbekker
Github user wardbekker commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r173398330 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/metron_service.py --- @@ -70,

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-09 Thread wardbekker
Github user wardbekker commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r173397551 --- Diff: pom.xml --- @@ -97,7 +97,7 @@ ${base_hadoop_version} ${base_hbase_version} ${base_flume_version} -

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-06 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r172544606 --- Diff: metron-deployment/packaging/ambari/metron-mpack/src/main/resources/common-services/METRON/CURRENT/package/scripts/metron_service.py --- @@ -70,6

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-05 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r172351786 --- Diff: pom.xml --- @@ -97,7 +97,7 @@ ${base_hadoop_version} ${base_hbase_version} ${base_flume_version} -

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-01 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r171624949 --- Diff: metron-platform/elasticsearch-shaded/pom.xml --- @@ -31,7 +43,7 @@ org.elasticsearch.client -

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-01 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r171620565 --- Diff: metron-platform/elasticsearch-shaded/pom.xml --- @@ -31,7 +43,7 @@ org.elasticsearch.client -

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-01 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r171619920 --- Diff: metron-platform/elasticsearch-shaded/pom.xml --- @@ -31,7 +43,7 @@ org.elasticsearch.client -

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-01 Thread cestella
Github user cestella commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r171618015 --- Diff: metron-platform/elasticsearch-shaded/pom.xml --- @@ -31,7 +43,7 @@ org.elasticsearch.client -

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-01 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r171617250 --- Diff: metron-platform/elasticsearch-shaded/pom.xml --- @@ -31,7 +43,7 @@ org.elasticsearch.client -

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-01 Thread justinleet
Github user justinleet commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r171614360 --- Diff: metron-platform/elasticsearch-shaded/pom.xml --- @@ -31,7 +43,7 @@ org.elasticsearch.client -

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-01 Thread wardbekker
Github user wardbekker commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r171599970 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/utils/ElasticsearchUtils.java --- @@ -111,19 +111,24 @@ publ

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-01 Thread mmiklavc
Github user mmiklavc commented on a diff in the pull request: https://github.com/apache/metron/pull/946#discussion_r171596212 --- Diff: metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/utils/ElasticsearchUtils.java --- @@ -111,19 +111,24 @@ public

[GitHub] metron pull request #946: METRON-1465:Support for Elasticsearch X-pack

2018-03-01 Thread wardbekker
GitHub user wardbekker opened a pull request: https://github.com/apache/metron/pull/946 METRON-1465:Support for Elasticsearch X-pack ## Contributor Comments [Please place any comments here. A description of the problem/enhancement, how to reproduce the issue, your testing metho