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
Add live and current variant to AllEvents query and their documentation #4523
Add live and current variant to AllEvents query and their documentation #4523
Conversation
@Arkatufus still waiting on docs for this, right? |
yes, documentation should be in now |
@Arkatufus looks like we have a compilation error here |
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.
LGTM - great work @Arkatufus
_tickCancelable = Context.System.Scheduler.ScheduleTellRepeatedlyCancelable(refreshInterval, refreshInterval, Self, EventsByTagPublisher.Continue.Instance, Self); | ||
} | ||
|
||
protected override long ToOffset => long.MaxValue; |
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.
LGTM - need this to make IIsTimeForReplay
function correctly on live queries
@@ -37,42 +33,35 @@ public void ReadJournal_should_implement_IAllEventsQuery() | |||
} | |||
|
|||
[Fact] | |||
public virtual void ReadJournal_query_AllEvents_should_find_existing_events() | |||
public virtual void ReadJournal_query_AllEvents_should_find_new_events() |
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.
LGTM
|
||
probe.ExpectComplete(); | ||
probe.ExpectNoMsg(TimeSpan.FromMilliseconds(500)); | ||
// everything before "streaming" are not included, since exclusive offset |
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.
LGTM
Continuation of #4521
Closes #4518
Updates:
AllEvents
andCurrentAllEvents
documentationAllPersistenceIds
andPersistenceIds
documentation