Presently invalidateBlock(..) before adding a replica into invalidates, checks whether any block replica is on stale storage, if any replica is on stale storage, it postpones deletion of the replica.
In case of corrupt replica, we can skip this logic and delete the corrupt replica immediately, as a corrupt replica can't get corrected.
One outcome of this behavior presently is namenodes showing different block states post failover, as:
If a replica is marked corrupt, the Active NN, will mark it as corrupt, and mark it for deletion and remove it from corruptReplica's and excessRedundancyMap.
If before the deletion of replica, Failover happens.
The standby Namenode will mark all the storages as stale.
Then will start processing IBR's, Now since the replica's would be on stale storage, it will skip deletion, and removal from corruptReplica's
Hence both the namenode will show different numbers and different corrupt replicas.