rabbitmq / rabbitmq-stomp

RabbitMQ STOMP plugin
https://www.rabbitmq.com/stomp.html
Other
49 stars 28 forks source link

Add support for consumer priority (x-priority header) #122

Closed yuhusolutions closed 6 years ago

yuhusolutions commented 6 years ago

Hi, According to https://www.rabbitmq.com/consumer-priority.html it should be possible to pass the "x-priority" header in the subscribe method, however this seems to have no effect on the consumer definition when using the Stomp adapter (over Websocket) https://www.rabbitmq.com/web-stomp.html.

While attempting with the Java lib, we can see that it works (see attachment). image

Other args for the queue like "x-message-ttl" work fine.

Rabbit mq version: 3.6.14 Erlang 19.2.3 More detail can be found in the following thread: discussion in the mailing list

Regards,

mindscratch commented 6 years ago

Try setting x-max-priority see code

michaelklishin commented 6 years ago

Thank you for your time.

Team RabbitMQ uses GitHub issues for specific actionable items engineers can work on. GitHub issues are not used for questions, investigations, root cause analysis, discussions of potential issues, etc (as defined by this team).

We get at least a dozen of questions through various venues every single day, often light on details. At that rate GitHub issues can very quickly turn into a something impossible to navigate and make sense of even for our team. Because GitHub is a tool our team uses heavily nearly every day, the signal/noise ratio of issues is something we care about a lot.

Please post this to rabbitmq-users.

Thank you.

michaelklishin commented 6 years ago

@mindscratch is correct, the header name is x-max-priority.