[ 
https://issues.apache.org/jira/browse/FLUME-2876?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15390616#comment-15390616
 ] 

Lior Zeno commented on FLUME-2876:
----------------------------------

[~yh123555], thank you for the configuration example. Maybe I was not clear 
enough, but I meant adding another test in your patch that catches this issue, 
i.e. the test fails without the extra code in your patch, and succeeds with it.

Now that I'm looking at the jira history, maybe you don't need to do that 
because this issue has MANY instances, and a patch with a unit test has already 
been contributed. For example: FLUME-2576 (includes a unit test in the patch), 
FLUME-2884, FLUME-2391, FLUME-2435, FLUME-2424.

> ElasticSearchSink: indexnameBuilderContext.putAll bug fixes
> -----------------------------------------------------------
>
>                 Key: FLUME-2876
>                 URL: https://issues.apache.org/jira/browse/FLUME-2876
>             Project: Flume
>          Issue Type: Bug
>          Components: Sinks+Sources
>    Affects Versions: v1.5.0, v1.6.0, v1.7.0
>            Reporter: yh123555
>              Labels: unit-test-missing
>             Fix For: v1.7.0
>
>         Attachments: 
> 0001-ElasticSearchSink-indexnameBuilderContext.putAll-bug.patch
>
>
> ElasticSearchSink: indexnameBuilderContext.putAll bug fixes
> org.apache.flume.sink.elasticsearch.ElasticSearchSink 
> indexnameBuilderContext.putAll wrong  serializerContext.putAll



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to