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

Fix performance issues in json_dom.cc and improve query optimization #582

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

Conversation

sjdevgo
Copy link

@sjdevgo sjdevgo commented Dec 9, 2024

Changes made:

  • Fixed performance bottlenecks in the json_dom.cc file by optimizing queries and reducing unnecessary operations.
  • Improved memory management in data processing, resulting in better overall efficiency.

Why these changes are important:

These optimizations improve the speed of JSON parsing and processing, which is crucial for larger datasets. The changes were tested on local builds and show significant performance improvements.

Testing:

  • Ran unit tests for the modified code.
  • Ensured backward compatibility with existing features.

Additional Information:

This change does not affect other parts of the codebase and should not introduce regressions.

@mysql-oca-bot
Copy link

Hi, thank you for submitting this pull request. In order to consider your code we need you to sign the Oracle Contribution Agreement (OCA). Please review the details and follow the instructions at https://oca.opensource.oracle.com/
Please make sure to include your MySQL bug system user (email) in the returned form.
Thanks

@mysql-oca-bot
Copy link

Hi, thank you for your contribution. Please confirm this code is submitted under the terms of the OCA (Oracle's Contribution Agreement) you have previously signed by cutting and pasting the following text as a comment:
"I confirm the code being submitted is offered under the terms of the OCA, and that I am authorized to contribute it."
Thanks

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