-
-
Notifications
You must be signed in to change notification settings - Fork 83
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
BAGv2 verwerken van mutaties #352
base: master
Are you sure you want to change the base?
Conversation
Hoi @petersaalbrink ik zie dat er helemaal nog niet gereageerd is! Is ook wel een flinke PR die ook ingrijpt in de basis-tabellen. Maar overall lijkt mij de aanpak goed. Met Stetl configs, zonder expliciete XSLT, Python, Jinja, code, respect! Werkt deze PR voor jou/jullie? Gaat veel tijd kosten voor review en testen. Ik heb zelf die tijd niet momenteel, maar kijk of er een sponsoring of andere samenwerking mogelijk is. Of dat iemand dit leest en hierin wil stappen? Nog vraag: begrijp ik dat de mutaties uiteindelijk in eigen tabellen komen en te apply-en op een huidige stand, dus de SQL equivalent van de XML mutatie-bestanden zijn? |
Hello, will this PR be merged soon? |
@justb4 Ja, deze manier van mutaties verwerken doet al ruim een jaar wat het moet doen voor ons! De mutaties_* tabellen vervullen daarbij een "tussenstation", inderdaad het SQL equivalent van de mutatiebestanden; niet per se noodzakelijk maar was even de makkelijkste oplossing oa. voor de door jou genoemde valkuilen. Ik kan me wel voorstellen dat de use cases voor anderen wellicht net wat verschillen, waardoor een andere aanpak nodig is. Ik heb deze PR dan ook vooral beschikbaar gesteld bij wijze van voorbeeld; kan me inderdaad voorstellen dat er nog wat extra stappen nodig zijn voor dit in NLExtract terecht kan komen. @WaffiAltaher1 When I worked on this more than a year ago, there was nothing I could find for BAG 2.0 for processing mutations; I don't know about the current situation. Feel free to use this, however! But I won't be able to provide any support. |
Resolves #349