cdw9bf opened a new pull request #701: Fix NullPointerException when message isn't included with exception URL: https://github.com/apache/incubator-iceberg/pull/701 ### _Why is this change needed?_ Some thrift exceptions are thrown without a message. When no message is passed, Iceberg will fail with a `NullPointerException` when trying to check if the string is in the exception message. ### _How does this fix the problem?_ Does a `null` check in the first clause of the `if` statement avoiding checking to see if the string is contained within the message is the message is `null`. It should not effect the functionality of the check. ### _Any other important information?_ I discovered this error when testing Iceberg for the first time with an improper Hive configuration file. The Thrift exception thrown was because the localhost port it tried to connect to rejected the connection attempt.
---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
