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

Schema compatibility (dbtoyaml) #50

Open
danielporto opened this issue Jan 5, 2023 · 1 comment
Open

Schema compatibility (dbtoyaml) #50

danielporto opened this issue Jan 5, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@danielporto
Copy link

Thanks for the great tool!
It would be nice if this is compatible with dbtoyaml: https://pyrseas.readthedocs.io/en/latest/
Currently, to make the yaml compatible, it is required some transformation that could be avoided.

@hkage
Copy link
Contributor

hkage commented Jan 9, 2023

Hi, thanks for the feedback and I'm glad you like the tool.

It think using the same notations for the YAML definition like the output from dbtoyaml should be possible and it would make writing schema files more easy. This would also be an issue for us if we start using the tool "productiv" and our database increased over time.

I would have to check if there are any keywords clashing in both schemes first, but I think there aren't any. The only "problem" I see is that a YAML file generated from dbtoyaml would still contain a lot of values not necessary for pganonymize and the file could be a bit bloated. And on the other hand if the DB schema changes for a lot tables and columns you would have to re-create the file with dbtoyaml and add all the pganonymize definition afterwards. But that is something pganonymize couldn't prevent and has to be done manually.

@hkage hkage added the enhancement New feature or request label Jan 17, 2023
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

No branches or pull requests

2 participants