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

BAGv2 verwerken van mutaties #352

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

petersaalbrink
Copy link

Resolves #349

@justb4
Copy link
Contributor

justb4 commented Nov 29, 2022

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!
Ik heb ooit voor BAG v1 mutatie-verwerking ontwikkeld, herinner met wat valkuilen, m.n. bijv sortering XML elementen in datum/tijd.

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?

@WaffiAltaher1
Copy link

Hello, will this PR be merged soon?
If I need to process daily mutations, is there another option or is the PR the only way?

@petersaalbrink
Copy link
Author

@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.

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

Successfully merging this pull request may close these issues.

BAGv2 verwerken van mutaties
4 participants