NIFI-16321 Add Attribute Handling for Counters and Gauges

[exceptionfactory]

Summary

NIFI-16321 Adds handling for the Map of attributes to the adjustCounter and recordGauge methods in implementations of ProcessSession across the application.

Changes incorporate upgrading the NiFi API from 2.11.0 to 2.12.0, introducing the new signatures for the metrics tracking methods. Adjustments include modifications to several tests to verify handling of attributes passed to these ProcessSession methods.

Tracking

Please complete the following tracking steps prior to pull request creation.

Issue Tracking

• Apache NiFi Jira 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 contains commits signed with a registered key indicating Verified status

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 according to the License Policy
• New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

• Documentation formatting appears as expected in rendered files

[joewitt]

The PR body is NIFI-16321. Text is 16321, link is 16231 (Pierre’s dependency bump, already resolved). The feature branch is also named NIFI-16231. Easy copy-paste; worth fixing the markdown so Jira/GitHub linking is not a mess. Branch rename is optional.

[joewitt]

Mock runner never exercises session-committed counters. testAdjustCounter always uses true / CommitTiming.NOW. Gauges have both NOW and SESSION_COMMITTED. StandardProcessSessionTest covers commit-time counters; MockProcessSession does not. A one-liner processor that calls adjustCounter(..., CommitTiming.SESSION_COMMITTED) would catch a commit/rollback bug in the mock.

SharedSessionState.getGaugeValues(name) returns a live ArrayList; the attributed overload returns List.copyOf. Callers of the old method can mutate the store. Pre-existing flavor of leak, now more visible. Cheap to return List.copyOf(gaugeValues) on both.

[exceptionfactory]

The PR body is NIFI-16321. Text is 16321, link is 16231 (Pierre’s dependency bump, already resolved). The feature branch is also named NIFI-16231. Easy copy-paste; worth fixing the markdown so Jira/GitHub linking is not a mess. Branch rename is optional.

Thanks, yes, I noticed this after I opened the pull request, then fixed the commit and description, although the branch name is the same, but that is transitory.

[pvillard31]

Changes are looking good, thanks @exceptionfactory