nats-io / nats-server

High-Performance server for NATS.io, the cloud and edge native messaging system.
https://nats.io
Apache License 2.0
16k stars 1.41k forks source link

Improve pinning/unpinning in Priority Groups #6110

Closed Jarema closed 1 week ago

Jarema commented 1 week ago

After round of feedback, few changes were requested:

Fix unpin advisory name

Copy-paste mistake was made in unpin advisory name.

Fix PinnedTS

It was meant to return time when pinned happened, not the daedline.

Add check for group name

As the ADR for Priority Groups was also updated after feedbach, this PR implements that.

cc @ripienaar @jnmoyne

Signed-off-by: Tomasz Pietrek tomasz@nats.io