Skip to content

Conversation

@YO-WHATS-UP2
Copy link

@YO-WHATS-UP2 YO-WHATS-UP2 commented Jan 12, 2026

Description of what I changed

In this pull request, I have introduced SLF4J performance markers to the log.debug statements that were originally added in TRUNK-6436 to monitor OpenMRS startup performance. This change allows developers and system administrators to easily filter or redirect performance-related logs by targeting the "performance" marker.

The following changes were implemented:

  • Created a reusable PERFORMANCE_MARKER using MarkerFactory.getMarker("performance") in multiple core and web classes.
  • Updated logging calls to include the marker in ModuleUtil.java, WebModuleUtil.java, Listener.java and InitializationFilter.java.
  • Verified that all strings are parameterized to follow OpenMRS logging best practices.

Issue I worked on

see https://issues.openmrs.org/browse/TRUNK-6514

Checklist: I completed these to help reviewers :)

  • My IDE is configured to follow the code style of this project.
  • I have added tests to cover my changes. (N/A: This is a logging enhancement; verified by existing startup performance tests)
  • I ran mvn clean package right before creating this pull request and added all formatting changes to my commit.
  • All new and existing tests passed.
  • My pull request is based on the latest changes of the master branch.

@YO-WHATS-UP2 YO-WHATS-UP2 marked this pull request as ready for review January 12, 2026 08:26
@YO-WHATS-UP2 YO-WHATS-UP2 force-pushed the TRUNK-6514-performance-markers branch from c67b88f to 7fd6772 Compare January 12, 2026 12:18
@sonarqubecloud
Copy link

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