Opened 7 years ago

Last modified 5 years ago

#1124 open defect

Wait longer for tagset ack

Reported by: zzz Owned by:
Priority: minor Milestone:
Component: router/general Version:
Keywords: performance Cc:
Parent Tickets: Sensitive: no


In OCMOSJ, we don't recognize the DeliveryStatusMessage? ack of a tagset delivery after timeout, as both dieFatal() and SendSuccessJob?.runJob() check and set AtomicBoolean? _finished.

Acks routinely come in after the message expiration and are lost, thus that tagset is wasted. It appears the min expiration is 8 seconds, should that be extended? And we should accept success after timeout. So do we need an additional timeout field?

As with tunnel builds, we must listen for responses long after the initial timeout.

Logging: WARN in OCMOSJ and InNetMessagePool? ("Dropping unhandled delivery status message")


Change History (2)

comment:1 Changed 6 years ago by str4d

Keywords: performance added
Milestone: 0.9.10

comment:2 Changed 5 years ago by str4d

Status: newopen
Note: See TracTickets for help on using tickets.