Complexity and Error Handling The new implementation in `isDownCheck` function is significantly complex and introduces multiple nested conditions and loops. This could lead to maintenance challenges and potential bugs. Additionally, the error handling for null checks and exceptions could be improved to prevent runtime errors.
Possible Bug The variable `nearNodeAsk` is used to make a request using `nearNode`, but in the subsequent request for `targetNodeAsk`, `nearNode` is used instead of `node`. This seems like a potential bug where the wrong node might be queried.
PR Reviewer Guide 🔍
Complexity and Error Handling
The new implementation in `isDownCheck` function is significantly complex and introduces multiple nested conditions and loops. This could lead to maintenance challenges and potential bugs. Additionally, the error handling for null checks and exceptions could be improved to prevent runtime errors. Possible Bug
The variable `nearNodeAsk` is used to make a request using `nearNode`, but in the subsequent request for `targetNodeAsk`, `nearNode` is used instead of `node`. This seems like a potential bug where the wrong node might be queried.