Skip to content

Conversation

@geertoff
Copy link
Contributor

@geertoff geertoff commented Dec 19, 2023

De volgende issues worden behandeld met deze pull request:

  • BRMO-338 : De materialized views die worden gebruikt als standaard kaartlagen.
  • BRMO-336: Zorgen dat zakelijke rechten die andere zakelijke rechten belasten ook in de materialized views terechtkomen.
  • BRMO-339: Ontbreken van eigenaarsinformatie wanneer het een VVE betreft.
  • BRMO-340: Ontbreken van gedeelde eigenaarsinformatie wanneer het een mandelige zaak betreft.
  • BRMO-341: Voeg postcode toe aan woonadres attribuut in mb_subject
  • BRMO-342: Ontbreken van zakelijke rechten die eigendomsrechten belasten in de mb_kadastraleonroerendezakenmetadres view.

Voor PostgreSQL en Oracle zijn deze fixes toegepast.

Upgrade notities:
Voor het toepassen van deze update zal de vb_util_zk_recht moeten worden gedropt en opnieuw worden aangemaakt. Veel materialized views zijn afhankelijk van deze view, waardoor deze komen te vervallen na het verwijderen van de view. De materialized views zullen dus opnieuw moeten worden aangemaakt.

@geertoff geertoff requested a review from mprins December 19, 2023 14:44
Copy link
Member

@mprins mprins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Er lijken wat discrepanties te zijn/ontstaan tussen Oracle en PostgreSQL met deze PR; het kan zijn dat dit een bewuste keuze van de product owner is; ik weet het niet.
Zo zie ik geen aanpassing voor Oracle voor de view vb_util_zk_recht_op_koz, geen materialized views voor postgres op dezelfde plek als Oracle mat views... enz.

Ik zie geen updates of beschrijving voor de upgrade notities of installatiehandleiding, dat betekent dat bestaande instanties niet

@mprins mprins requested a review from KJLammers December 29, 2023 10:04
@geertoff geertoff requested a review from mprins January 2, 2024 10:37
@mprins
Copy link
Member

mprins commented Jan 5, 2024

@geertoff
Copy link
Contributor Author

geertoff commented Jan 8, 2024

Als je de 210_... scripts opneemt in respectievelijk https://github.com/B3Partners/brmo/blob/master/.build/ci/oracle-setup-bag2_views.sh en https://github.com/B3Partners/brmo/blob/master/.build/ci/pgsql-setup-bag2_views.sh zoals de 209_... zijn opgenomen dan worden deze views geïnstalleerd als onderdeel van de CI, mogelijk moet er nog ergens een testcase aangepast om te checken of de views er ook echt zijn, bijv. https://github.com/B3Partners/brmo/blob/master/datamodel/src/test/java/nl/b3p/brmo/datamodel/MaterializedViewsTest.java en/of https://github.com/B3Partners/brmo/blob/master/datamodel/src/test/java/nl/b3p/brmo/datamodel/BAGViewsTest.java

Als het goed is heb ik dat nu gedaan, zou je het willen controleren?

Copy link
Member

@mprins mprins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Er zit nog iets fout in een Oracle view: https://github.com/B3Partners/brmo/actions/runs/7459511412/job/20295681128?pr=1992#step:7:62509
namelijk

...
Session altered.

       REPLACE(COALESCE(a.openbareruimtenaam, '') || ' ' || COALESCE(a.huisnummer::text, '') ||
                                                                                 *
ERROR at line 41:
ORA-00907: missing right parenthesis
...

volgens mij is dat (::text) PostgreSQL syntax

@mprins
Copy link
Member

mprins commented Jan 19, 2024

Het aanmaken van jouw nieuwe views mislukt omdat ze verwijzen naar het Oracle schema BRMO_BRK maar in de CI omgeving heet het BRK schema om historische redenen JENKINS_BRK, zie

CREATE USER "JENKINS_BRK" IDENTIFIED BY "jenkins_brk" DEFAULT TABLESPACE "USERS" TEMPORARY TABLESPACE "TEMP";
-- QUOTAS
ALTER USER "JENKINS_BRK" QUOTA UNLIMITED ON USERS;
-- ROLES
GRANT "CONNECT" TO "JENKINS_BRK";
GRANT "RESOURCE" TO "JENKINS_BRK";
ALTER USER "JENKINS_BRK" DEFAULT ROLE "CONNECT","RESOURCE";
-- SYSTEM PRIVILEGES
GRANT CREATE TRIGGER TO "JENKINS_BRK";
GRANT ALTER SESSION TO "JENKINS_BRK";
GRANT CREATE MATERIALIZED VIEW TO "JENKINS_BRK";
GRANT CREATE OPERATOR TO "JENKINS_BRK";
GRANT CREATE VIEW TO "JENKINS_BRK";
GRANT CREATE SESSION TO "JENKINS_BRK";
GRANT CREATE TABLE TO "JENKINS_BRK";
GRANT CREATE SEQUENCE TO "JENKINS_BRK";
GRANT UNLIMITED TABLESPACE TO "JENKINS_BRK";
GRANT CREATE PROCEDURE TO "JENKINS_BRK";

@mprins mprins merged commit 1c09003 into master Feb 2, 2024
@mprins mprins deleted the GOffenberg-patch2 branch February 2, 2024 10:52
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.

4 participants