-
Notifications
You must be signed in to change notification settings - Fork 19
inefficiëntie BAG/BRK materialized views verbeteren en duplicaten voorkomen #2617
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
base: master
Are you sure you want to change the base?
Conversation
vb_util_zk_recht is uitgebreid met 2 case statements voor 2 nieuwe kolommen. brk.mb_zr_rechth is uitgebreid met 2 kolommen en heeft nu de eerdere joins.
vb_util_zk_recht is uitgebreid met 2 case statements voor 2 nieuwe kolommen. brk.mb_zr_rechth is uitgebreid met 2 kolommen en heeft nu de eerdere joins.
mb_kadastraleonroerendezakenmetadres heeft nu een GROUP BY, om dubbellingen te voorkomen. Deze ontstaan als een verblijfsobject in meerdere panden zit. mb_onroerendezakenmetrechthebbenden is voorzien van de 2 nieuwe kolommen, maar zonder joins. mb_avg_onroerendezakenmetrechthebbenden is voorzien van dezelfde aanpassingen.
mb_kadastraleonroerendezakenmetadres heeft nu een GROUP BY, om dubbellingen te voorkomen. Deze ontstaan als een verblijfsobject in meerdere panden zit. mb_onroerendezakenmetrechthebbenden is voorzien van de 2 nieuwe kolommen, maar zonder joins. mb_avg_onroerendezakenmetrechthebbenden is voorzien van dezelfde aanpassingen.
|
Er zullen ook update scripts gemaakt moeten worden. |
::character varying en ::text zouden overbodig zijn tegenwoordig heb ik begrepen. daarnaast werkt dat ook niet in oracle.
|
in postgres build checks krijg ik deze error |
mprins
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
overbodige casts zoals ::character varying en ::text verwijderen; de groupby uitlijnen op verschillende plaatsen en de syntax fouten oplossen.
oa.
psql:datamodel/brk/brk2.0_postgresql_views.sql:581: ERROR: column "aantekeningen" specified more than once
Error: Process completed with exit code 3.
zr_identif,
*
ERROR at line 3:
ORA-01730: invalid number of column names specified
Help: https://docs.oracle.com/error-help/db/ora-01730/
upgrade scripts in de juiste files onder https://github.com/B3Partners/brmo/tree/master/datamodel/upgrade_scripts/6.0.1-6.0.2 toevoegen. Als je een view van pgsql aanpast zullen alle views die die view gebruiken ook gedropped moeten worden, en in de juiste volgorde; de upgrade scripts worden automatisch getest, bijv. https://github.com/B3Partners/brmo/actions/workflows/upgrade-oracle.yml |
de-cast en 3 spaties weggehaald
oopsie verwijderd
|
error: 1: de view waar regel 581 toe behoort heeft uitgebreide stukken betreft 'aantekeningen'. 2: volgens de 'view changes' in github is in mijn changes geen enkele keer de tekst 'aantekeningen' voorgekomen. (0 keer toegevoegd, en 0 keer verwijderd) 3: de stukken waar 'aantekeningen' wel in genoemd wordt, zijn voor geen byte of pixel gemuteerd in deze PR. ook geen enkele verwijzing naar bijvoorbeeld waar 'aantekeningen' nu more than once specified is |
|
Daarnaast een error in de oracle builds. ERROR at line 45: Hier kom ik niet uit waarom dit mis zou gaan. het is een nieuwe column die via nieuwe joins wordt gekoppeld (st2). deze is op de zelfde wijze ingebouwd als bij postgres. ik weet (nog) geen oracle builds te maken om te troubleshooten |
| LEFT JOIN BRMO_BRK.objectlocatie o2 ON o2.heeft = o.identificatie | ||
| LEFT JOIN BRMO_BRK.adres a2 ON a2.identificatie = o2.betreft | ||
| LEFT JOIN mb_adresseerbaar_object_geometrie_bag maogb ON maogb.identificatie = a2.adresseerbaarobject; | ||
| -- [BRMO-401] GROUP BY, om dubbellingen te voorkomen. Deze ontstaan als een verblijfsobject in meerdere panden zit. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@KJLammers de group by is niet geschikt voor oracle, het lukt mij niet om compleet geschikt te maken. het is een bijzondere group by met soortvan nested grouped waardes en aan het einde 'AS array_agg' wat ik erg apart vind.
AI beweert dat wat de group by probeert, niet mogelijk is met oracle. (de view zou dus ingrijpender gewijzigd moeten worden). maar dat geloof ik nietecht.
Uh oh!
There was an error while loading. Please reload this page.