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

Merge of various PRs - 1 #4838

Open
wants to merge 74 commits into
base: v3.0
Choose a base branch
from
Open

Merge of various PRs - 1 #4838

wants to merge 74 commits into from

Conversation

rahim-kanji and others added 30 commits January 22, 2025 00:30
SET statements on the backend.
* Added set_charset: Manage character set configuration.
* Added connect_start_SetCharset: Enables character set
initialization during the connection start process.
* client_encoding/names
* datestyle
* timezone/time zone
* standard_conforming_strings
… yet)

* New tracked variables added:
- bytea_output
- client_min_messages
- escape_string_warning
Few optimizations
Code cleanup
- DateStyle consists of two values: a format and an order (e.g., ISO,
MDY). Unlike other parameters, DateStyle can be set with one or both
values, and the missing value retains its previous setting.

Examples:
SET DateStyle TO 'ISO, MDY' → Format: ISO, Order: MDY
SET DateStyle TO 'ISO' → Format: ISO, Order remains unchanged i.e MDY
SET DateStyle TO 'DMY' → Format remains unchanged i,e ISO, Order: DMY
SET DateStyle TO 'Postgres' → Format: Postgres, Order remains unchanged
i.e DMY

Improved RESET statements and DEFAULT parameter values handling

- Enhance RESET and DEFAULT value handling:
    • When a parameter is RESET or set to DEFAULT, ProxySQL now first
    checks if a value is provided in the connection options.
    • If a connection value is specified, it is applied; otherwise,
	the parameter falls back to the value defined by the
	corresponding pgsql-default_* variable.
enable_bitmapscan
enable_indexscan
enable_seqscan
maintenance_work_mem
Accept a reference to a vector instead of returning a deque
MySQL_Logger::insertMysqlEventsIntoDb() populate table `stats_mysql_query_events`
using mysql query events in the circular buffer
JavierJF and others added 28 commits February 14, 2025 12:43
SQLite amalgamation doesn't support DELETE with ORDER BY .
Thus the DELETE with ORDER BY was replaced by a SELECT to retrieve the maximum id,
followed by a DELETE up to that id.
* Removed async_set_names
* Removed async_set_option
* Removed dead code
* Removed async_set_autocommit
After the migration of mysql_sessions from PtrArray to vector , call to end_thread()
must be controlled in a different way
PgSQL_Connection_Placeholder to PgSQL_Connection
Removed dead code
Removed MySQL Prepare Statement related code
lib/PgSQL_Session.cpp # # Untracked files: #	.gitattributes #
.vs/ #	PR_proxysql_postg.txt #	include/query_cache - Copy.hpp #
lib/Query_Cache - Copy.cpp #	tash qqq #
test/tap/tests/pgsql-basic_tests-t - Copy.cpp #
test/tap/tests/pgsql-connection_test-t.cpp #
test/tap/tests/pgsql-set_parameter_test-t.cpp #
@JavierJF
Copy link
Collaborator

JavierJF commented Mar 3, 2025

Retest this please.

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.

3 participants