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

ASF GitHub Bot commented on NIFI-4625:
--------------------------------------

Github user mattyb149 commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/2287#discussion_r161789359
  
    --- Diff: 
nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-5-processors/src/test/java/org/apache/nifi/processors/elasticsearch/TestPutElasticsearch5.java
 ---
    @@ -250,6 +250,16 @@ public void 
testPutElasticsearch5OnTriggerWithExceptions() throws IOException {
     
             // This test generates an exception on execute(),routes to failure
             runner.assertTransferCount(PutElasticsearch5.REL_FAILURE, 1);
    +        
    +     // Node Closed exception
    +        processor.setExceptionToThrow(new 
NodeClosedException(mock(StreamInput.class)));
    --- End diff --
    
    Is this test related to the version stuff? As written it is causing the 
test to fail, as the exception will cause the flow file to be transferred to 
FAILURE not RETRY.


> Add External Versioning to PutElasticSearch5 Processor
> ------------------------------------------------------
>
>                 Key: NIFI-4625
>                 URL: https://issues.apache.org/jira/browse/NIFI-4625
>             Project: Apache NiFi
>          Issue Type: Improvement
>          Components: Core Framework
>    Affects Versions: 1.3.0
>         Environment: All
>            Reporter: Pedro Gomes
>            Assignee: Pedro Gomes
>            Priority: Major
>              Labels: elasticsearch, processor
>   Original Estimate: 48h
>  Remaining Estimate: 48h
>
> Currently the PutElasticSearch5 processor does not support external 
> versioning.
> The idea would be to add a property that follows the same logic as the Id 
> property, and allows index documents with an externally controlled version.
> I've changed the code already and added some tests. Right now the changes 
> proposed are:
> - Add a new property Version in the processor block.
> - Change the Index operation to support the versioning number and versioning 
> type = external
> - Check if the versioning is used with other operation types, fail if so.
> (Idea behind this is that the bulk api doest not support external versioning 
> for any other operation except Index)



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to