Consider dropping supervisor2

rabbitmq / rabbitmq-stomp

RabbitMQ STOMP plugin

https://www.rabbitmq.com/stomp.html

Other

49 stars 28 forks source link

Consider dropping supervisor2 #148

Closed michaelklishin closed 3 years ago

michaelklishin commented 4 years ago

supervisor2 hasn't been updated to support map flags and child specs. While this can be corrected, we should reconsider its use. It seems to be used only in a few places and replacing it with the standard supervisor might be quite straightforward.

lukebakken commented 3 years ago

supervisor2 is still necessary, see https://github.com/rabbitmq/rabbitmq-common/pull/411