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

Extract datum view #342

Merged
merged 4 commits into from
Jan 14, 2022
Merged

Extract datum view #342

merged 4 commits into from
Jan 14, 2022

Conversation

fsteggink
Copy link
Member

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

@sebastic
Copy link
Contributor

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

@fsteggink
Copy link
Member Author

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
Copy link
Contributor

justb4 commented Jan 13, 2022

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

@fsteggink
Copy link
Member Author

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 justb4 added this to the Versie 1.5.4 milestone Jan 14, 2022
@justb4 justb4 merged commit 5c3c1c6 into master Jan 14, 2022
@justb4
Copy link
Contributor

justb4 commented Jan 14, 2022

Ok, bedankt @fsteggink !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants