Skip to content

Conversation

@robll-v1
Copy link
Collaborator

What type of PR is this?

  • API-change
  • BUG
  • Improvement
  • Documentation
  • Feature
  • Test and CI
  • Code Refactoring

Which issue(s) this PR fixes:

issue #21512

What this PR does / why we need it:

The server was returning UUID values as strings, but the MySQL column type was still MYSQL_TYPE_UUID. Go MySQL drivers (e.g. usql) do not recognize this type and treated it as raw bytes, displaying Base64 or failing when the row encoder tried to read UUIDs as byte slices.

Fix: We downgraded the exposed column type for types.T_uuid to MYSQL_TYPE_VAR_STRING and updated the MySQL row encoding paths to handle UUIDs as strings under VAR_STRING. Added regression coverage to ensure UUIDs are encoded correctly with VAR_STRING and updated distributed SQL/result cases.

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/bug Something isn't working size/S Denotes a PR that changes [10,99] lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants