Github user nickwallen commented on a diff in the pull request:
https://github.com/apache/metron/pull/1242#discussion_r227774585
--- Diff:
metron-platform/metron-elasticsearch/src/main/java/org/apache/metron/elasticsearch/dao/ElasticsearchColumnMetadataDao.java
---
@@ -64,12 +62,12 @@
/**
* An Elasticsearch administrative client.
*/
- private transient AdminClient adminClient;
+ private transient ElasticsearchClient adminClient;
--- End diff --
Can we call this `client`,`elasticClient`, or `esClient` rather than
retaining the original name `adminClient`, which doesn't make sense any longer?
---
