Implement the DeliverySpec in trigger filter dispatcher

[slinkydeveloper]

Describe the bug
With BrokerSpec.Delivery and TriggerSpec.Delivery users can configure delivery options. We need to implement these options (retries)

Expected behavior
Retry works as expected

[slinkydeveloper]

/kind good-first-issue

[slinkydeveloper]

Depends on #4832

[slinkydeveloper]

Isn't the downstream dispatcher (channel dispatcher) responsible for retrying?

It kinda is, but does it really makes sense? Wouldn't it be better to give this concern to the filter handler which already run the filtering etc? filter_handler acts like a proxy but it's actually unaware of retries etc.
I believe it's also required to implement TriggerSpec.Delivery

[pierDipi]

I left a comment in #4832, which seems relevant to answer your question.

[slinkydeveloper]

/close

[knative-prow-robot]

@slinkydeveloper: Closing this issue.

Details

In response to this:

/close

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.