claim will lead to message ttl smaller

Bug #1730335 reported by yangzhenyu on 2017-11-06
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
zaqar
Medium
yangzhenyu

Bug Description

In order to avoid the expiration of the message is deleted, when the message is still claim status.
In this case, zaqar will modifies the message ttl. But the ttl gets smaller (currently this ttl=claim_ttl+claim_grace) and may be less than age. I think this is a problem, and it should use ttl=ttl+claim_ttl+claim_grace.

description: updated
Changed in zaqar:
assignee: nobody → yangzhenyu (cdyangzhenyu)

Fix proposed to branch: master
Review: https://review.openstack.org/517901

Changed in zaqar:
status: New → In Progress

Change abandoned by yangzhenyu (<email address hidden>) on branch: master
Review: https://review.openstack.org/517901

yangzhenyu (cdyangzhenyu) wrote :

But this ttl for each message may not be the same, only get a message from backend and then modify it, so there is a considerable performance consumption, so give up here to modify.

Feilong Wang (flwang) wrote :

Thanks for opening this bug. It's a very interesting question. I'm just trying to answer it from my PoV. The message TTL is only to make sure the message could be lived until it's consumed by a client. So that's said once it's processed by a client, message TTL is not necessary to be kept as the original value. But I totally understand your point here. I will think about this and comparing the behaviours on other messaging services. Will revisit this bug soon. Thanks.

Feilong Wang (flwang) on 2017-11-19
Changed in zaqar:
importance: Undecided → Medium
wanghao (wanghao749) on 2018-06-20
tags: added: bugsmash
To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Other bug subscribers