NIFI-6759: When encountering Exception during load-balanced connectio…#3800
NIFI-6759: When encountering Exception during load-balanced connectio…#3800markap14 wants to merge 1 commit into
Conversation
…n communications, after closing socket, return from method rather than continuing loop
phrocker
left a comment
There was a problem hiding this comment.
I actually saw this when I was pushing data into my warehouse. This change LGTM. Glad to see it was fixed. Thanks!
|
@phrocker can you please make sure to sign the commits when you merge them. Makes it easier when seeing the log who reviewed and/or did the actual merge. Ideally the signature is also associated with a key as described in here http://nifi.apache.org/gpg.html |
|
@joewitt I don't recall merging this -- unless I accidentally hit the merge button ( versus using the CLI)... I assumed Mark did when I saw the E-mail. |
|
Ah you must be right that Mark did. My bad. |
…n communications, after closing socket, return from method rather than continuing loop This closes apache#3800.
…n communications, after closing socket, return from method rather than continuing loop
Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:
Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?
Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.
Has your PR been rebased against the latest commit within the target branch (typically
master)?Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not
squashor use--forcewhen pushing to allow for clean monitoring of changes.For code changes:
mvn -Pcontrib-check clean installat the rootnififolder?LICENSEfile, including the mainLICENSEfile undernifi-assembly?NOTICEfile, including the mainNOTICEfile found undernifi-assembly?.displayNamein addition to .name (programmatic access) for each of the new properties?For documentation related changes:
Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.