Skip to content

Re-order error classification for MySQLExecuteError#4555

Open
jgao54 wants to merge 1 commit into
mainfrom
error-classifier-ordering
Open

Re-order error classification for MySQLExecuteError#4555
jgao54 wants to merge 1 commit into
mainfrom
error-classifier-ordering

Conversation

@jgao54

@jgao54 jgao54 commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

We can get MySQLGeometryParseError wrapped inside MySQLExecuteError:

[qrep] failed to pull records: MySQL execute error: could not convert mysql value for Points: failed to parse MySQL geometry WKB: IllegalArgumentException: Points of LinearRing do not form a closed linestring

Reorder the classification logic so MySQLExecuteError functions more like a catch-all before other more specific mysql errors.

@jgao54 jgao54 requested a review from a team as a code owner July 8, 2026 21:59
@claude

claude Bot commented Jul 8, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

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.

2 participants