Details
-
Type:
Bug
-
Status: Resolved
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: None
-
Fix Version/s: None
-
Component/s: None
-
Labels:None
-
Tags:security
Activity
- All
- Comments
- Work Log
- History
- Activity
- Transitions
for previous version, not for commit
Show
Boris Shkolnik
added a comment - for previous version, not for commit
+ 1 patch looks good. One small nit:
+ throw new IllegalArgumentException("Can't get Kerberos configuration",ke);
put a whitespace after comma. However, it might be considered like a formatting change from the previous state of the code. So, it's up to you.
Show
Konstantin Boudnik
added a comment - + 1 patch looks good. One small nit:
+ throw new IllegalArgumentException("Can't get Kerberos configuration",ke);
put a whitespace after comma. However, it might be considered like a formatting change from the previous state of the code. So, it's up to you.
will do in trunk commit.
Show
Boris Shkolnik
added a comment - will do in trunk commit.
in trunk done as part of HADOOP-6815.
Show
Boris Shkolnik
added a comment - in trunk done as part of HADOOP-6815 .
I believe it is caused by the fact that Kerberos needs a config file to merely instantiate its Config class. Very unfortunate.