Skip to content

Conversation

passalaqua
Copy link
Contributor

  • Add warning boxes indicating Datastorage_QueryPlan should not be used in production
  • Do not claim DatabaseJdbcUrl overrides DatabaseName in all cases; this depends on database type and we recommend to always set it

Copy link
Collaborator

@MarkvanMents MarkvanMents left a comment

Choose a reason for hiding this comment

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

Thanks for this Augusto (@passalaqua )
I've proposed one change which makes it clearer for me, is it still technically accurate.
Also, you seem to have accidentally deleted something in Mx10 - or was that a deliberate removal?
Let me know what you think and I can publish when you are happy.

@passalaqua passalaqua force-pushed the dat/docs/misc-fixes-queryplan-settings branch from a5ac893 to 2b735fa Compare September 30, 2025 15:26
@passalaqua passalaqua force-pushed the dat/docs/misc-fixes-queryplan-settings branch from 0da2e58 to b28f6f0 Compare October 1, 2025 08:05
Copy link
Contributor

@dahfjkg dahfjkg left a comment

Choose a reason for hiding this comment

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

Passed team review

Copy link
Collaborator

@MarkvanMents MarkvanMents left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, @passalaqua
Approving and waiting for confirmation to publish.

@MarkvanMents MarkvanMents merged commit 1b834ed into mendix:development Oct 2, 2025
2 checks passed
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.

3 participants