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

fix: EXPOSED-272 [MySQL, Oracle] Unsupported type BIGINT UNSIGNED for auto-increment #1982

Merged
merged 2 commits into from
Feb 2, 2024

Conversation

bog-walk
Copy link
Member

@bog-walk bog-walk commented Feb 1, 2024

With both Oracle and MySQL, attempting to create a table with an unsigned integer or long column that should auto-increment throws an Unsupported type exception.

This occurs because AutoIncColumnType, which delegates to the underlying type, is currently set up to only detect signed integer and long column types.

It is not possible to reuse the existing integerAutoincType() or longAutoincType() if the delegate is an unsigned column because of the type syntax differences between standard and auto-incrementing columns in some databases. So new types have been added to DataTypeProvider.

… auto-increment

With both Oracle and MySQL, attempting to create a table with an unsigned integer
or long column that should auto-increment throws an `Unsupported type` exception.

This occurs because `AutoIncColumnType`, which delegates to the underlying type,
is currently set up to only detect signed integer and long column types.

It is not possible to reuse the existing `integerAutoincType()` or `longAutoincType()`
if the delegate is an unsigned column because of the type syntax differences
between standard and auto-incrementing columns in some database.
So new types have been added to `DataTypeProvider`.
@bog-walk bog-walk requested review from e5l and joc-a February 1, 2024 22:05
@bog-walk bog-walk merged commit 3fcc22f into main Feb 2, 2024
5 checks passed
@bog-walk bog-walk deleted the bog-walk/fix-unsigned-autoinc branch February 2, 2024 22:04
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