inefficiëntie BAG/BRK materialized views verbeteren en duplicaten voorkomen - #2617
inefficiëntie BAG/BRK materialized views verbeteren en duplicaten voorkomen#2617de-jacco wants to merge 35 commits into
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
mprins
left a comment
There was a problem hiding this comment.
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 |
|
Op verzoek van @KJLammers deze PR heropend om te publiceren. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #2617 +/- ##
========================================
+ Coverage 37% 37% +1%
Complexity 74 74
========================================
Files 199 199
Lines 13551 13551
Branches 1431 1431
========================================
+ Hits 4908 4909 +1
Misses 8236 8236
+ Partials 407 406 -1 🚀 New features to boost your workflow:
|
|
In overleg met Kees-Jan is besloten de group-by toevoeging weg te halen in mb_kadastraleonroerendezakenmetadres. |
|
ik krijg error "ORA-00904: "ZRR"."TIJDSTIPAANBIEDING2": invalid identifier" in de check dit terwijl tijdstipaanbieding2 wel in de view voorkomt waarnaar gerefereerd wordt @mprins ik kan deze rommel niet opruimen ik kom er met AI ook niet uit. dit is ook niet mijn bouwsel geweest. kan jij het afmaken zodat de checks gewoon goed gaan? |
| uzr.isgebaseerdop, | ||
| st1.tijdstipaanbieding, | ||
| uzr.isgebaseerdop2, | ||
| st2.tijdstipaanbieding AS tijdstipaanbieding2, |
There was a problem hiding this comment.
Hier wordt duidelijk tijdstipaanbieding en tijdstipaanbieding2 aangemaakt
| st2.tijdstipaanbieding as tijdstipaanbieding_stuk2 | ||
| -- BRMO-401: tijdstipaanbieding nu uit mb_zr_rechth te halen. | ||
| zrr.tijdstipaanbieding AS tijdstipaanbieding_stuk, | ||
| zrr.tijdstipaanbieding2 AS tijdstipaanbieding_stuk2 |
There was a problem hiding this comment.
in de view mb_avg_zr_rechth bestaat tijdstipaanbieding2 niet...
|
lokaal een oracle instaltie in de lucht brenhen met onderstaande (shell en Maven) scripts: mvn clean install -Dmaven.test.skip=true -Ddocker.skip=true -Dtest.onlyITs= -B -V -fae -Poracle -DskipQA=true
projectversion=$(grep "<version>.*<.version>" -m1 pom.xml | sed -e "s/^.*<version/<version/" | cut -f2 -d">"| cut -f1 -d"<")
echo $projectversion
export PROJECTVERSION=$projectversion
sed -i s/\${project.version}/$projectversion/g ./brmo-persistence/db/create-brmo-persistence-oracle.sql
sed -i s/\${project.version}/$projectversion/g ./datamodel/brk/brk2.0_oracle.sql
./.build/ci/oracle-start-docker.sh oracle-free:23.26.2-full
./.build/ci/oracle-setup.sh
mvn -e verify -B -Poracle -T1 -Dtest.onlyITs=true -pl 'bag2-loader' -DskipQA=true
./.build/ci/oracle-setup-bag2_views.shdaarmee zouden alle "standaard" tabellen en views aangemaakt moeten worden, "extra: views moeten met de hand gemaakt worden |
|
Zijn er nog Jira tickets die hiermee worden afgesloten? dan is het handig om die in de titel van deze PR op te nemen, verder zijn alle relevante vinkjes groen 🟩 dus wat mij betreft 💚 |
2 participants
Uh oh!
There was an error while loading. Please reload this page.