Skip to content

fix: Change column type from tinyint to bool in mysql #6936

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix: Change column type from tinyint to bool in mysql #6936

wants to merge 1 commit into from

Conversation

@quangdutran
Copy link
Contributor

@quangdutran quangdutran commented May 7, 2025 •
edited
Loading

Impact

  • [ x] Bug fix (non-breaking change which fixes expected existing functionality)
  • Enhancement/New feature (adds functionality without impacting existing logic)
  • Breaking change (fix or feature that would cause existing functionality to change)

Description

Change column type from tinyint to bool in mysql
Fix #6763

Things to be aware of

Things to worry about

Additional Context

@quangdutran quangdutran requested a review from filipelautert as a code owner May 7, 2025 11:01
@quangdutran
Copy link
Contributor Author

quangdutran commented May 7, 2025

Test

Test with MySQL 5, MySQL 8, MySQL Connector/J 8.3.0, 9.3.0

Changeset:

databaseChangeLog:
  - changeSet:
      id: create-table-users
      author: YourName
      changes:
        - createTable:
            tableName: users
            columns:
              - column:
                  name: id
                  type: INT
                  constraints:
                    primaryKey: true
                    nullable: false
              - column:
                  name: name
                  type: VARCHAR(255)
              - column:
                  name: dob
                  type: timestamp
              - column:
                  name: married
                  type: boolean

Through Connector/J, married column has name=married, type=BIT (-7), precision=1

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

Reviewers

@filipelautert filipelautert Awaiting requested review from filipelautert filipelautert is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

MySQL and BOOLEAN column types - needs a bit more work.

1 participant

@quangdutran