On 5/8/2023 11:18 AM, Chang S. Bae wrote:
I thought this is something benign to stay here. But, yes, I agree that it is better to simplify the code.
After staring at this a bit, I realized that at least the feature flag check needs to stay there. This can populate a proper error code when the feature abruptly gets disabled (most likely due to the backup failure).
Thanks, Chang
