diff options
author | Richard van der Hoff <richard@matrix.org> | 2020-04-23 15:31:25 +0100 |
---|---|---|
committer | Richard van der Hoff <richard@matrix.org> | 2020-04-23 15:44:35 +0100 |
commit | 23b28266acf0ff58eb290a0b86141d50a4fedba4 (patch) | |
tree | 30e15a43c7ddef03ee753aa595a7822c28faacb1 /synapse/replication/http/streams.py | |
parent | Revert "Revert "Merge pull request #7315 from matrix-org/babolivier/request_t... (diff) | |
download | synapse-23b28266acf0ff58eb290a0b86141d50a4fedba4.tar.xz |
Remove 'limit' param from `get_repl_stream_updates` API
there doesn't seem to be much point in passing this limit all around, since both sides agree it's meant to be 100.
Diffstat (limited to 'synapse/replication/http/streams.py')
-rw-r--r-- | synapse/replication/http/streams.py | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/synapse/replication/http/streams.py b/synapse/replication/http/streams.py index ffd4c61993..f35cebc710 100644 --- a/synapse/replication/http/streams.py +++ b/synapse/replication/http/streams.py @@ -28,7 +28,7 @@ class ReplicationGetStreamUpdates(ReplicationEndpoint): The API looks like: - GET /_synapse/replication/get_repl_stream_updates/events?from_token=0&to_token=10&limit=100 + GET /_synapse/replication/get_repl_stream_updates/<stream name>?from_token=0&to_token=10 200 OK @@ -38,6 +38,9 @@ class ReplicationGetStreamUpdates(ReplicationEndpoint): limited: False, } + If there are more rows than can sensibly be returned in one lump, `limited` will be + set to true, and the caller should call again with a new `from_token`. + """ NAME = "get_repl_stream_updates" @@ -52,8 +55,8 @@ class ReplicationGetStreamUpdates(ReplicationEndpoint): self.streams = hs.get_replication_streamer().get_streams() @staticmethod - def _serialize_payload(stream_name, from_token, upto_token, limit): - return {"from_token": from_token, "upto_token": upto_token, "limit": limit} + def _serialize_payload(stream_name, from_token, upto_token): + return {"from_token": from_token, "upto_token": upto_token} async def _handle_request(self, request, stream_name): stream = self.streams.get(stream_name) @@ -62,10 +65,9 @@ class ReplicationGetStreamUpdates(ReplicationEndpoint): from_token = parse_integer(request, "from_token", required=True) upto_token = parse_integer(request, "upto_token", required=True) - limit = parse_integer(request, "limit", required=True) updates, upto_token, limited = await stream.get_updates_since( - from_token, upto_token, limit + from_token, upto_token ) return ( |