Skip to content

7720: Mysql SET column type values are missing when using the repr method#7772

Closed
nishinishidayo wants to merge 1 commit into
sqlalchemy:mainfrom
nishinishidayo:set-repr
Closed

7720: Mysql SET column type values are missing when using the repr method#7772
nishinishidayo wants to merge 1 commit into
sqlalchemy:mainfrom
nishinishidayo:set-repr

Conversation

@nishinishidayo
Copy link
Copy Markdown
Contributor

@nishinishidayo nishinishidayo commented Feb 26, 2022

Description

Fixes #7720
Mysql SET column type values are missing when using the repr method.

Checklist

This pull request is:

  • A documentation / typographical error fix
    • Good to go, no issue or tests are needed
  • A short code fix
    • please include the issue number, and create an issue if none exists, which
      must include a complete example of the issue. one line code fixes without an
      issue and demonstration will not be accepted.
    • Please include: Fixes: #<issue number> in the commit message
    • please include tests. one line code fixes without tests will not be accepted.
  • A new feature implementation
    • please include the issue number, and create an issue if none exists, which must
      include a complete example of how the feature would look.
    • Please include: Fixes: #<issue number> in the commit message
    • please include tests.

Have a nice day!

@nishinishidayo nishinishidayo changed the title Mysql SET column type values are missing when using the repr method 7720: Mysql SET column type values are missing when using the repr method Feb 26, 2022
Copy link
Copy Markdown
Collaborator

@sqla-tester sqla-tester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, this is sqla-tester setting up my work on behalf of CaselIT to try to get revision d588454 of this pull request into gerrit so we can run tests and reviews and stuff

@sqla-tester
Copy link
Copy Markdown
Collaborator

New Gerrit review created for change d588454: https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/3636

@sqla-tester
Copy link
Copy Markdown
Collaborator

Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/3636 has been merged. Congratulations! :)

@sqla-tester
Copy link
Copy Markdown
Collaborator

mike bayer (zzzeek) wrote:

oracle failure is unrelated

View this in Gerrit at https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/3643

@sqla-tester
Copy link
Copy Markdown
Collaborator

Gerrit review https://gerrit.sqlalchemy.org/c/sqlalchemy/sqlalchemy/+/3643 has been merged. Congratulations! :)

sqlalchemy-bot pushed a commit that referenced this pull request Mar 4, 2022
Fixed issues in :class:`_mysql.SET` datatype as well as :class:`.Enum`
where the ``__repr__()`` method would not render all optional parameters in
the string output, impacting the use of these types in Alembic
autogenerate. Pull request for MySQL courtesy Yuki Nishimine.

Fixes: #7720
Fixes: #7789
Closes: #7772
Pull-request: #7772
Pull-request-sha: d588454
Co-authored-by: Mike Bayer <mike_mp@zzzcomputing.com>

Change-Id: Idcec23eab4258511d9f32f4e3d78e511ea6021f1
(cherry picked from commit a926dea)
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.

SET class in lib/sqlalchemy/dialects/mysql/enumerated.py is missing __repr__method

2 participants