txWatch: Remove the txWatch limits - Githubissues

paritytech / json-rpc-interface-spec

30 stars 3 forks source link

txWatch: Remove the txWatch limits #153

Closed lexnv closed 7 months ago

lexnv commented 7 months ago

This PR is a followup of the proposal from: https://github.com/paritytech/json-rpc-interface-spec/pull/150#pullrequestreview-1988787293

In this PR, the concept of limits is removed from the transactionWatch, as noted in: https://github.com/paritytech/json-rpc-interface-spec/pull/150.

Servers can still return Transaction::Dropped if the node cannot keep up with the number of transactions (as per spec), however servers are not required to support a minimum of 4 subscriptions per client

cc @paritytech/subxt-team @tomaka

tomaka commented 7 months ago

So this PR is identical to #150