You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@nifi.apache.org by wgriffiths <gi...@git.apache.org> on 2017/12/05 17:01:54 UTC

[GitHub] nifi pull request #2317: NIFI-4663 FetchElasticsearch Logging

GitHub user wgriffiths opened a pull request:

    https://github.com/apache/nifi/pull/2317

    NIFI-4663 FetchElasticsearch Logging

    Remove warn log when document not found by FetchElasticsearch

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/wgriffiths/nifi FetchElasticsearch_Logging

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/nifi/pull/2317.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #2317
    
----
commit 39c102b596c8766783abcfa071a4f52f2434d99d
Author: William Griffiths <wi...@thebookpeople.co.uk>
Date:   2017-12-05T16:40:00Z

    NIFI-4663 #comment Change log for document not found from warn to debug as we have a NOT_FOUND relationship

----


---

[GitHub] nifi issue #2317: NIFI-4663 FetchElasticsearch Logging

Posted by mattyb149 <gi...@git.apache.org>.

Github user mattyb149 commented on the issue:

    https://github.com/apache/nifi/pull/2317
  
    +1 LGTM, nice catch! This should improve performance when attempting to fetch many "missing" documents. Thanks for the improvement!


---

[GitHub] nifi issue #2317: NIFI-4663 FetchElasticsearch Logging

Posted by joewitt <gi...@git.apache.org>.

Github user joewitt commented on the issue:

    https://github.com/apache/nifi/pull/2317
  
    @mattyb149 you good with this?  The change makes total sense to me


---

[GitHub] nifi issue #2317: NIFI-4663 FetchElasticsearch Logging

Posted by joewitt <gi...@git.apache.org>.

Github user joewitt commented on the issue:

    https://github.com/apache/nifi/pull/2317
  
    no performance implication really but will be less confusing to the user that already knew to specify the relationship :)  +1 will merge to master shortly


---

[GitHub] nifi pull request #2317: NIFI-4663 FetchElasticsearch Logging

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/nifi/pull/2317


---