Details
-
Sub-task
-
Status: Resolved
-
Critical
-
Resolution: Fixed
-
Impala 2.10.0
-
None
-
ghx-label-9
Description
The debug action is only effective if the client is already registered. In many cases this is not true because buffer pool client registration was deferred until later in Open() as part of the IMPALA-4862 changes.
if (buffer_pool_client_.is_registered()) { // Parse [0.0, 1.0] probability. StringParser::ParseResult parse_result; double probability = StringParser::StringToFloat<double>( debug_action_param_.c_str(), debug_action_param_.size(), &parse_result); if (parse_result != StringParser::PARSE_SUCCESS || probability < 0.0 || probability > 1.0) { return Status(Substitute( "Invalid SET_DENY_RESERVATION_PROBABILITY param: '$0'", debug_action_param_)); } buffer_pool_client_.SetDebugDenyIncreaseReservation(probability); }
This means that we didn't have quite as much test coverage as we thought we had.