Skip to content

NIFI-14770: Allowing the user to reset all counters in a single action.#10115

Merged
rfellows merged 3 commits intoapache:mainfrom
mcgilman:NIFI-14770
Jul 23, 2025
Merged

NIFI-14770: Allowing the user to reset all counters in a single action.#10115
rfellows merged 3 commits intoapache:mainfrom
mcgilman:NIFI-14770

Conversation

@mcgilman
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Contributor

@rfellows rfellows left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I reviewed this from a UI perspective...

The concern that I have is that the "Reset All" button and dialog might be a bit ambiguous when the list of Counters has been filtered. The users might assume that only the Counters that match the filter criteria will be reset. Maybe the description in the confirmation dialog should indicate the total number of Counters getting reset?

I suppose another, more involved option would be to give the user the option to reset only the "filtered" Counters in addition to all. Not sure how beneficial that might be however. Might be enough to just make it more clear in the dialog that you are resetting X number of Counters.

@mcgilman
Copy link
Copy Markdown
Contributor Author

I reviewed this from a UI perspective...

The concern that I have is that the "Reset All" button and dialog might be a bit ambiguous when the list of Counters has been filtered. The users might assume that only the Counters that match the filter criteria will be reset. Maybe the description in the confirmation dialog should indicate the total number of Counters getting reset?

I suppose another, more involved option would be to give the user the option to reset only the "filtered" Counters in addition to all. Not sure how beneficial that might be however. Might be enough to just make it more clear in the dialog that you are resetting X number of Counters.

Thanks for reviewing @rfellows! I've pushed another commit that incorporates the total count into the reset prompt. Happy to tweak the language if you have further suggestions.

Copy link
Copy Markdown
Contributor

@exceptionfactory exceptionfactory left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for working on this @mcgilman! On a quick review of the REST API, I was wondering about the potential use of the DELETE method instead of PUT, but given the behavior, PUT looks good. I suppose DELETE would work as a way to actually delete existing counters, but that is a more destructive, so this looks like the right way to go.

@mcgilman mcgilman added the ui Pull requests for work relating to the user interface label Jul 22, 2025
Copy link
Copy Markdown
Contributor

@rfellows rfellows left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the changes @mcgilman. This is a nice productivity boosting feature.

@rfellows rfellows merged commit 9b474f9 into apache:main Jul 23, 2025
9 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ui Pull requests for work relating to the user interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants