Details
-
Improvement
-
Status: Resolved
-
Major
-
Resolution: Fixed
-
None
-
None
-
None
-
Reviewed
Description
I came across a case where tgt.getEndTime() was returned null and it resulted in an NPE, this observation was popped out of a test suite execution on a cluster. The reason for logging the IOException is that it helps to troubleshoot what caused the exception, as it can come from two different calls from the try-catch.
I can see that gabor.bota handled this with HADOOP-15593, but apart from logging the fact that the ticket's endDate was null, we have not logged the exception at all.
With the current code, the exception is swallowed and the thread terminates in case the ticket's endDate is null.
As this can happen with OpenJDK for example, it is required to print the exception (stack trace, message) to the log.
The code should be updated here: https://github.com/apache/hadoop/blob/trunk/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/security/UserGroupInformation.java#L918