dss icon indicating copy to clipboard operation
dss copied to clipboard

Coalesce Subscriptions per callback URL

Open BenjaminPelletier opened this issue 3 years ago • 1 comments

Currently, we report a Subscriber for each Subscription. Instead, we should coalesce per URL so that each unique URL only produces one SubscriberToNotify.

Is it correct to assume that there is always only one subscription ?

Originally posted by @barroco in https://github.com/interuss/dss/pull/784#discussion_r890210741

BenjaminPelletier avatar Jun 10 '22 20:06 BenjaminPelletier

There may be many Subscriptions with the same callback URL, but that callback URL only needs to be invoked once. The SubscriberToNotify structure contains a list of Subscriptions that are triggering this (one) notification.

BenjaminPelletier avatar Sep 20 '22 23:09 BenjaminPelletier