Up to you if you want to address the static stuff. It would make me happy. ;)
I'm (almost) always against these types of things in static blocks. They're untestable as a contained unit. Whether they really need to be tested as an atom is another story (they probably don't). The problem with this is that when makeDirs() runs depends on when FileUtils is class loaded which is really sketchy.
FLUME-627: Make all temp files created by File.createTempFile or FileUtil.mktmpdir live in one dir.
Review Request #1736 - Created May 3, 2011 and submitted
Recent build failures have been attributed to permissions errors due to poor hygiene with temp files in tests. This puts most temp files in the testsuite into one directory per class, and allows for an environment var so that forked test runs can all write to the same tmp dir. - Replace instances of File.createTempFile with FileUtil.createTempFile. There are two interesting files (TestFileUtil, FileUtil) and the rest are essentially trivial changes. Remaining temp file locations are from other subsystems that choose their own temp file locations (zk), or hard coded tmp file locations, and are in separate patches.
Tests pass (except for a flaky, and the broken jetty tests) Also, using FLUME_TMP_DIR environment like puts most tmp files into specified dir. $ FLUME_TMP_DIR=foobar mvn package Puts almost all temp files into /tmp/foobar/*