-
-
Notifications
You must be signed in to change notification settings - Fork 623
Description
Problem
The DBeaver integration test is failing after the recent Buddy version bump from 3.40.6 to 3.40.7.
Failed CI run: https://github.com/manticoresoftware/manticoresearch/actions/runs/21453587010/job/61792337124
Failed test: test/clt-tests/integrations/dbeaver/test-integrations-dbeaver.rec
Root Cause
Commit 570e10d09 (Bump buddy version from 3.40.6 to 3.40.7 #4193) added support for the @@tx_isolation MySQL system variable to improve DBeaver compatibility (fixes #1618).
However, the DBeaver integration test still expects the old behavior (error response).
Test Failure Details
Location: test/clt-tests/integrations/dbeaver/test-integrations-dbeaver.rec:523-525
Expected output (old behavior):
ERROR 1064 (42000) at line 1: unknown sysvar @@tx_isolation
Actual output (new behavior):
+-----------------+
| @@tx_isolation |
+-----------------+
| REPEATABLE-READ |
+-----------------+
Fix Required
Update the test expectations in test-integrations-dbeaver.rec at lines 523-525 to reflect the new behavior:
––– input –––
mysql -h0 -P9306 -e "SELECT @@tx_isolation"
––– output –––
-ERROR 1064 (42000) at line 1: unknown sysvar @@tx_isolation
++-----------------+
+| @@tx_isolation |
++-----------------+
+| REPEATABLE-READ |
++-----------------+Related
- Related issue: Implementing some MySQL wire protocol so that Manticore Search could support JPA and MyBatis framework #1618
- Buddy PR: Fixed handling of transaction variable manticoresearch-buddy#636
- Commit: 570e10d
Note
This is an expected behavior change (improvement) for MySQL client compatibility, not a regression. Only the test expectations need to be updated.