szetszwo commented on pull request #461: URL: https://github.com/apache/ratis/pull/461#issuecomment-822610712
@softgitron , thanks a lot for contributing to Ratis! > Test case fix is not strictly related to the NPE fixes, but I decided to add it to this pull request. Please focus on a single aspect in each pull request. Please feel free to file multiple pull requests if there are different changes. A single big pull request is hard to review and easy to make mistakes. Indeed, it is not clear what does "Avoid NPE" mean. Which part (which class) of it try to fix? I suggest to start with a single class first. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
