Skip to content

Conversation

@muskan124947
Copy link
Contributor

@muskan124947 muskan124947 commented Nov 12, 2025

Overview

This PR exposes new APIs to register and unregister PerformanceLogger instances at the SQLServerDriver level. This change provides more flexible and centralized management of performance logging within the Microsoft JDBC Driver for SQL Server.

Details

  • Added: Public registerPerformanceLogger() and unregisterPerformanceLogger() APIs directly on SQLServerDriver.
  • Purpose: Enables users and integrators to globally register or unregister custom PerformanceLogger implementations, streamlining performance logging setup and teardown—especially for driver-wide or application-level scenarios.
  • How it works:
    • Developers can now call these APIs on SQLServerDriver to register a PerformanceLogger once, affecting all relevant connections and operations.

Example Usage

SQLServerDriver.registerPerformanceLogger(customPerformanceLogger);
// ... driver usage ...
SQLServerDriver.unregisterPerformanceLogger();

Motivation

  • Improves API ergonomics by allowing global configuration.
  • Eases performance monitoring for enterprise integrations and diagnostics.

@muskan124947 muskan124947 self-assigned this Nov 12, 2025
@muskan124947 muskan124947 added this to the 13.3.0 milestone Nov 12, 2025
@codecov
Copy link

codecov bot commented Nov 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 56.34%. Comparing base (a5b1412) to head (4c29506).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #2838   +/-   ##
=========================================
  Coverage     56.33%   56.34%           
- Complexity     4516     4518    +2     
=========================================
  Files           151      151           
  Lines         34431    34435    +4     
  Branches       5736     5736           
=========================================
+ Hits          19397    19401    +4     
+ Misses        12406    12404    -2     
- Partials       2628     2630    +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@muskan124947 muskan124947 merged commit f58f35a into main Nov 12, 2025
19 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.

4 participants