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

[CT-1940] [Feature] Stand-alone Python module for PostgresColumn #6772

Closed
3 tasks done
dbeatty10 opened this issue Jan 27, 2023 · 2 comments · Fixed by #6773
Closed
3 tasks done

[CT-1940] [Feature] Stand-alone Python module for PostgresColumn #6772

dbeatty10 opened this issue Jan 27, 2023 · 2 comments · Fixed by #6773
Labels
enhancement New feature or request

Comments

@dbeatty10
Copy link
Contributor

dbeatty10 commented Jan 27, 2023

See also: dbt-labs/dbt-redshift#290

Is this your first time submitting a feature request?

  • I have read the expectations for open source contributors
  • I have searched the existing issues, and I could not find an existing issue for this feature
  • I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature

There is a stand-alone {adapter}/column.py module almost everywhere:

The two exceptions are:

It's easy enough to standardize on the convention by splitting PostgresColumn into its own file.

Describe alternatives you've considered

It won't affect the functionality of the dbt-postgres adapter if it stays as-is. But it also won't affect the functionality if it is moved.

Who will this benefit?

It is easier to find the definition or compare definitions across adapters if things are in the standard locations. This is helpful for anyone inspecting the code, including adapter maintainers outside of dbt-postgres that want to compare/contrast implementation details.

It will also allow for slightly more standardization of this file:

from dbt.adapters.postgres.relation import PostgresColumn # noqa

Are you interested in contributing this feature?

Yep 👍

Anything else?

No response

@dbeatty10 dbeatty10 added enhancement New feature or request triage labels Jan 27, 2023
@github-actions github-actions bot changed the title [Feature] Stand-alone Python module for PostgresColumn [CT-1940] [Feature] Stand-alone Python module for PostgresColumn Jan 27, 2023
@nssalian
Copy link
Contributor

@dbeatty10, I can pick this up along with a similar change in Redshift.

@dbeatty10
Copy link
Contributor Author

@nssalian That sounds perfect! Here's the files to create or update:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants