search

nlextract / NLExtract

Convert (ETL) and visualize free Dutch geo-datasets.
https://nlextract.nl
GNU General Public License v3.0
149 stars 84 forks source link

Extract datum view #342

Closed fsteggink closed 2 years ago

fsteggink commented 2 years ago

PR voor issue #341. Viewdefinities aangepast en ook de formattering van het SQL-script verbeterd. Beide zaken in een eigen commit.

sebastic commented 2 years ago

Waarom de keus voor een VIEW, geeft dit een betere performance dan een functie?

fsteggink commented 2 years ago

Het gaat niet om de view an sich, maar om het feit dat de functie een tabel (nlx_bag_info) gebruikt die niet voorafgegaan is door de schemanaam. Dus dan moet het schema, bijv "bagactueel", in je search path staan, anders krijg je de fout die ik in #341 heb aangegeven. De performance is hetzelfde. Zie het query plan dat ik heb gepost in #341.

justb4 commented 2 years ago

@sebastic net met @fsteggink doorgenomen. #341 inderdaad een bug. De fix zoals hier lijkt mij afdoende.

fsteggink commented 2 years ago

Ik heb de master branch gemerged. Veel conflicten in het SQL script. In 1e instantie was ik een paar wijzigingen vergeten (onderin het script), maar dat is nu opgelost. Er zijn nu geen voorkomens meer van >= extract_datum.waarde in het script.

justb4 commented 2 years ago

Ok, bedankt @fsteggink !