-
Notifications
You must be signed in to change notification settings - Fork 143
RDoc-3547 Evict single entity: Fix article #2166
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
Conversation
| </Tabs> | ||
|
|
||
| ## Example I | ||
| ## Evict an entity from the session's delete queue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@arekpalinski
This is the related issue:
https://issues.hibernatingrhinos.com/issue/RavenDB-25297/Load-after-Delete-Evict-does-not-trigger-a-server-call
(The example here is fine, since it doesn't perform another load afterward)
|
|
||
| ## Syntax | ||
| * Use the `Clear()` method to clear the session’s state: | ||
| it **removes ALL tracked entities** and **cancels ALL pending operations** (e.g., _Store_, _Delete_). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
But "removes ALL tracked entities" isn't something that is happening currently, right?
So we first need to check - https://issues.hibernatingrhinos.com/issue/RavenDB-25297/Load-after-Delete-Evict-does-not-trigger-a-server-call
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@arekpalinski
That issue (RavenDB-25297) only occurs when Evict is used in the flow.
I just re-ran this to test again, and from what I see -
when calling Clear, all entities are cleared, including those from the delete queue, as expected.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh, right. It's about clearing. Everything okay then
Related issue:
https://issues.hibernatingrhinos.com/issue/RDoc-3547/Evict-single-entity-Fix-article
C# @arekpalinski - pls review these latest files (for version 7.1)
(For earlier versions – 7.0, 6.2, 6.0, and 5.4 – the content was simply copied over)
Node.js @M4xymm - pls review these latest files (for version 7.1)
(For earlier versions – 7.0, 6.2, 6.0, and 5.4 – the content was simply copied over)
Any fixes/improvements needed for other languages (Java, PHP, Python) should be handled in a separate PR.