umamaheswararao commented on pull request #2378: URL: https://github.com/apache/hadoop/pull/2378#issuecomment-706934847
Since NN needs fs for TrashEmptier and it's always HDFS. So, it's ok to initialize DFS in this case. Otherwise if user's set fs.hdfs.impl to ViewDistributedFileSystem, it will try to initialize mount points, that's unnecessary and may take longer time if more mount points. Alternatively we can reset fs.hdfs.impl to DFS class in NN, but that will make all tests also to get DFS when initiated via MiniDFsCluster. Since DFS would be cached in FS, even though user though tests try to set fs.hdfs.impl with VIewDistributedFileSystem. So, I did not try to reset config. Instead I inited DFS in trashEmptier case. ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
