Hi Florian, On Mon, Feb 18, 2019 at 01:28:21PM +0100, Florian Tham wrote: > Hello Pablo, > > On 2/18/19 10:53 AM, Pablo Neira Ayuso wrote: > > On Mon, Feb 18, 2019 at 10:02:52AM +0100, Florian Tham wrote: > >> I assumed max_expected to refer to the max number of related connections > >> to expect. If this is not correct, I will drop this change and test again. > > > > This is the maximum number of alive expectations. Once the expectation > > is confirmed, you could have as many related connections as you want. > > I tested again with max_expected = 3, leaving the other patch hunks > unmodified. Amanda reports "connection refused" and aborts. > > After I set max_expected to 4, the backup completes successfully. Then, the new protocol version needs to set up more than 3 expectations in parallel. Please, document this in your patch description. Thanks!
