rabbit_runtime_parameters:value_global/2 was only used in rabbit_nodes:cluster_name/0 since near the beginning of the commit history of the server and its usage was eliminated in 06932b9fcbc (#3085, released in v3.8.17+ and v3.9.0+).
rabbit_runtime_parameters:value/4 doesn't appear to have been ever used since it was introduced near the beginning of the commit history. It may have been added just to mirror value_global/2's interface.
Eliminating these dead functions allows us to also eliminate a somewhat complicated function rabbit_db_rtparams:get_or_set/2.
This change is ported out of #10915 and was introduced there so we don't need to refactor these functions to handle database errors.
rabbit_runtime_parameters:value_global/2was only used inrabbit_nodes:cluster_name/0since near the beginning of the commit history of the server and its usage was eliminated in 06932b9fcbc (#3085, released in v3.8.17+ and v3.9.0+).rabbit_runtime_parameters:value/4doesn't appear to have been ever used since it was introduced near the beginning of the commit history. It may have been added just to mirrorvalue_global/2's interface.Eliminating these dead functions allows us to also eliminate a somewhat complicated function
rabbit_db_rtparams:get_or_set/2.This change is ported out of #10915 and was introduced there so we don't need to refactor these functions to handle database errors.