Skip to content

Commit

Permalink
Remove redundant copy of adjust_for_message_handling_proc/0
Browse files Browse the repository at this point in the history
  • Loading branch information
mkuratczyk committed Sep 20, 2024
1 parent e346cdc commit 3b7cdde
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 15 deletions.
13 changes: 0 additions & 13 deletions deps/rabbit/src/rabbit_channel.erl
Original file line number Diff line number Diff line change
Expand Up @@ -2786,16 +2786,3 @@ maybe_decrease_global_publishers(#ch{publishing_mode = true}) ->

is_global_qos_permitted() ->
rabbit_deprecated_features:is_permitted(global_qos).

adjust_vheap() ->
case code_version:get_otp_version() of
OtpMaj when OtpMaj >= 27 ->
%% 46422 is the default min_bin_vheap_size and for OTP 27 and above
%% we want to substantially increase it for processes that may buffer
%% messages. 32x has proven workable in testing whilst not being
%% rediculously large
process_flag(min_bin_vheap_size, 46422 * 32);
_ ->
ok
end.

3 changes: 1 addition & 2 deletions deps/rabbit/src/rabbit_process_flag.erl
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
-export([adjust_for_message_handling_proc/0
]).

%% @doc Enqueues a message.
%% Adjust process flags for processes that handle RabbitMQ messages.
%% For example any process that uses the `rabbit_queue_type' module
%% may benefit from this tuning.
Expand All @@ -24,7 +23,7 @@ adjust_for_message_handling_proc() ->
%% 46422 is the default min_bin_vheap_size and for OTP 27 and above
%% we want to substantially increase it for processes that may buffer
%% messages. 32x has proven workable in testing whilst not being
%% rediculously large
%% ridiculously large
process_flag(min_bin_vheap_size, 46422 * 32),
ok;
_ ->
Expand Down

0 comments on commit 3b7cdde

Please sign in to comment.