CurrentEventsByTag does not return more than a 100 events

[ptjhuang]

Akka.Net 1.3.11
.NET Core Windows
Call SqlReadJournal.CurrentEventsByTag with empty an event log throws ArgumentException: To offset must be a positive number

Test Case: https://gist.github.com/ptjhuang/fc44fa9ca80e87fe7835f71e06aa40da

Looked at the Scala version and it looks different. Is this a bug?

Scala version

.NET version

[ismaelhamed]

LIMIT for CurrentEventsByTag is controlled by the query.journal.sql.max-buffer-size option --which is 100 by default. See line #85.

Any differences with the JVM might be worth looking at, in case they made any improvements.

cc @Horusiath @Aaronontheweb

[Horusiath]

CurrentEventsByTag should return all events, then complete the stream, once it reached the end. MaxBufferSize only describes the size of the batched, in which we internally poll for the data from the database.

[ismaelhamed]

In the current implementation CurrentEventsByTag polls the database only once, hence why the query.journal.sql.max-buffer-size acts as a limit here. Potential bug then?

[Aaronontheweb]

close via #3698