In normalizeAndValidateRequest, we check for the node label disabled check before looking for isRecovery flag. We could do some optimization here, but its a public api. I will check all references.
Basically, if we check isRecovery outside, then the isRecovery flag parameter is redundant. I was checking if other methods in normalizeAndValidateRequest was needed for recovery such as SchedulerUtils.normalizeNodeLabelExpressionInRequest.
I think this is still required if node label is enabled and this is recovery ？
This could be removed. Just a thought ,that log is helpful rt. If we need that log, i guess some checks are needed there.
Just for logging, we could do it inside normalizeAndValidateRequest itself which will be simpler