Github user ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/2990
@mcgilman Incorporated the last review comments.
The original intent of having `operatePermissions` at each concrete entity
implementation is to clarify which entity is `OperationPermissible`. I thought
adding that to ComponentEntity is to vague as ComponentEntity is the super
class of various entities, including those do not have running status such as
`AccessPolicySummaryEntity`, `TemplateEntity` or `UserGroupEntity`.
I wasn't consistent enough while I was refactoring the code at some point
and added it to ComponentEntity. But I still think the operatePermissions
should exist with only component those can be operated. So, I removed
operatePermissions from ComponentEntity.
I hope this we have polished this PR enough to get merged. Thanks for your
comprehensive review comments!
---
