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

SET autocommit=(0|1) causes [ERROR] Unable to parse query. #4453

Closed
5 tasks done
GregorFritzE2 opened this issue Feb 15, 2024 · 3 comments
Closed
5 tasks done

SET autocommit=(0|1) causes [ERROR] Unable to parse query. #4453

GregorFritzE2 opened this issue Feb 15, 2024 · 3 comments

Comments

@GregorFritzE2
Copy link

reproducible bug for ProxySQL.

If you are submitting a reproducible bug report, please provide:

  • A clear description of the issue
    As of version 2.5.4 ProxySQL emits an error message when using SQL command SET autocommit=(0|1) disable or enable commit behaviour of MySQL.
    We are aware that SET autocommit itself does not start or end a transaction. We were able to verify that SET autocommitis received by MySQL AND that the queries that happen between START TRANSACTION and COMMIT are routed to the correct hostgroup despite the log output.
    Code that relies heavily on changing the commit behaviour of MySQL on a session basis will clutter the logs with output like this.
  • ProxySQL version: 2.5.4 and 2.5.5
  • OS version: CentOS from proxysql/proxysql:2.5.4-centos and proxysql/proxysql:2.5.5-centos
  • The steps to reproduce the issue
  • Send SET autocommit=0 to MySQL via ProxySQL
  • See error in log.
  • The full ProxySQL error log (default location: /var/lib/proxysql/proxysql.log)
    proxysql.log
@renecannao
Copy link
Contributor

Hi @GregorFritzE2 , and thank you for the report.

Although, the issue is not related to autocommit at all .
It has to do with trailing semicolon at the end of the SET statement.
Therefore this issue is a duplicate of #4369 and #4380 , both solved in PR #4385 .

Unless you have any objection, I will proceed closing this ticket.

@GregorFritzE2
Copy link
Author

Hi @renecannao thanks for the fast reply!

I have no objections and look forward to the next release.

@renecannao
Copy link
Contributor

Thank you!

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

No branches or pull requests

2 participants