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

Move Transforms to Separate Package #554

Open
joshua-oss opened this issue May 11, 2023 · 1 comment
Open

Move Transforms to Separate Package #554

joshua-oss opened this issue May 11, 2023 · 1 comment

Comments

@joshua-oss
Copy link
Contributor

Transforms are now capable of generating metadata, which introduces a dependency between smartnoise-sql and smartnoise-synth. These packages should be able to be independently installed. A possible solution is to move transforms to a separate package and move metadata into the transforms package. Then each of the larger packages could include transforms as a dependency.

Splitting this off would be a breaking change for smartnoise-synth, because namespaces would change. It need not be a breaking change for smartnoise-sql.

@joshua-oss
Copy link
Contributor Author

Could ideally move the Metadata class at the same time, to make it easier to infer metadata from data, and instantiate transformers from Metadata. This would then also be a breaking change for smartnoise-sql.

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

No branches or pull requests

1 participant