Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@paulineribeyre
Copy link
Contributor

@paulineribeyre paulineribeyre commented Nov 17, 2025

  File "/Users/paulineribeyre/.virtualenvs/sheepdog-3.9-v2/lib/python3.9/site-packages/datamodelutils/postgres_admin.py", line 166, in check_version
    if not (driver.engine.dialect.has_table(driver.engine, "node_root")):
  File "/Users/paulineribeyre/.virtualenvs/sheepdog-3.9-v2/lib/python3.9/site-packages/sqlalchemy/dialects/postgresql/base.py", line 3636, in has_table
    self._ensure_has_table_connection(connection)
  File "/Users/paulineribeyre/.virtualenvs/sheepdog-3.9-v2/lib/python3.9/site-packages/sqlalchemy/engine/default.py", line 342, in _ensure_has_table_connection
    raise exc.ArgumentError(
sqlalchemy.exc.ArgumentError: The argument passed to Dialect.has_table() should be a <class
'sqlalchemy.engine.base.Connection'>, got <class 'sqlalchemy.engine.base.Engine'>. Additionally,
the Dialect.has_table() method is for internal dialect use only; please use
``inspect(some_engine).has_table(<tablename>>)`` for public API use.

Link to JIRA ticket if there is one: https://ctds-planx.atlassian.net/browse/PD-13

New Features

Breaking Changes

Bug Fixes

  • Fix call to sqlalchemy 'has_table'

Improvements

Dependency updates

Deployment changes

@github-actions
Copy link

The style in this PR agrees with black. ✔️

This formatting comment was generated automatically by a script in uc-cdis/wool.

Copy link
Contributor

@george42-ctds george42-ctds left a comment

Choose a reason for hiding this comment

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

Looks good.

@paulineribeyre paulineribeyre changed the title Fix call to sqlalchemy 'has_table' PD-13 Fix call to sqlalchemy 'has_table' Nov 17, 2025
@paulineribeyre paulineribeyre merged commit 6073caf into master Nov 17, 2025
5 checks passed
@paulineribeyre paulineribeyre deleted the fix/has_table branch November 17, 2025 18:46
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