rabbitmq / ra

A Raft implementation for Erlang and Elixir that strives to be efficient and make it easier to use multiple Raft clusters in a single system.
Other
813 stars 96 forks source link

Restore process_command/3 backwards compatibility #323

Closed kjnilsson closed 1 year ago

kjnilsson commented 1 year ago

As older versions of Ra cannot understand the new reply mode format we use the old format whever a reply_mode is not specificed.

This also has smaller binary data overhead.

kjnilsson commented 1 year ago

I'm going to do an early merge of this as it is needed to fix an regression in RabbitMQ. @the-mikedavis can always revert the PR if changes are needed or we just fix foward.