Skip to content

NIFI-16132 Add Component ID to Standard Component Log MDC#11452

Merged
pvillard31 merged 1 commit into
apache:mainfrom
exceptionfactory:NIFI-16132
Jul 22, 2026
Merged

NIFI-16132 Add Component ID to Standard Component Log MDC#11452
pvillard31 merged 1 commit into
apache:mainfrom
exceptionfactory:NIFI-16132

Conversation

@exceptionfactory

Copy link
Copy Markdown
Contributor

Summary

NIFI-16132 Adds a componentId key to the SLF4J Message Diagnostic Context in the standard implementation of the ComponentLog to provide unambiguous reference to the component performing logging operations.

The adjustment includes requiring the componentId argument in both constructors, setting it in the MDC along with existing logging attributes.

Additional changes include renaming SimpleProcessLogger to StandardComponentLog, aligning with common project naming conventions for classes.

Tracking

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

Issue Tracking

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

- Renamed SimpleProcessLogger to StandardComponentLog
- Adjusted constructors to require componentId in both cases
@pvillard31
pvillard31 merged commit 59cff97 into apache:main Jul 22, 2026
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants