InvisibleProgrammer opened a new pull request, #3451: URL: https://github.com/apache/hive/pull/3451
Ticket: https://issues.apache.org/jira/browse/HIVE-26288 ### What changes were proposed in this pull request? Avoid null pointer exception in the catch block by checking if the `ci` variable has value. ### Why are the changes needed? When MetaStore client wants to get the next compaction request, it can throw an error. In that case, the variable `ci` has a null value but as there is an Exception, the method doesn't exit the method at the `if (ci==null)` null check, and the error handling part assumes ci has a value (based on the previous null check). ### Does this PR introduce _any_ user-facing change? No ### How was this patch tested? Added a new test in `TestWorker` called `testFindNextCompactThrowsTException` -- 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]
