[FFmpeg-devel] [PATCH] avformat: AMQP: add option delivery_mode

[Florian Levis]

That's OK for me

--
Florian LEVIS


Le sam. 20 juin 2020 à 20:56, Andriy Gelman <andriy.gelman at gmail.com> a
écrit :

> On Sat, 20. Jun 19:16, Levis Florian wrote:
> > Signed-off-by: Levis Florian <levis.florian at gmail.com>
> > ---
> >  doc/protocols.texi    | 15 +++++++++++++++
> >  libavformat/libamqp.c |  6 +++++-
> >  2 files changed, 20 insertions(+), 1 deletion(-)
> >
> > diff --git a/doc/protocols.texi b/doc/protocols.texi
> > index 7aa758541c..2e15925eeb 100644
> > --- a/doc/protocols.texi
> > +++ b/doc/protocols.texi
> > @@ -109,6 +109,21 @@ the received message may be truncated causing
> decoding errors.
> >  The timeout in seconds during the initial connection to the broker. The
> >  default value is rw_timeout, or 5 seconds if rw_timeout is not set.
> >
> > + at item delivery_mode @var{mode}
> > +Sets the delivery mode of each messages sent.
> > +
> > +The following values are recognized:
> > + at table @samp
> > + at item persistent
> > +Delivery mode set to "persistent" (2). This is the default value.
> > +Messages may be written to the broker's disk depending on its setup.
> > +
>
> > + at item non-persistent
> > +Delivery mode set to "non-persistent" (1)
> > +Messages may be stay in the broker's memory depending on its setup.
> > +
>
> I feel the following is clearer:
>
> Messages will stay in broker's memory unless the broker is under memory
> pressure.
>
> Let me know if you agree and I'll push in a few days if there are no more
> comments.
>
> Thanks,
> --
> Andriy
>