-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NIFI-13868: add pg-delete command to Nifi Toolkit #9388
base: main
Are you sure you want to change the base?
Conversation
Co-authored-by: David Handermann <[email protected]>
a2e515a
to
abec337
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for putting together this PR @grishick. I noted some styling and wording suggestions, but it looks straightforward on initial review.
*/ | ||
public class PGDelete extends AbstractNiFiCommand<StringResult> { | ||
|
||
public PGDelete() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The spacing of these class properties looks like it needs to be adjusted from eight spaces to four, for the initial indentation.
|
||
@Override | ||
public String getDescription() { | ||
return "Deletes the given process group."; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As @pvillard31 on the other PR, it would be helpful to add a bit more detail here. Recommend the following:
return "Deletes the given process group."; | |
return "Deletes the given process group. Deleting a process group requires, stopping all Processors, disabling all Controller Services, and emptying all Queues."; |
import org.apache.nifi.toolkit.cli.impl.result.StringResult; | ||
import org.apache.nifi.web.api.entity.ProcessGroupEntity; | ||
|
||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Spacing
Summary
NIFI-13868
Adding
pg-delete
command to Nifi Toolkit. The command works the same way as deleting a process group via UI. If there are messages in PG queue, deletion will fail. I ported this change from Anetac's Nifi fork where I made it a while ago. We have been using this feature in production at Anetac for months now.The code change to
main
branch is smaller than the one to support/nifi-1.x branch, because @markap14 has already added some of the functionality under NIFI-12898Related PR for support/nifi-1.x branch: #9387
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking
Pull Request Tracking
NIFI-00000
NIFI-00000
Pull Request Formatting
main
branchVerification
Please indicate the verification steps performed prior to pull request creation.
Build
mvn clean install -P contrib-check
Licensing
LICENSE
andNOTICE
filesDocumentation