René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"The initial ACK deadline set for the messages received could be too low","articleBody":"Is there a reason that https://github.com/googleapis/python-pubsub/blob/master/google/cloud/pubsub_v1/subscriber/_protocol/streaming_pull_manager.py#L493 calls `self.ack_histogram.percentile(99)` directly rather than using https://github.com/googleapis/python-pubsub/blob/master/google/cloud/pubsub_v1/subscriber/_protocol/streaming_pull_manager.py#L225 which is what the Leaser thread uses for how long it sleeps between updates?\r\n\r\nI think this is causing a bug where the the average message time decrease and so the 99th percentile decreases quicker than the \"sticky\" `ack_deadline` method. The initial ack_deadline is then shorter than the time between Leaser refreshes and the messages expire before the lease is renewed. ","author":{"url":"https://github.com/davidbraden","@type":"Person","name":"davidbraden"},"datePublished":"2021-05-17T10:52:19.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":4},"url":"https://github.com/413/python-pubsub/issues/413"}