[
https://issues.apache.org/jira/browse/METRON-694?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15863738#comment-15863738
]
ASF GitHub Bot commented on METRON-694:
---------------------------------------
Github user cestella commented on a diff in the pull request:
https://github.com/apache/incubator-metron/pull/453#discussion_r100801585
--- Diff:
metron-platform/metron-writer/src/main/java/org/apache/metron/writer/bolt/BulkMessageWriterBolt.java
---
@@ -74,7 +81,11 @@ public void prepare(Map stormConf, TopologyContext
context, OutputCollector coll
this.writerComponent = new BulkWriterComponent<>(collector);
this.collector = collector;
super.prepare(stormConf, context, collector);
- messageGetter = MessageGetters.valueOf(messageGetterStr);
+ if
(messageGetStrategyType.equals(MessageGetters.JSON_FROM_FIELD.name()) &&
messageGetField != null) {
--- End diff --
Now that I think about it, you might want to change the `arg` to `String`
from `Object` and either:
* insist that the implementing `MessageGetter`'s provide an appropriate
constructor
* convert in the enum (e.g. `arg ->
BytesFromPosition(ConversionUtils.convert(arg, Integer.class)`)
> Index Errors from Topologies
> ----------------------------
>
> Key: METRON-694
> URL: https://issues.apache.org/jira/browse/METRON-694
> Project: Metron
> Issue Type: Bug
> Reporter: Ryan Merriman
>
> Need to make sure (and review) that all the bolts write into the error queue.
> Errors should then be consumed from the error queue and indexed.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
