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

Versie 2 van de bag en brk views #434

Merged
merged 35 commits into from
Aug 31, 2018
Merged

Versie 2 van de bag en brk views #434

merged 35 commits into from
Aug 31, 2018

Conversation

cvanlith
Copy link
Contributor

@cvanlith cvanlith commented Apr 23, 2018

Dit is de tweede versie van de BAG en BRK views. Deze versie moet zorgen voor een beter onderhoudbare situatie van de views. De meest gebruikte combinaties van de vele RSGB-tabellen zijn hier al samengevoegd. Doel is dat voor klantspecifieke versie nog slechts een extra niveau van views nodig zal zijn.

Tijdens de review van deze views moet nagegaan worden of inderdaad de meest gevraagde informatie via de views nu beschikbaar is. Het is waarschijnlijk nodig nog archief-informatie toe te voegen in een volgende versie.

databases

  • oracle
  • postgresql
  • ms sqlserver

Sorry, something went wrong.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
@cvanlith cvanlith requested a review from mprins April 23, 2018 14:35
@cvanlith
Copy link
Contributor Author

PR uitgebreid met materialized views en indices

--REFRESH MATERIALIZED VIEW m2_benoemd_obj_locatie_adres;
--REFRESH MATERIALIZED VIEW m2_volledig_adres;

set session authorization flamingo;
Copy link
Member

Choose a reason for hiding this comment

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

dit moet er straks wel uit voor productie anders gaat er van alles mis; de installatiehandleiding schrijft voor dat sql door de schema eigenaar wordt uitgevoerd

@mprins mprins added this to the Future milestone Jun 3, 2018
@mprins mprins changed the title versie 2 van de bag en brk views [WIP] versie 2 van de bag en brk views Aug 14, 2018
@mprins
Copy link
Member

mprins commented Aug 14, 2018

Er lijkt ook iets fout te gaan in de soap-service:

Ik heb de naam van de view in de code aangepast naar vb_util_app_re_kad_perceel
zie: 66b0169

BRMO-SOAP-SERVICE-TEST: TRACE 09:00:28 (n.b.b.s.d.EigendomInfo#findZakelijkeRechten:555) -  SELECT  gpa.perceel_identif as perceel_identif  FROM  v_bd_app_re_all_kad_perceel gpa  WHERE  gpa.app_re_identif = ?  LIMIT 10001
BRMO-SOAP-SERVICE-TEST: TRACE 09:00:28 (n.b.b.s.d.EigendomInfo#findZakelijkeRechten:556) - 66860489870000
BRMO-SOAP-SERVICE-TEST: ERROR 09:00:28 (n.b.b.s.d.EigendomInfo#createEigendomMutatieResponse:78) - EigendomMutatie opzoeken is mislukt.
org.postgresql.util.PSQLException: ERROR: relation "v_bd_app_re_all_kad_perceel" does not exist

Merge master naar bag-brk-views-v2
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.

voor sqlserver moet alles nog gedaan worden, voor oracle moet 105_...sql nog verwijderd.

22-6-2018
*/
-- DROP VIEWS
--drop view v_adres cascade;
Copy link
Member

Choose a reason for hiding this comment

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

dit is geen geldige oracle sql, in 11.2 is er wel drop view <view> cascade constraints

--DROP INDEX m_benoemd_obj_adres_identif cascade;
--DROP INDEX m_ben_obj_adr_geom_idx cascade;

--INSERT INTO gt_pk_metadata (table_schema, table_name, pk_column, pk_policy) VALUES ('basis', 'v_pand', 'objectid', 'assigned');
Copy link
Member

Choose a reason for hiding this comment

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

er moet ook een basis schema worden gemaakt.

Copy link
Member

Choose a reason for hiding this comment

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

basis doen we niet, te veel problemen met andere database smaken

versie 2
26-6-2018
*/
--drop view v_subject cascade;
Copy link
Member

Choose a reason for hiding this comment

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

dit is geen geldige oracle sql, in 11.2 is er wel drop view cascade constraints

@mprins
Copy link
Member

mprins commented Aug 25, 2018

Geen probleem als we geenbasis schema doen

ook bedenken wat we doen met het "materialized view bijwerken" automatisch proces als dat moet gebeuren in een ander schema moeten de permissies goed staan en de queries aangepast, zie:

https://github.com/B3Partners/jdbc-util/blob/master/src/main/java/nl/b3p/loader/jdbc/PostgisJdbcConverter.java#L110-L118

https://github.com/B3Partners/jdbc-util/blob/master/src/main/java/nl/b3p/loader/jdbc/PostgisJdbcConverter.java#L110-L118

mprins and others added 8 commits August 27, 2018 18:56
- transformatie uitgezet zodat spatial niet nodig is
- datum als tekst yyyy-mm-dd, zodat onvolledige datum opgenomen kunnen worden
BRK met data getest, BAG nog niet getest met data
Versie 2 BAG / BRK scripts voor MS SQL Server
…ze.sql en verwerk 120_increase_identifsize eerder dan het aanmkane van de views
@mprins mprins mentioned this pull request Aug 30, 2018
@mprins
Copy link
Member

mprins commented Aug 30, 2018

@cvanlith
In 107_brk_views.sql werd ook de tabel prs_eigendom gemaakt, maar die ben ik nu kwijt... volgens mij werd daar al nooit data ingezet, het is een tabel waar alleen een subject kolom in zit.

Wat doen we bij de upgrades? droppen? of laten staan, maar niet meer aanmaken bij nieuwe instanties?

In 120_increase_identifsize.sql wordt de kolombreedte aangepast, maar dat gaat nu niet goed...

Ik heb een PR (#505) klaar die ervan uitgaat dat we deze tabel niet meer nodig hebben;

  • dus niet meer aanmaken in nieuwe schema's,
  • het stukje dat de aanpassingen doet uitzetten

@mprins mprins self-assigned this Aug 30, 2018
@mprins mprins changed the title [WIP] versie 2 van de bag en brk views Versie 2 van de bag en brk views Aug 31, 2018
@mprins mprins merged commit 81fc3b6 into master Aug 31, 2018
@mprins mprins modified the milestones: Future, 1.6.0 Aug 31, 2018
@mprins mprins deleted the bag-brk-views-v2 branch September 3, 2018 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants