pvillard31 opened a new pull request, #10356:
URL: https://github.com/apache/nifi/pull/10356

   # Summary
   
   NIFI-15016 - Ensure nested versioned groups detect scheduled-state changes
   
   The versioned flow stores the same flow coordinates on every embedded 
process group in a versioned flow. When we do flow design changes or 
configuration edits, we clear those coordinates and the old comparator logic 
would see this as a condition to check the changes in a recursive manner (deep 
comparison). When we do operational changes like starting/stopping, 
enabling/disabling, we do not clear the coordinates and this is why we were 
missing enabling/disabling a component as a local change in nested process 
groups. Clearing coordinates when scheduling changes occur would falsely mark 
the group as referencing a different flow version, contradicting the 
runtime-versus-definition. Consequently, I have updated the recursion condition 
to inspect any non-empty child group. It preserves the coordinate contract 
while surfacing schedule state changes exactly like other edits.
   
   # Tracking
   
   Please complete the following tracking steps prior to pull request creation.
   
   ### Issue Tracking
   
   - [ ] [Apache NiFi Jira](https://issues.apache.org/jira/browse/NIFI) issue 
created
   
   ### Pull Request Tracking
   
   - [ ] Pull Request title starts with Apache NiFi Jira issue number, such as 
`NIFI-00000`
   - [ ] Pull Request commit message starts with Apache NiFi Jira issue number, 
as such `NIFI-00000`
   
   ### Pull Request Formatting
   
   - [ ] Pull Request based on current revision of the `main` branch
   - [ ] Pull Request refers to a feature branch with one commit containing 
changes
   
   # Verification
   
   Please indicate the verification steps performed prior to pull request 
creation.
   
   ### Build
   
   - [ ] Build completed using `./mvnw clean install -P contrib-check`
     - [ ] JDK 21
     - [ ] JDK 25
   
   ### Licensing
   
   - [ ] New dependencies are compatible with the [Apache License 
2.0](https://apache.org/licenses/LICENSE-2.0) according to the [License 
Policy](https://www.apache.org/legal/resolved.html)
   - [ ] New dependencies are documented in applicable `LICENSE` and `NOTICE` 
files
   
   ### Documentation
   
   - [ ] Documentation formatting appears as expected in rendered files
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to