Fix postgres detect serial in autogenerate (#1479)

[JabberWocky-22]

Fixes: https://github.com/sqlalchemy/alembic/issues/1479

Description

In https://github.com/sqlalchemy/alembic/issues/73, it tries to detact postgresql serial in autogenerate, so it won't take nextval('seq'::regclass) as server default for that column. But it takes not effect for tables not in search path. This PR fixed it.

Checklist

This pull request is:

• [ ] A documentation / typographical error fix
  • Good to go, no issue or tests are needed
• [x] 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!

[JabberWocky-22]

change log and more test has been added Please review it XD

[CaselIT]

thanks. I'll bring in it gerrit were the complete pipeline can run

[sqla-tester]

New Gerrit review created for change 24df8f906d281df92c531df5a9e1f64d8cdb8527: https://gerrit.sqlalchemy.org/c/sqlalchemy/alembic/+/5333

[sqla-tester]

Michael Bayer (zzzeek) wrote:

it's documented here: https://www.postgresql.org/docs/current/datatype-oid.html

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

[sqla-tester]

Michael Bayer (zzzeek) wrote:

code review left on gerrit

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

[sqla-tester]

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