Skip to content

NIFI-1777 Prevent deleting a connection going to a running processor#357

Closed
pvillard31 wants to merge 2 commits intoapache:masterfrom
pvillard31:NIFI-1777
Closed

NIFI-1777 Prevent deleting a connection going to a running processor#357
pvillard31 wants to merge 2 commits intoapache:masterfrom
pvillard31:NIFI-1777

Conversation

@pvillard31
Copy link
Copy Markdown
Contributor

No description provided.

@olegz
Copy link
Copy Markdown
Contributor

olegz commented Apr 17, 2016

@pvillard31 I know this is a simple one, but would you consider adding a simple unit test for it as well?

@pvillard31
Copy link
Copy Markdown
Contributor Author

@olegz You're right, just added a unit test. (Note: I believe that travis build failure is not related to this PR)

@olegz
Copy link
Copy Markdown
Contributor

olegz commented Apr 18, 2016

Perfect, reviewing now

@olegz
Copy link
Copy Markdown
Contributor

olegz commented Apr 18, 2016

@pvillard31 also, when you submit PR click on "Submit Patch" in JIRA (without writing anything in description). I know it may be a bit awkward but this is how we signal that JIRA is ready for review/resolution.

@olegz
Copy link
Copy Markdown
Contributor

olegz commented Apr 18, 2016

Was able to validate that when/if someone attempts to delete a connected processor a window pops up with the error message "Destination of Connection (LogAttribute[id=7a37a5bb-1017-4732-8c31-a6c1f44eea18]) is running". So all good, adequate tests provided, +1.

@mcgilman @scottyaslan perhaps in the future we can further address it (as an additional feature) in the UI and disable "Delete" option all together.

@asfgit asfgit closed this in f719cbf Apr 18, 2016
asfgit pushed a commit that referenced this pull request Apr 18, 2016
NIFI-1777 Added unit tests to test processor removal
This closes #357
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants