[
https://issues.apache.org/jira/browse/STORM-974?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14694629#comment-14694629
]
ASF GitHub Bot commented on STORM-974:
--------------------------------------
Github user caofangkun commented on a diff in the pull request:
https://github.com/apache/storm/pull/679#discussion_r36938144
--- Diff:
external/storm-elasticsearch/src/main/java/org/apache/storm/elasticsearch/trident/EsStateFactory.java
---
@@ -19,6 +19,7 @@
import backtype.storm.task.IMetricsContext;
import org.apache.storm.elasticsearch.common.EsConfig;
+import org.apache.storm.elasticsearch.common.EsTupleMapper;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
--- End diff --
this two line slf4j imports could be removed too
> [storm-elasticsearch] Introduces Tuple -> ES document mapper to get rid of
> constant field mapping (source, index, type)
> -----------------------------------------------------------------------------------------------------------------------
>
> Key: STORM-974
> URL: https://issues.apache.org/jira/browse/STORM-974
> Project: Apache Storm
> Issue Type: Improvement
> Reporter: Jungtaek Lim
> Assignee: Jungtaek Lim
>
> For now EsIndexBolt uses constant field mapping (source, index, type) which
> is not flexible.
> We can introduce tuple -> ES document mapper interface to let users define
> their relationship, as other external modules did.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
