tomscut edited a comment on pull request #4082: URL: https://github.com/apache/hadoop/pull/4082#issuecomment-1082509136
In order not to affect the basic logic, I introduced the suggestion @virajjasani mentioned. Just change the `assert` to the `Preconditions.checkArgument`. This avoids the potential security implications of disabling assertions in production. @tasanuma @Hexiaoqiao @ayushtkn Please also take a look at this. Thanks a lot. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
