Skip to content

Conversation

@hartig
Copy link
Contributor

@hartig hartig commented Dec 12, 2025

This PR implements step 5 of #302. That is, it adds the formal definition of the evaluation of EXISTS expressions into Section 17.4.1.4 NOT EXISTS and EXISTS.

Additionally, the PR removes some text that is now obsolete in that section, and it adds a Note at the beginning of Section 18.6.2 Evaluation Semantics to clarify that the evaluation of EXISTS changes the context solution mapping μctx that is passed to the eval function.


Preview | Diff

Copy link
Member

@rubensworks rubensworks left a comment

Choose a reason for hiding this comment

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

We should also make sure that this update to the definition of EXISTS is also mentioned in the changelog. (fine for me if you prefer doing that in the final PR for #302)

Co-authored-by: Thomas Tanon <thomas@pellissier-tanon.fr>
@hartig
Copy link
Contributor Author

hartig commented Dec 12, 2025

We should also make sure that this update to the definition of EXISTS is also mentioned in the changelog. (fine for me if you prefer doing that in the final PR for #302)

Good point. I have added this as another step in #302 (which may be implemented within the same PR as the second-to-last step).

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