You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We used to set maxPipelinedRequests = 0 from NettyConnection.onClosing() which lead to misleading error messages. With the improved API on RequestConcurrencyController to indicate RejectedPermanently, we can minimize unnecessary failed requests on connection closure by triggering RejectedPermanently from NettyConnection.onClosing() however it’s not exposed.
AbstractStreamingHttpConnection has been modified to work around the shortcoming that ConcurrentRequestsHttpConnectionFilter has no access to onClosing(), once we make it available on NettyConnectionContext we must also update AbstractStreamingHttpConnection to leverage onClosing() again.
The text was updated successfully, but these errors were encountered:
We used to set
maxPipelinedRequests = 0
fromNettyConnection.onClosing()
which lead to misleading error messages. With the improved API onRequestConcurrencyController
to indicateRejectedPermanently
, we can minimize unnecessary failed requests on connection closure by triggeringRejectedPermanently
fromNettyConnection.onClosing()
however it’s not exposed.AbstractStreamingHttpConnection
has been modified to work around the shortcoming thatConcurrentRequestsHttpConnectionFilter
has no access toonClosing()
, once we make it available onNettyConnectionContext
we must also updateAbstractStreamingHttpConnection
to leverageonClosing()
again.The text was updated successfully, but these errors were encountered: