hadoop-common-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Hemanth Yamijala <yhema...@gmail.com>
Subject Unused API in LocalDirAllocator
Date Tue, 04 Sep 2012 11:34:55 GMT
Hi,

Stumbled on the fact that LocalDirAllocator.ifExists() is not used
anywhere. The previous usage of this API was in the IsolationRunner
that was removed in MAPREDUCE-2606.

This API doesn't call the confChanged method and hence there is an
uninitialised variable that causes a NullPointerException. So, either
we can fix that, or remove the API if it's not required. This is also
one of the reasons why the IsolationRunner was broken in 1.0.

Thoughts ?

Thanks
hemanth

Mime
View raw message