Skip to content

added optional SQL-prepared statements usage #10

New issue

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

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

Already on GitHub? # to your account

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

Conversation

IVDeg
Copy link

@IVDeg IVDeg commented May 12, 2023

On our project, we encountered memory leaks when using SQL-prepared statements. Therefore, it was decided to add the ability to disable this functionality

@mhthnz
Copy link
Owner

mhthnz commented May 12, 2023

Hey! Thank you for your pr. Can you provide more information? Because may be we will find another way to patch that code.
Tell me please:

  1. Is memory leaks on php side or on database side (like this Segmentation fault while using prepared statements without closing them tarantool/tarantool#6422)
  2. What part of project it was? (cron task/console command/php-fpm handler)
  3. Can you provide reproducible script where memory leak is?

@mhthnz mhthnz added the question Further information is requested label May 13, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants