> On June 20, 2018, 1:12 p.m., Kenneth Giusti wrote: > > include/qpid/dispatch/router_core.h > > Lines 350 (patched) > > <https://reviews.apache.org/r/67661/diff/1/?file=2042643#file2042643line350> > > > > nit: > > > > I hate to be "that guy" but I find the the name of the function > > ambiguous. I'd expect a "check" to either succeed or fail. Perhaps > > something along the lines of > > > > qdr_terminus_set_default_expiry(*term) > > > > or > > > > qdr_terminus_set_default_expiry(*term, default) > > > > would be more flexible?
I agree it is not a great name... how about qdr_terminus_set_link_expire_if_unset? Is that too long? - Gordon ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/67661/#review205063 ----------------------------------------------------------- On June 19, 2018, 10:25 p.m., Gordon Sim wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/67661/ > ----------------------------------------------------------- > > (Updated June 19, 2018, 10:25 p.m.) > > > Review request for qpid, Alan Conway and Ted Ross. > > > Bugs: DISPATCH-1020 > https://issues.apache.org/jira/browse/DISPATCH-1020 > > > Repository: qpid-dispatch > > > Description > ------- > > The default expiry policy for a terminus is session-end. Now that link routed > links share a session by default, this is likely not what is wanted (terminus > state on a broker would have to be kept until the session ended, which is > when the connection between broker and router ends). This change propagates > an explicit link-detach policy where none is explicitly requested by the > client. If the client does request an explicit policy, that is propagated > without alteration. > > Note this requires a change to proton PROTON-1866, > https://reviews.apache.org/r/67659/ > > > Diffs > ----- > > include/qpid/dispatch/router_core.h 8f144b0 > src/router_core/connections.c 5fdc3bf > src/router_core/terminus.c 4c0e0a3 > tests/system_tests_link_routes.py 57e6d41 > > > Diff: https://reviews.apache.org/r/67661/diff/1/ > > > Testing > ------- > > Automated tests added, all existing tests pass. > > > Thanks, > > Gordon Sim > >
