Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Performance Enhancements in MySQL Database Management #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wiktor-jozwiak
Copy link

@wiktor-jozwiak wiktor-jozwiak commented Oct 18, 2024

Performance Improvements from Code Changes

  1. Better Connection Handling:

    • Enhanced error handling in the getConnection() method helps quickly identify connection issues, reducing downtime and improving reliability.
  2. Default Values:

    • Using sensible default values for connection pool settings prevents misconfigurations, ensuring smooth operation even if specific settings are missing.
  3. Updated Driver:

    • Switching to com.mysql.cj.jdbc.Driver ensures compatibility with the latest MySQL features, leading to faster query execution and reduced latency.
  4. JDBC URL Parameters:

    • Adding parameters like ?useSSL=false&serverTimezone=UTC avoids warnings and stabilizes connections, enhancing reliability.
  5. Safe Resource Management:

    • Checking for null before closing dataSource prevents errors and resource leaks, maintaining application performance over time.

Overall Impact

These changes should result in:

  • Reduced Latency: Faster query execution.
  • Increased Throughput: Better handling of multiple requests.
  • Improved Resource Use: Less chance of overloading the database.

Overall, expect a more stable and responsive plugin with better performance metrics!

@wiktor-jozwiak
Copy link
Author

wiktor-jozwiak commented Oct 18, 2024

You can test at your env that there was a momentary spike in MSPT earlier while receiving the reward

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.

1 participant