-
Notifications
You must be signed in to change notification settings - Fork 2
Open
Description
Hi how are you doing ?
I just saw that you are creating a new repo for your database utils, and wanted to test the database diff 😄
(I don't know which postgresql version you aim to target, but i will still inform you of the following)
I tried it with :
- python3.11.2
- postgresql13
- your repo @ adeca16
I encountered 2 issues :
- According to my understanding of postgresql doc,
colliculocaleonly exists in postgresql 15,16, then got renamed in postgresql 17+
I didn't find a column in psql14 or older
I managed to get it working by adding a-- 15_AND_LATERguard after the existing-- 16_AND_EARLIER, (and reworking the commas) effectively targeting only 15,16 , however i am not sure how the empty local field is going to behave ? It doesn't looks like it is being used
-- 16_AND_EARLIER colliculocale as locale - In I have the issue where the code returns
results/results/schemainspect/misc.py
Line 79 in adeca16
module_name = external_caller() schemainspect.misc.pg.objinstead ofschemainspect.misc.pg.
For testing purposes I basically removed the last part, and it is working. However i am not very familiar with python'simportlibusage, and have not yet dug in the problem to understand what is going on.
I never tried migra before, but it is honestly the first tool that actually works as i want, it's really cool !
I am planning to use it to store my code as schema, migrations hand-written (maybe inspired byt results output) on the side and use your tool in CI to ensure that my migrations do generate an identical database as my updated schema 👍
Thanks
amerryma
Metadata
Metadata
Assignees
Labels
No labels