You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by gi...@apache.org on 2018/05/10 14:48:20 UTC
[20/34] hbase-site git commit: Published site at
8ba2a7eeb967a80d04020e30b2fe8e8652ea1758.
http://git-wip-us.apache.org/repos/asf/hbase-site/blob/15a025c1/devapidocs/src-html/org/apache/hadoop/hbase/master/cleaner/CleanerChore.CleanerTask.html
----------------------------------------------------------------------
diff --git a/devapidocs/src-html/org/apache/hadoop/hbase/master/cleaner/CleanerChore.CleanerTask.html b/devapidocs/src-html/org/apache/hadoop/hbase/master/cleaner/CleanerChore.CleanerTask.html
index 41c0dae..f5ac8fc 100644
--- a/devapidocs/src-html/org/apache/hadoop/hbase/master/cleaner/CleanerChore.CleanerTask.html
+++ b/devapidocs/src-html/org/apache/hadoop/hbase/master/cleaner/CleanerChore.CleanerTask.html
@@ -280,303 +280,309 @@
<span class="sourceLineNo">272</span> try {<a name="line.272"></a>
<span class="sourceLineNo">273</span> POOL.latchCountUp();<a name="line.273"></a>
<span class="sourceLineNo">274</span> if (runCleaner()) {<a name="line.274"></a>
-<span class="sourceLineNo">275</span> LOG.debug("Cleaned all WALs under {}", oldFileDir);<a name="line.275"></a>
-<span class="sourceLineNo">276</span> } else {<a name="line.276"></a>
-<span class="sourceLineNo">277</span> LOG.warn("WALs outstanding under {}", oldFileDir);<a name="line.277"></a>
-<span class="sourceLineNo">278</span> }<a name="line.278"></a>
-<span class="sourceLineNo">279</span> } finally {<a name="line.279"></a>
-<span class="sourceLineNo">280</span> POOL.latchCountDown();<a name="line.280"></a>
-<span class="sourceLineNo">281</span> }<a name="line.281"></a>
-<span class="sourceLineNo">282</span> // After each cleaner chore, checks if received reconfigure notification while cleaning.<a name="line.282"></a>
-<span class="sourceLineNo">283</span> // First in cleaner turns off notification, to avoid another cleaner updating pool again.<a name="line.283"></a>
-<span class="sourceLineNo">284</span> if (POOL.reconfigNotification.compareAndSet(true, false)) {<a name="line.284"></a>
-<span class="sourceLineNo">285</span> // This cleaner is waiting for other cleaners finishing their jobs.<a name="line.285"></a>
-<span class="sourceLineNo">286</span> // To avoid missing next chore, only wait 0.8 * period, then shutdown.<a name="line.286"></a>
-<span class="sourceLineNo">287</span> POOL.updatePool((long) (0.8 * getTimeUnit().toMillis(getPeriod())));<a name="line.287"></a>
-<span class="sourceLineNo">288</span> }<a name="line.288"></a>
-<span class="sourceLineNo">289</span> } else {<a name="line.289"></a>
-<span class="sourceLineNo">290</span> LOG.debug("Cleaner chore disabled! Not cleaning.");<a name="line.290"></a>
-<span class="sourceLineNo">291</span> }<a name="line.291"></a>
-<span class="sourceLineNo">292</span> }<a name="line.292"></a>
-<span class="sourceLineNo">293</span><a name="line.293"></a>
-<span class="sourceLineNo">294</span> private void preRunCleaner() {<a name="line.294"></a>
-<span class="sourceLineNo">295</span> cleanersChain.forEach(FileCleanerDelegate::preClean);<a name="line.295"></a>
-<span class="sourceLineNo">296</span> }<a name="line.296"></a>
-<span class="sourceLineNo">297</span><a name="line.297"></a>
-<span class="sourceLineNo">298</span> public Boolean runCleaner() {<a name="line.298"></a>
-<span class="sourceLineNo">299</span> preRunCleaner();<a name="line.299"></a>
-<span class="sourceLineNo">300</span> CleanerTask task = new CleanerTask(this.oldFileDir, true);<a name="line.300"></a>
-<span class="sourceLineNo">301</span> POOL.submit(task);<a name="line.301"></a>
-<span class="sourceLineNo">302</span> return task.join();<a name="line.302"></a>
-<span class="sourceLineNo">303</span> }<a name="line.303"></a>
-<span class="sourceLineNo">304</span><a name="line.304"></a>
-<span class="sourceLineNo">305</span> /**<a name="line.305"></a>
-<span class="sourceLineNo">306</span> * Sort the given list in (descending) order of the space each element takes<a name="line.306"></a>
-<span class="sourceLineNo">307</span> * @param dirs the list to sort, element in it should be directory (not file)<a name="line.307"></a>
-<span class="sourceLineNo">308</span> */<a name="line.308"></a>
-<span class="sourceLineNo">309</span> private void sortByConsumedSpace(List<FileStatus> dirs) {<a name="line.309"></a>
-<span class="sourceLineNo">310</span> if (dirs == null || dirs.size() < 2) {<a name="line.310"></a>
-<span class="sourceLineNo">311</span> // no need to sort for empty or single directory<a name="line.311"></a>
-<span class="sourceLineNo">312</span> return;<a name="line.312"></a>
-<span class="sourceLineNo">313</span> }<a name="line.313"></a>
-<span class="sourceLineNo">314</span> dirs.sort(new Comparator<FileStatus>() {<a name="line.314"></a>
-<span class="sourceLineNo">315</span> HashMap<FileStatus, Long> directorySpaces = new HashMap<>();<a name="line.315"></a>
-<span class="sourceLineNo">316</span><a name="line.316"></a>
-<span class="sourceLineNo">317</span> @Override<a name="line.317"></a>
-<span class="sourceLineNo">318</span> public int compare(FileStatus f1, FileStatus f2) {<a name="line.318"></a>
-<span class="sourceLineNo">319</span> long f1ConsumedSpace = getSpace(f1);<a name="line.319"></a>
-<span class="sourceLineNo">320</span> long f2ConsumedSpace = getSpace(f2);<a name="line.320"></a>
-<span class="sourceLineNo">321</span> return Long.compare(f2ConsumedSpace, f1ConsumedSpace);<a name="line.321"></a>
-<span class="sourceLineNo">322</span> }<a name="line.322"></a>
-<span class="sourceLineNo">323</span><a name="line.323"></a>
-<span class="sourceLineNo">324</span> private long getSpace(FileStatus f) {<a name="line.324"></a>
-<span class="sourceLineNo">325</span> Long cached = directorySpaces.get(f);<a name="line.325"></a>
-<span class="sourceLineNo">326</span> if (cached != null) {<a name="line.326"></a>
-<span class="sourceLineNo">327</span> return cached;<a name="line.327"></a>
-<span class="sourceLineNo">328</span> }<a name="line.328"></a>
-<span class="sourceLineNo">329</span> try {<a name="line.329"></a>
-<span class="sourceLineNo">330</span> long space =<a name="line.330"></a>
-<span class="sourceLineNo">331</span> f.isDirectory() ? fs.getContentSummary(f.getPath()).getSpaceConsumed() : f.getLen();<a name="line.331"></a>
-<span class="sourceLineNo">332</span> directorySpaces.put(f, space);<a name="line.332"></a>
-<span class="sourceLineNo">333</span> return space;<a name="line.333"></a>
-<span class="sourceLineNo">334</span> } catch (IOException e) {<a name="line.334"></a>
-<span class="sourceLineNo">335</span> LOG.trace("Failed to get space consumed by path={}", f, e);<a name="line.335"></a>
-<span class="sourceLineNo">336</span> return -1;<a name="line.336"></a>
-<span class="sourceLineNo">337</span> }<a name="line.337"></a>
-<span class="sourceLineNo">338</span> }<a name="line.338"></a>
-<span class="sourceLineNo">339</span> });<a name="line.339"></a>
-<span class="sourceLineNo">340</span> }<a name="line.340"></a>
-<span class="sourceLineNo">341</span><a name="line.341"></a>
-<span class="sourceLineNo">342</span> /**<a name="line.342"></a>
-<span class="sourceLineNo">343</span> * Run the given files through each of the cleaners to see if it should be deleted, deleting it if<a name="line.343"></a>
-<span class="sourceLineNo">344</span> * necessary.<a name="line.344"></a>
-<span class="sourceLineNo">345</span> * @param files List of FileStatus for the files to check (and possibly delete)<a name="line.345"></a>
-<span class="sourceLineNo">346</span> * @return true iff successfully deleted all files<a name="line.346"></a>
-<span class="sourceLineNo">347</span> */<a name="line.347"></a>
-<span class="sourceLineNo">348</span> private boolean checkAndDeleteFiles(List<FileStatus> files) {<a name="line.348"></a>
-<span class="sourceLineNo">349</span> if (files == null) {<a name="line.349"></a>
-<span class="sourceLineNo">350</span> return true;<a name="line.350"></a>
-<span class="sourceLineNo">351</span> }<a name="line.351"></a>
-<span class="sourceLineNo">352</span><a name="line.352"></a>
-<span class="sourceLineNo">353</span> // first check to see if the path is valid<a name="line.353"></a>
-<span class="sourceLineNo">354</span> List<FileStatus> validFiles = Lists.newArrayListWithCapacity(files.size());<a name="line.354"></a>
-<span class="sourceLineNo">355</span> List<FileStatus> invalidFiles = Lists.newArrayList();<a name="line.355"></a>
-<span class="sourceLineNo">356</span> for (FileStatus file : files) {<a name="line.356"></a>
-<span class="sourceLineNo">357</span> if (validate(file.getPath())) {<a name="line.357"></a>
-<span class="sourceLineNo">358</span> validFiles.add(file);<a name="line.358"></a>
-<span class="sourceLineNo">359</span> } else {<a name="line.359"></a>
-<span class="sourceLineNo">360</span> LOG.warn("Found a wrongly formatted file: " + file.getPath() + " - will delete it.");<a name="line.360"></a>
-<span class="sourceLineNo">361</span> invalidFiles.add(file);<a name="line.361"></a>
-<span class="sourceLineNo">362</span> }<a name="line.362"></a>
-<span class="sourceLineNo">363</span> }<a name="line.363"></a>
-<span class="sourceLineNo">364</span><a name="line.364"></a>
-<span class="sourceLineNo">365</span> Iterable<FileStatus> deletableValidFiles = validFiles;<a name="line.365"></a>
-<span class="sourceLineNo">366</span> // check each of the cleaners for the valid files<a name="line.366"></a>
-<span class="sourceLineNo">367</span> for (T cleaner : cleanersChain) {<a name="line.367"></a>
-<span class="sourceLineNo">368</span> if (cleaner.isStopped() || this.getStopper().isStopped()) {<a name="line.368"></a>
-<span class="sourceLineNo">369</span> LOG.warn("A file cleaner" + this.getName() + " is stopped, won't delete any more files in:"<a name="line.369"></a>
-<span class="sourceLineNo">370</span> + this.oldFileDir);<a name="line.370"></a>
-<span class="sourceLineNo">371</span> return false;<a name="line.371"></a>
-<span class="sourceLineNo">372</span> }<a name="line.372"></a>
-<span class="sourceLineNo">373</span><a name="line.373"></a>
-<span class="sourceLineNo">374</span> Iterable<FileStatus> filteredFiles = cleaner.getDeletableFiles(deletableValidFiles);<a name="line.374"></a>
-<span class="sourceLineNo">375</span><a name="line.375"></a>
-<span class="sourceLineNo">376</span> // trace which cleaner is holding on to each file<a name="line.376"></a>
-<span class="sourceLineNo">377</span> if (LOG.isTraceEnabled()) {<a name="line.377"></a>
-<span class="sourceLineNo">378</span> ImmutableSet<FileStatus> filteredFileSet = ImmutableSet.copyOf(filteredFiles);<a name="line.378"></a>
-<span class="sourceLineNo">379</span> for (FileStatus file : deletableValidFiles) {<a name="line.379"></a>
-<span class="sourceLineNo">380</span> if (!filteredFileSet.contains(file)) {<a name="line.380"></a>
-<span class="sourceLineNo">381</span> LOG.trace(file.getPath() + " is not deletable according to:" + cleaner);<a name="line.381"></a>
-<span class="sourceLineNo">382</span> }<a name="line.382"></a>
-<span class="sourceLineNo">383</span> }<a name="line.383"></a>
-<span class="sourceLineNo">384</span> }<a name="line.384"></a>
-<span class="sourceLineNo">385</span><a name="line.385"></a>
-<span class="sourceLineNo">386</span> deletableValidFiles = filteredFiles;<a name="line.386"></a>
-<span class="sourceLineNo">387</span> }<a name="line.387"></a>
-<span class="sourceLineNo">388</span><a name="line.388"></a>
-<span class="sourceLineNo">389</span> Iterable<FileStatus> filesToDelete = Iterables.concat(invalidFiles, deletableValidFiles);<a name="line.389"></a>
-<span class="sourceLineNo">390</span> return deleteFiles(filesToDelete) == files.size();<a name="line.390"></a>
-<span class="sourceLineNo">391</span> }<a name="line.391"></a>
-<span class="sourceLineNo">392</span><a name="line.392"></a>
-<span class="sourceLineNo">393</span> /**<a name="line.393"></a>
-<span class="sourceLineNo">394</span> * Delete the given files<a name="line.394"></a>
-<span class="sourceLineNo">395</span> * @param filesToDelete files to delete<a name="line.395"></a>
-<span class="sourceLineNo">396</span> * @return number of deleted files<a name="line.396"></a>
-<span class="sourceLineNo">397</span> */<a name="line.397"></a>
-<span class="sourceLineNo">398</span> protected int deleteFiles(Iterable<FileStatus> filesToDelete) {<a name="line.398"></a>
-<span class="sourceLineNo">399</span> int deletedFileCount = 0;<a name="line.399"></a>
-<span class="sourceLineNo">400</span> for (FileStatus file : filesToDelete) {<a name="line.400"></a>
-<span class="sourceLineNo">401</span> Path filePath = file.getPath();<a name="line.401"></a>
-<span class="sourceLineNo">402</span> LOG.trace("Removing {} from archive", filePath);<a name="line.402"></a>
-<span class="sourceLineNo">403</span> try {<a name="line.403"></a>
-<span class="sourceLineNo">404</span> boolean success = this.fs.delete(filePath, false);<a name="line.404"></a>
-<span class="sourceLineNo">405</span> if (success) {<a name="line.405"></a>
-<span class="sourceLineNo">406</span> deletedFileCount++;<a name="line.406"></a>
-<span class="sourceLineNo">407</span> } else {<a name="line.407"></a>
-<span class="sourceLineNo">408</span> LOG.warn("Attempted to delete:" + filePath<a name="line.408"></a>
-<span class="sourceLineNo">409</span> + ", but couldn't. Run cleaner chain and attempt to delete on next pass.");<a name="line.409"></a>
-<span class="sourceLineNo">410</span> }<a name="line.410"></a>
-<span class="sourceLineNo">411</span> } catch (IOException e) {<a name="line.411"></a>
-<span class="sourceLineNo">412</span> e = e instanceof RemoteException ?<a name="line.412"></a>
-<span class="sourceLineNo">413</span> ((RemoteException)e).unwrapRemoteException() : e;<a name="line.413"></a>
-<span class="sourceLineNo">414</span> LOG.warn("Error while deleting: " + filePath, e);<a name="line.414"></a>
-<span class="sourceLineNo">415</span> }<a name="line.415"></a>
-<span class="sourceLineNo">416</span> }<a name="line.416"></a>
-<span class="sourceLineNo">417</span> return deletedFileCount;<a name="line.417"></a>
-<span class="sourceLineNo">418</span> }<a name="line.418"></a>
-<span class="sourceLineNo">419</span><a name="line.419"></a>
-<span class="sourceLineNo">420</span> @Override<a name="line.420"></a>
-<span class="sourceLineNo">421</span> public synchronized void cleanup() {<a name="line.421"></a>
-<span class="sourceLineNo">422</span> for (T lc : this.cleanersChain) {<a name="line.422"></a>
-<span class="sourceLineNo">423</span> try {<a name="line.423"></a>
-<span class="sourceLineNo">424</span> lc.stop("Exiting");<a name="line.424"></a>
-<span class="sourceLineNo">425</span> } catch (Throwable t) {<a name="line.425"></a>
-<span class="sourceLineNo">426</span> LOG.warn("Stopping", t);<a name="line.426"></a>
-<span class="sourceLineNo">427</span> }<a name="line.427"></a>
-<span class="sourceLineNo">428</span> }<a name="line.428"></a>
-<span class="sourceLineNo">429</span> }<a name="line.429"></a>
-<span class="sourceLineNo">430</span><a name="line.430"></a>
-<span class="sourceLineNo">431</span> @VisibleForTesting<a name="line.431"></a>
-<span class="sourceLineNo">432</span> int getChorePoolSize() {<a name="line.432"></a>
-<span class="sourceLineNo">433</span> return POOL.size;<a name="line.433"></a>
-<span class="sourceLineNo">434</span> }<a name="line.434"></a>
-<span class="sourceLineNo">435</span><a name="line.435"></a>
-<span class="sourceLineNo">436</span> /**<a name="line.436"></a>
-<span class="sourceLineNo">437</span> * @param enabled<a name="line.437"></a>
-<span class="sourceLineNo">438</span> */<a name="line.438"></a>
-<span class="sourceLineNo">439</span> public boolean setEnabled(final boolean enabled) {<a name="line.439"></a>
-<span class="sourceLineNo">440</span> return this.enabled.getAndSet(enabled);<a name="line.440"></a>
-<span class="sourceLineNo">441</span> }<a name="line.441"></a>
-<span class="sourceLineNo">442</span><a name="line.442"></a>
-<span class="sourceLineNo">443</span> public boolean getEnabled() { return this.enabled.get();<a name="line.443"></a>
-<span class="sourceLineNo">444</span> }<a name="line.444"></a>
-<span class="sourceLineNo">445</span><a name="line.445"></a>
-<span class="sourceLineNo">446</span> private interface Action<T> {<a name="line.446"></a>
-<span class="sourceLineNo">447</span> T act() throws IOException;<a name="line.447"></a>
-<span class="sourceLineNo">448</span> }<a name="line.448"></a>
-<span class="sourceLineNo">449</span><a name="line.449"></a>
-<span class="sourceLineNo">450</span> /**<a name="line.450"></a>
-<span class="sourceLineNo">451</span> * Attemps to clean up a directory, its subdirectories, and files.<a name="line.451"></a>
-<span class="sourceLineNo">452</span> * Return value is true if everything was deleted. false on partial / total failures.<a name="line.452"></a>
-<span class="sourceLineNo">453</span> */<a name="line.453"></a>
-<span class="sourceLineNo">454</span> private class CleanerTask extends RecursiveTask<Boolean> {<a name="line.454"></a>
-<span class="sourceLineNo">455</span> private final Path dir;<a name="line.455"></a>
-<span class="sourceLineNo">456</span> private final boolean root;<a name="line.456"></a>
-<span class="sourceLineNo">457</span><a name="line.457"></a>
-<span class="sourceLineNo">458</span> CleanerTask(final FileStatus dir, final boolean root) {<a name="line.458"></a>
-<span class="sourceLineNo">459</span> this(dir.getPath(), root);<a name="line.459"></a>
-<span class="sourceLineNo">460</span> }<a name="line.460"></a>
-<span class="sourceLineNo">461</span><a name="line.461"></a>
-<span class="sourceLineNo">462</span> CleanerTask(final Path dir, final boolean root) {<a name="line.462"></a>
-<span class="sourceLineNo">463</span> this.dir = dir;<a name="line.463"></a>
-<span class="sourceLineNo">464</span> this.root = root;<a name="line.464"></a>
-<span class="sourceLineNo">465</span> }<a name="line.465"></a>
-<span class="sourceLineNo">466</span><a name="line.466"></a>
-<span class="sourceLineNo">467</span> @Override<a name="line.467"></a>
-<span class="sourceLineNo">468</span> protected Boolean compute() {<a name="line.468"></a>
-<span class="sourceLineNo">469</span> LOG.debug("Cleaning under {}", dir);<a name="line.469"></a>
-<span class="sourceLineNo">470</span> List<FileStatus> subDirs;<a name="line.470"></a>
-<span class="sourceLineNo">471</span> List<FileStatus> files;<a name="line.471"></a>
-<span class="sourceLineNo">472</span> try {<a name="line.472"></a>
-<span class="sourceLineNo">473</span> // if dir doesn't exist, we'll get null back for both of these<a name="line.473"></a>
-<span class="sourceLineNo">474</span> // which will fall through to succeeding.<a name="line.474"></a>
-<span class="sourceLineNo">475</span> subDirs = getFilteredStatus(status -> status.isDirectory());<a name="line.475"></a>
-<span class="sourceLineNo">476</span> files = getFilteredStatus(status -> status.isFile());<a name="line.476"></a>
-<span class="sourceLineNo">477</span> } catch (IOException ioe) {<a name="line.477"></a>
-<span class="sourceLineNo">478</span> LOG.warn("failed to get FileStatus for contents of '{}'", dir, ioe);<a name="line.478"></a>
-<span class="sourceLineNo">479</span> return false;<a name="line.479"></a>
-<span class="sourceLineNo">480</span> }<a name="line.480"></a>
-<span class="sourceLineNo">481</span><a name="line.481"></a>
-<span class="sourceLineNo">482</span> boolean nullSubDirs = subDirs == null;<a name="line.482"></a>
-<span class="sourceLineNo">483</span> if (nullSubDirs) {<a name="line.483"></a>
-<span class="sourceLineNo">484</span> LOG.trace("There is no subdir under {}", dir);<a name="line.484"></a>
-<span class="sourceLineNo">485</span> }<a name="line.485"></a>
-<span class="sourceLineNo">486</span> if (files == null) {<a name="line.486"></a>
-<span class="sourceLineNo">487</span> LOG.trace("There is no file under {}", dir);<a name="line.487"></a>
-<span class="sourceLineNo">488</span> }<a name="line.488"></a>
-<span class="sourceLineNo">489</span><a name="line.489"></a>
-<span class="sourceLineNo">490</span> int capacity = nullSubDirs ? 0 : subDirs.size();<a name="line.490"></a>
-<span class="sourceLineNo">491</span> List<CleanerTask> tasks = Lists.newArrayListWithCapacity(capacity);<a name="line.491"></a>
-<span class="sourceLineNo">492</span> if (!nullSubDirs) {<a name="line.492"></a>
-<span class="sourceLineNo">493</span> sortByConsumedSpace(subDirs);<a name="line.493"></a>
-<span class="sourceLineNo">494</span> for (FileStatus subdir : subDirs) {<a name="line.494"></a>
-<span class="sourceLineNo">495</span> CleanerTask task = new CleanerTask(subdir, false);<a name="line.495"></a>
-<span class="sourceLineNo">496</span> tasks.add(task);<a name="line.496"></a>
-<span class="sourceLineNo">497</span> task.fork();<a name="line.497"></a>
-<span class="sourceLineNo">498</span> }<a name="line.498"></a>
-<span class="sourceLineNo">499</span> }<a name="line.499"></a>
-<span class="sourceLineNo">500</span><a name="line.500"></a>
-<span class="sourceLineNo">501</span> boolean result = true;<a name="line.501"></a>
-<span class="sourceLineNo">502</span> result &= deleteAction(() -> checkAndDeleteFiles(files), "files");<a name="line.502"></a>
-<span class="sourceLineNo">503</span> result &= deleteAction(() -> getCleanResult(tasks), "subdirs");<a name="line.503"></a>
-<span class="sourceLineNo">504</span> // if and only if files and subdirs under current dir are deleted successfully, and<a name="line.504"></a>
-<span class="sourceLineNo">505</span> // it is not the root dir, then task will try to delete it.<a name="line.505"></a>
-<span class="sourceLineNo">506</span> if (result && !root) {<a name="line.506"></a>
-<span class="sourceLineNo">507</span> result &= deleteAction(() -> fs.delete(dir, false), "dir");<a name="line.507"></a>
-<span class="sourceLineNo">508</span> }<a name="line.508"></a>
-<span class="sourceLineNo">509</span> return result;<a name="line.509"></a>
-<span class="sourceLineNo">510</span> }<a name="line.510"></a>
-<span class="sourceLineNo">511</span><a name="line.511"></a>
-<span class="sourceLineNo">512</span> /**<a name="line.512"></a>
-<span class="sourceLineNo">513</span> * Get FileStatus with filter.<a name="line.513"></a>
-<span class="sourceLineNo">514</span> * Pay attention that FSUtils #listStatusWithStatusFilter would return null,<a name="line.514"></a>
-<span class="sourceLineNo">515</span> * even though status is empty but not null.<a name="line.515"></a>
-<span class="sourceLineNo">516</span> * @param function a filter function<a name="line.516"></a>
-<span class="sourceLineNo">517</span> * @return filtered FileStatus or null if dir doesn't exist<a name="line.517"></a>
-<span class="sourceLineNo">518</span> * @throws IOException if there's an error other than dir not existing<a name="line.518"></a>
-<span class="sourceLineNo">519</span> */<a name="line.519"></a>
-<span class="sourceLineNo">520</span> private List<FileStatus> getFilteredStatus(Predicate<FileStatus> function) throws IOException {<a name="line.520"></a>
-<span class="sourceLineNo">521</span> return FSUtils.listStatusWithStatusFilter(fs, dir, status -> function.test(status));<a name="line.521"></a>
-<span class="sourceLineNo">522</span> }<a name="line.522"></a>
-<span class="sourceLineNo">523</span><a name="line.523"></a>
-<span class="sourceLineNo">524</span> /**<a name="line.524"></a>
-<span class="sourceLineNo">525</span> * Perform a delete on a specified type.<a name="line.525"></a>
-<span class="sourceLineNo">526</span> * @param deletion a delete<a name="line.526"></a>
-<span class="sourceLineNo">527</span> * @param type possible values are 'files', 'subdirs', 'dirs'<a name="line.527"></a>
-<span class="sourceLineNo">528</span> * @return true if it deleted successfully, false otherwise<a name="line.528"></a>
-<span class="sourceLineNo">529</span> */<a name="line.529"></a>
-<span class="sourceLineNo">530</span> private boolean deleteAction(Action<Boolean> deletion, String type) {<a name="line.530"></a>
-<span class="sourceLineNo">531</span> boolean deleted;<a name="line.531"></a>
-<span class="sourceLineNo">532</span> try {<a name="line.532"></a>
-<span class="sourceLineNo">533</span> LOG.trace("Start deleting {} under {}", type, dir);<a name="line.533"></a>
-<span class="sourceLineNo">534</span> deleted = deletion.act();<a name="line.534"></a>
-<span class="sourceLineNo">535</span> } catch (PathIsNotEmptyDirectoryException exception) {<a name="line.535"></a>
-<span class="sourceLineNo">536</span> // N.B. HDFS throws this exception when we try to delete a non-empty directory, but<a name="line.536"></a>
-<span class="sourceLineNo">537</span> // LocalFileSystem throws a bare IOException. So some test code will get the verbose<a name="line.537"></a>
-<span class="sourceLineNo">538</span> // message below.<a name="line.538"></a>
-<span class="sourceLineNo">539</span> LOG.debug("Couldn't delete '{}' yet because it isn't empty. Probably transient. " +<a name="line.539"></a>
-<span class="sourceLineNo">540</span> "exception details at TRACE.", dir);<a name="line.540"></a>
-<span class="sourceLineNo">541</span> LOG.trace("Couldn't delete '{}' yet because it isn't empty w/exception.", dir, exception);<a name="line.541"></a>
-<span class="sourceLineNo">542</span> deleted = false;<a name="line.542"></a>
-<span class="sourceLineNo">543</span> } catch (IOException ioe) {<a name="line.543"></a>
-<span class="sourceLineNo">544</span> LOG.info("Could not delete {} under {}. might be transient; we'll retry. if it keeps " +<a name="line.544"></a>
-<span class="sourceLineNo">545</span> "happening, use following exception when asking on mailing list.",<a name="line.545"></a>
-<span class="sourceLineNo">546</span> type, dir, ioe);<a name="line.546"></a>
-<span class="sourceLineNo">547</span> deleted = false;<a name="line.547"></a>
-<span class="sourceLineNo">548</span> }<a name="line.548"></a>
-<span class="sourceLineNo">549</span> LOG.trace("Finish deleting {} under {}, deleted=", type, dir, deleted);<a name="line.549"></a>
-<span class="sourceLineNo">550</span> return deleted;<a name="line.550"></a>
-<span class="sourceLineNo">551</span> }<a name="line.551"></a>
-<span class="sourceLineNo">552</span><a name="line.552"></a>
-<span class="sourceLineNo">553</span> /**<a name="line.553"></a>
-<span class="sourceLineNo">554</span> * Get cleaner results of subdirs.<a name="line.554"></a>
-<span class="sourceLineNo">555</span> * @param tasks subdirs cleaner tasks<a name="line.555"></a>
-<span class="sourceLineNo">556</span> * @return true if all subdirs deleted successfully, false for patial/all failures<a name="line.556"></a>
-<span class="sourceLineNo">557</span> * @throws IOException something happen during computation<a name="line.557"></a>
-<span class="sourceLineNo">558</span> */<a name="line.558"></a>
-<span class="sourceLineNo">559</span> private boolean getCleanResult(List<CleanerTask> tasks) throws IOException {<a name="line.559"></a>
-<span class="sourceLineNo">560</span> boolean cleaned = true;<a name="line.560"></a>
-<span class="sourceLineNo">561</span> try {<a name="line.561"></a>
-<span class="sourceLineNo">562</span> for (CleanerTask task : tasks) {<a name="line.562"></a>
-<span class="sourceLineNo">563</span> cleaned &= task.get();<a name="line.563"></a>
-<span class="sourceLineNo">564</span> }<a name="line.564"></a>
-<span class="sourceLineNo">565</span> } catch (InterruptedException | ExecutionException e) {<a name="line.565"></a>
-<span class="sourceLineNo">566</span> throw new IOException(e);<a name="line.566"></a>
-<span class="sourceLineNo">567</span> }<a name="line.567"></a>
-<span class="sourceLineNo">568</span> return cleaned;<a name="line.568"></a>
-<span class="sourceLineNo">569</span> }<a name="line.569"></a>
-<span class="sourceLineNo">570</span> }<a name="line.570"></a>
-<span class="sourceLineNo">571</span>}<a name="line.571"></a>
+<span class="sourceLineNo">275</span> if (LOG.isTraceEnabled()) {<a name="line.275"></a>
+<span class="sourceLineNo">276</span> LOG.trace("Cleaned all WALs under {}", oldFileDir);<a name="line.276"></a>
+<span class="sourceLineNo">277</span> }<a name="line.277"></a>
+<span class="sourceLineNo">278</span> } else {<a name="line.278"></a>
+<span class="sourceLineNo">279</span> if (LOG.isTraceEnabled()) {<a name="line.279"></a>
+<span class="sourceLineNo">280</span> LOG.trace("WALs outstanding under {}", oldFileDir);<a name="line.280"></a>
+<span class="sourceLineNo">281</span> }<a name="line.281"></a>
+<span class="sourceLineNo">282</span> }<a name="line.282"></a>
+<span class="sourceLineNo">283</span> } finally {<a name="line.283"></a>
+<span class="sourceLineNo">284</span> POOL.latchCountDown();<a name="line.284"></a>
+<span class="sourceLineNo">285</span> }<a name="line.285"></a>
+<span class="sourceLineNo">286</span> // After each cleaner chore, checks if received reconfigure notification while cleaning.<a name="line.286"></a>
+<span class="sourceLineNo">287</span> // First in cleaner turns off notification, to avoid another cleaner updating pool again.<a name="line.287"></a>
+<span class="sourceLineNo">288</span> if (POOL.reconfigNotification.compareAndSet(true, false)) {<a name="line.288"></a>
+<span class="sourceLineNo">289</span> // This cleaner is waiting for other cleaners finishing their jobs.<a name="line.289"></a>
+<span class="sourceLineNo">290</span> // To avoid missing next chore, only wait 0.8 * period, then shutdown.<a name="line.290"></a>
+<span class="sourceLineNo">291</span> POOL.updatePool((long) (0.8 * getTimeUnit().toMillis(getPeriod())));<a name="line.291"></a>
+<span class="sourceLineNo">292</span> }<a name="line.292"></a>
+<span class="sourceLineNo">293</span> } else {<a name="line.293"></a>
+<span class="sourceLineNo">294</span> if (LOG.isTraceEnabled()) {<a name="line.294"></a>
+<span class="sourceLineNo">295</span> LOG.trace("Cleaner chore disabled! Not cleaning.");<a name="line.295"></a>
+<span class="sourceLineNo">296</span> }<a name="line.296"></a>
+<span class="sourceLineNo">297</span> }<a name="line.297"></a>
+<span class="sourceLineNo">298</span> }<a name="line.298"></a>
+<span class="sourceLineNo">299</span><a name="line.299"></a>
+<span class="sourceLineNo">300</span> private void preRunCleaner() {<a name="line.300"></a>
+<span class="sourceLineNo">301</span> cleanersChain.forEach(FileCleanerDelegate::preClean);<a name="line.301"></a>
+<span class="sourceLineNo">302</span> }<a name="line.302"></a>
+<span class="sourceLineNo">303</span><a name="line.303"></a>
+<span class="sourceLineNo">304</span> public Boolean runCleaner() {<a name="line.304"></a>
+<span class="sourceLineNo">305</span> preRunCleaner();<a name="line.305"></a>
+<span class="sourceLineNo">306</span> CleanerTask task = new CleanerTask(this.oldFileDir, true);<a name="line.306"></a>
+<span class="sourceLineNo">307</span> POOL.submit(task);<a name="line.307"></a>
+<span class="sourceLineNo">308</span> return task.join();<a name="line.308"></a>
+<span class="sourceLineNo">309</span> }<a name="line.309"></a>
+<span class="sourceLineNo">310</span><a name="line.310"></a>
+<span class="sourceLineNo">311</span> /**<a name="line.311"></a>
+<span class="sourceLineNo">312</span> * Sort the given list in (descending) order of the space each element takes<a name="line.312"></a>
+<span class="sourceLineNo">313</span> * @param dirs the list to sort, element in it should be directory (not file)<a name="line.313"></a>
+<span class="sourceLineNo">314</span> */<a name="line.314"></a>
+<span class="sourceLineNo">315</span> private void sortByConsumedSpace(List<FileStatus> dirs) {<a name="line.315"></a>
+<span class="sourceLineNo">316</span> if (dirs == null || dirs.size() < 2) {<a name="line.316"></a>
+<span class="sourceLineNo">317</span> // no need to sort for empty or single directory<a name="line.317"></a>
+<span class="sourceLineNo">318</span> return;<a name="line.318"></a>
+<span class="sourceLineNo">319</span> }<a name="line.319"></a>
+<span class="sourceLineNo">320</span> dirs.sort(new Comparator<FileStatus>() {<a name="line.320"></a>
+<span class="sourceLineNo">321</span> HashMap<FileStatus, Long> directorySpaces = new HashMap<>();<a name="line.321"></a>
+<span class="sourceLineNo">322</span><a name="line.322"></a>
+<span class="sourceLineNo">323</span> @Override<a name="line.323"></a>
+<span class="sourceLineNo">324</span> public int compare(FileStatus f1, FileStatus f2) {<a name="line.324"></a>
+<span class="sourceLineNo">325</span> long f1ConsumedSpace = getSpace(f1);<a name="line.325"></a>
+<span class="sourceLineNo">326</span> long f2ConsumedSpace = getSpace(f2);<a name="line.326"></a>
+<span class="sourceLineNo">327</span> return Long.compare(f2ConsumedSpace, f1ConsumedSpace);<a name="line.327"></a>
+<span class="sourceLineNo">328</span> }<a name="line.328"></a>
+<span class="sourceLineNo">329</span><a name="line.329"></a>
+<span class="sourceLineNo">330</span> private long getSpace(FileStatus f) {<a name="line.330"></a>
+<span class="sourceLineNo">331</span> Long cached = directorySpaces.get(f);<a name="line.331"></a>
+<span class="sourceLineNo">332</span> if (cached != null) {<a name="line.332"></a>
+<span class="sourceLineNo">333</span> return cached;<a name="line.333"></a>
+<span class="sourceLineNo">334</span> }<a name="line.334"></a>
+<span class="sourceLineNo">335</span> try {<a name="line.335"></a>
+<span class="sourceLineNo">336</span> long space =<a name="line.336"></a>
+<span class="sourceLineNo">337</span> f.isDirectory() ? fs.getContentSummary(f.getPath()).getSpaceConsumed() : f.getLen();<a name="line.337"></a>
+<span class="sourceLineNo">338</span> directorySpaces.put(f, space);<a name="line.338"></a>
+<span class="sourceLineNo">339</span> return space;<a name="line.339"></a>
+<span class="sourceLineNo">340</span> } catch (IOException e) {<a name="line.340"></a>
+<span class="sourceLineNo">341</span> LOG.trace("Failed to get space consumed by path={}", f, e);<a name="line.341"></a>
+<span class="sourceLineNo">342</span> return -1;<a name="line.342"></a>
+<span class="sourceLineNo">343</span> }<a name="line.343"></a>
+<span class="sourceLineNo">344</span> }<a name="line.344"></a>
+<span class="sourceLineNo">345</span> });<a name="line.345"></a>
+<span class="sourceLineNo">346</span> }<a name="line.346"></a>
+<span class="sourceLineNo">347</span><a name="line.347"></a>
+<span class="sourceLineNo">348</span> /**<a name="line.348"></a>
+<span class="sourceLineNo">349</span> * Run the given files through each of the cleaners to see if it should be deleted, deleting it if<a name="line.349"></a>
+<span class="sourceLineNo">350</span> * necessary.<a name="line.350"></a>
+<span class="sourceLineNo">351</span> * @param files List of FileStatus for the files to check (and possibly delete)<a name="line.351"></a>
+<span class="sourceLineNo">352</span> * @return true iff successfully deleted all files<a name="line.352"></a>
+<span class="sourceLineNo">353</span> */<a name="line.353"></a>
+<span class="sourceLineNo">354</span> private boolean checkAndDeleteFiles(List<FileStatus> files) {<a name="line.354"></a>
+<span class="sourceLineNo">355</span> if (files == null) {<a name="line.355"></a>
+<span class="sourceLineNo">356</span> return true;<a name="line.356"></a>
+<span class="sourceLineNo">357</span> }<a name="line.357"></a>
+<span class="sourceLineNo">358</span><a name="line.358"></a>
+<span class="sourceLineNo">359</span> // first check to see if the path is valid<a name="line.359"></a>
+<span class="sourceLineNo">360</span> List<FileStatus> validFiles = Lists.newArrayListWithCapacity(files.size());<a name="line.360"></a>
+<span class="sourceLineNo">361</span> List<FileStatus> invalidFiles = Lists.newArrayList();<a name="line.361"></a>
+<span class="sourceLineNo">362</span> for (FileStatus file : files) {<a name="line.362"></a>
+<span class="sourceLineNo">363</span> if (validate(file.getPath())) {<a name="line.363"></a>
+<span class="sourceLineNo">364</span> validFiles.add(file);<a name="line.364"></a>
+<span class="sourceLineNo">365</span> } else {<a name="line.365"></a>
+<span class="sourceLineNo">366</span> LOG.warn("Found a wrongly formatted file: " + file.getPath() + " - will delete it.");<a name="line.366"></a>
+<span class="sourceLineNo">367</span> invalidFiles.add(file);<a name="line.367"></a>
+<span class="sourceLineNo">368</span> }<a name="line.368"></a>
+<span class="sourceLineNo">369</span> }<a name="line.369"></a>
+<span class="sourceLineNo">370</span><a name="line.370"></a>
+<span class="sourceLineNo">371</span> Iterable<FileStatus> deletableValidFiles = validFiles;<a name="line.371"></a>
+<span class="sourceLineNo">372</span> // check each of the cleaners for the valid files<a name="line.372"></a>
+<span class="sourceLineNo">373</span> for (T cleaner : cleanersChain) {<a name="line.373"></a>
+<span class="sourceLineNo">374</span> if (cleaner.isStopped() || this.getStopper().isStopped()) {<a name="line.374"></a>
+<span class="sourceLineNo">375</span> LOG.warn("A file cleaner" + this.getName() + " is stopped, won't delete any more files in:"<a name="line.375"></a>
+<span class="sourceLineNo">376</span> + this.oldFileDir);<a name="line.376"></a>
+<span class="sourceLineNo">377</span> return false;<a name="line.377"></a>
+<span class="sourceLineNo">378</span> }<a name="line.378"></a>
+<span class="sourceLineNo">379</span><a name="line.379"></a>
+<span class="sourceLineNo">380</span> Iterable<FileStatus> filteredFiles = cleaner.getDeletableFiles(deletableValidFiles);<a name="line.380"></a>
+<span class="sourceLineNo">381</span><a name="line.381"></a>
+<span class="sourceLineNo">382</span> // trace which cleaner is holding on to each file<a name="line.382"></a>
+<span class="sourceLineNo">383</span> if (LOG.isTraceEnabled()) {<a name="line.383"></a>
+<span class="sourceLineNo">384</span> ImmutableSet<FileStatus> filteredFileSet = ImmutableSet.copyOf(filteredFiles);<a name="line.384"></a>
+<span class="sourceLineNo">385</span> for (FileStatus file : deletableValidFiles) {<a name="line.385"></a>
+<span class="sourceLineNo">386</span> if (!filteredFileSet.contains(file)) {<a name="line.386"></a>
+<span class="sourceLineNo">387</span> LOG.trace(file.getPath() + " is not deletable according to:" + cleaner);<a name="line.387"></a>
+<span class="sourceLineNo">388</span> }<a name="line.388"></a>
+<span class="sourceLineNo">389</span> }<a name="line.389"></a>
+<span class="sourceLineNo">390</span> }<a name="line.390"></a>
+<span class="sourceLineNo">391</span><a name="line.391"></a>
+<span class="sourceLineNo">392</span> deletableValidFiles = filteredFiles;<a name="line.392"></a>
+<span class="sourceLineNo">393</span> }<a name="line.393"></a>
+<span class="sourceLineNo">394</span><a name="line.394"></a>
+<span class="sourceLineNo">395</span> Iterable<FileStatus> filesToDelete = Iterables.concat(invalidFiles, deletableValidFiles);<a name="line.395"></a>
+<span class="sourceLineNo">396</span> return deleteFiles(filesToDelete) == files.size();<a name="line.396"></a>
+<span class="sourceLineNo">397</span> }<a name="line.397"></a>
+<span class="sourceLineNo">398</span><a name="line.398"></a>
+<span class="sourceLineNo">399</span> /**<a name="line.399"></a>
+<span class="sourceLineNo">400</span> * Delete the given files<a name="line.400"></a>
+<span class="sourceLineNo">401</span> * @param filesToDelete files to delete<a name="line.401"></a>
+<span class="sourceLineNo">402</span> * @return number of deleted files<a name="line.402"></a>
+<span class="sourceLineNo">403</span> */<a name="line.403"></a>
+<span class="sourceLineNo">404</span> protected int deleteFiles(Iterable<FileStatus> filesToDelete) {<a name="line.404"></a>
+<span class="sourceLineNo">405</span> int deletedFileCount = 0;<a name="line.405"></a>
+<span class="sourceLineNo">406</span> for (FileStatus file : filesToDelete) {<a name="line.406"></a>
+<span class="sourceLineNo">407</span> Path filePath = file.getPath();<a name="line.407"></a>
+<span class="sourceLineNo">408</span> LOG.trace("Removing {} from archive", filePath);<a name="line.408"></a>
+<span class="sourceLineNo">409</span> try {<a name="line.409"></a>
+<span class="sourceLineNo">410</span> boolean success = this.fs.delete(filePath, false);<a name="line.410"></a>
+<span class="sourceLineNo">411</span> if (success) {<a name="line.411"></a>
+<span class="sourceLineNo">412</span> deletedFileCount++;<a name="line.412"></a>
+<span class="sourceLineNo">413</span> } else {<a name="line.413"></a>
+<span class="sourceLineNo">414</span> LOG.warn("Attempted to delete:" + filePath<a name="line.414"></a>
+<span class="sourceLineNo">415</span> + ", but couldn't. Run cleaner chain and attempt to delete on next pass.");<a name="line.415"></a>
+<span class="sourceLineNo">416</span> }<a name="line.416"></a>
+<span class="sourceLineNo">417</span> } catch (IOException e) {<a name="line.417"></a>
+<span class="sourceLineNo">418</span> e = e instanceof RemoteException ?<a name="line.418"></a>
+<span class="sourceLineNo">419</span> ((RemoteException)e).unwrapRemoteException() : e;<a name="line.419"></a>
+<span class="sourceLineNo">420</span> LOG.warn("Error while deleting: " + filePath, e);<a name="line.420"></a>
+<span class="sourceLineNo">421</span> }<a name="line.421"></a>
+<span class="sourceLineNo">422</span> }<a name="line.422"></a>
+<span class="sourceLineNo">423</span> return deletedFileCount;<a name="line.423"></a>
+<span class="sourceLineNo">424</span> }<a name="line.424"></a>
+<span class="sourceLineNo">425</span><a name="line.425"></a>
+<span class="sourceLineNo">426</span> @Override<a name="line.426"></a>
+<span class="sourceLineNo">427</span> public synchronized void cleanup() {<a name="line.427"></a>
+<span class="sourceLineNo">428</span> for (T lc : this.cleanersChain) {<a name="line.428"></a>
+<span class="sourceLineNo">429</span> try {<a name="line.429"></a>
+<span class="sourceLineNo">430</span> lc.stop("Exiting");<a name="line.430"></a>
+<span class="sourceLineNo">431</span> } catch (Throwable t) {<a name="line.431"></a>
+<span class="sourceLineNo">432</span> LOG.warn("Stopping", t);<a name="line.432"></a>
+<span class="sourceLineNo">433</span> }<a name="line.433"></a>
+<span class="sourceLineNo">434</span> }<a name="line.434"></a>
+<span class="sourceLineNo">435</span> }<a name="line.435"></a>
+<span class="sourceLineNo">436</span><a name="line.436"></a>
+<span class="sourceLineNo">437</span> @VisibleForTesting<a name="line.437"></a>
+<span class="sourceLineNo">438</span> int getChorePoolSize() {<a name="line.438"></a>
+<span class="sourceLineNo">439</span> return POOL.size;<a name="line.439"></a>
+<span class="sourceLineNo">440</span> }<a name="line.440"></a>
+<span class="sourceLineNo">441</span><a name="line.441"></a>
+<span class="sourceLineNo">442</span> /**<a name="line.442"></a>
+<span class="sourceLineNo">443</span> * @param enabled<a name="line.443"></a>
+<span class="sourceLineNo">444</span> */<a name="line.444"></a>
+<span class="sourceLineNo">445</span> public boolean setEnabled(final boolean enabled) {<a name="line.445"></a>
+<span class="sourceLineNo">446</span> return this.enabled.getAndSet(enabled);<a name="line.446"></a>
+<span class="sourceLineNo">447</span> }<a name="line.447"></a>
+<span class="sourceLineNo">448</span><a name="line.448"></a>
+<span class="sourceLineNo">449</span> public boolean getEnabled() { return this.enabled.get();<a name="line.449"></a>
+<span class="sourceLineNo">450</span> }<a name="line.450"></a>
+<span class="sourceLineNo">451</span><a name="line.451"></a>
+<span class="sourceLineNo">452</span> private interface Action<T> {<a name="line.452"></a>
+<span class="sourceLineNo">453</span> T act() throws IOException;<a name="line.453"></a>
+<span class="sourceLineNo">454</span> }<a name="line.454"></a>
+<span class="sourceLineNo">455</span><a name="line.455"></a>
+<span class="sourceLineNo">456</span> /**<a name="line.456"></a>
+<span class="sourceLineNo">457</span> * Attemps to clean up a directory, its subdirectories, and files.<a name="line.457"></a>
+<span class="sourceLineNo">458</span> * Return value is true if everything was deleted. false on partial / total failures.<a name="line.458"></a>
+<span class="sourceLineNo">459</span> */<a name="line.459"></a>
+<span class="sourceLineNo">460</span> private class CleanerTask extends RecursiveTask<Boolean> {<a name="line.460"></a>
+<span class="sourceLineNo">461</span> private final Path dir;<a name="line.461"></a>
+<span class="sourceLineNo">462</span> private final boolean root;<a name="line.462"></a>
+<span class="sourceLineNo">463</span><a name="line.463"></a>
+<span class="sourceLineNo">464</span> CleanerTask(final FileStatus dir, final boolean root) {<a name="line.464"></a>
+<span class="sourceLineNo">465</span> this(dir.getPath(), root);<a name="line.465"></a>
+<span class="sourceLineNo">466</span> }<a name="line.466"></a>
+<span class="sourceLineNo">467</span><a name="line.467"></a>
+<span class="sourceLineNo">468</span> CleanerTask(final Path dir, final boolean root) {<a name="line.468"></a>
+<span class="sourceLineNo">469</span> this.dir = dir;<a name="line.469"></a>
+<span class="sourceLineNo">470</span> this.root = root;<a name="line.470"></a>
+<span class="sourceLineNo">471</span> }<a name="line.471"></a>
+<span class="sourceLineNo">472</span><a name="line.472"></a>
+<span class="sourceLineNo">473</span> @Override<a name="line.473"></a>
+<span class="sourceLineNo">474</span> protected Boolean compute() {<a name="line.474"></a>
+<span class="sourceLineNo">475</span> LOG.debug("Cleaning under {}", dir);<a name="line.475"></a>
+<span class="sourceLineNo">476</span> List<FileStatus> subDirs;<a name="line.476"></a>
+<span class="sourceLineNo">477</span> List<FileStatus> files;<a name="line.477"></a>
+<span class="sourceLineNo">478</span> try {<a name="line.478"></a>
+<span class="sourceLineNo">479</span> // if dir doesn't exist, we'll get null back for both of these<a name="line.479"></a>
+<span class="sourceLineNo">480</span> // which will fall through to succeeding.<a name="line.480"></a>
+<span class="sourceLineNo">481</span> subDirs = getFilteredStatus(status -> status.isDirectory());<a name="line.481"></a>
+<span class="sourceLineNo">482</span> files = getFilteredStatus(status -> status.isFile());<a name="line.482"></a>
+<span class="sourceLineNo">483</span> } catch (IOException ioe) {<a name="line.483"></a>
+<span class="sourceLineNo">484</span> LOG.warn("failed to get FileStatus for contents of '{}'", dir, ioe);<a name="line.484"></a>
+<span class="sourceLineNo">485</span> return false;<a name="line.485"></a>
+<span class="sourceLineNo">486</span> }<a name="line.486"></a>
+<span class="sourceLineNo">487</span><a name="line.487"></a>
+<span class="sourceLineNo">488</span> boolean nullSubDirs = subDirs == null;<a name="line.488"></a>
+<span class="sourceLineNo">489</span> if (nullSubDirs) {<a name="line.489"></a>
+<span class="sourceLineNo">490</span> LOG.trace("There is no subdir under {}", dir);<a name="line.490"></a>
+<span class="sourceLineNo">491</span> }<a name="line.491"></a>
+<span class="sourceLineNo">492</span> if (files == null) {<a name="line.492"></a>
+<span class="sourceLineNo">493</span> LOG.trace("There is no file under {}", dir);<a name="line.493"></a>
+<span class="sourceLineNo">494</span> }<a name="line.494"></a>
+<span class="sourceLineNo">495</span><a name="line.495"></a>
+<span class="sourceLineNo">496</span> int capacity = nullSubDirs ? 0 : subDirs.size();<a name="line.496"></a>
+<span class="sourceLineNo">497</span> List<CleanerTask> tasks = Lists.newArrayListWithCapacity(capacity);<a name="line.497"></a>
+<span class="sourceLineNo">498</span> if (!nullSubDirs) {<a name="line.498"></a>
+<span class="sourceLineNo">499</span> sortByConsumedSpace(subDirs);<a name="line.499"></a>
+<span class="sourceLineNo">500</span> for (FileStatus subdir : subDirs) {<a name="line.500"></a>
+<span class="sourceLineNo">501</span> CleanerTask task = new CleanerTask(subdir, false);<a name="line.501"></a>
+<span class="sourceLineNo">502</span> tasks.add(task);<a name="line.502"></a>
+<span class="sourceLineNo">503</span> task.fork();<a name="line.503"></a>
+<span class="sourceLineNo">504</span> }<a name="line.504"></a>
+<span class="sourceLineNo">505</span> }<a name="line.505"></a>
+<span class="sourceLineNo">506</span><a name="line.506"></a>
+<span class="sourceLineNo">507</span> boolean result = true;<a name="line.507"></a>
+<span class="sourceLineNo">508</span> result &= deleteAction(() -> checkAndDeleteFiles(files), "files");<a name="line.508"></a>
+<span class="sourceLineNo">509</span> result &= deleteAction(() -> getCleanResult(tasks), "subdirs");<a name="line.509"></a>
+<span class="sourceLineNo">510</span> // if and only if files and subdirs under current dir are deleted successfully, and<a name="line.510"></a>
+<span class="sourceLineNo">511</span> // it is not the root dir, then task will try to delete it.<a name="line.511"></a>
+<span class="sourceLineNo">512</span> if (result && !root) {<a name="line.512"></a>
+<span class="sourceLineNo">513</span> result &= deleteAction(() -> fs.delete(dir, false), "dir");<a name="line.513"></a>
+<span class="sourceLineNo">514</span> }<a name="line.514"></a>
+<span class="sourceLineNo">515</span> return result;<a name="line.515"></a>
+<span class="sourceLineNo">516</span> }<a name="line.516"></a>
+<span class="sourceLineNo">517</span><a name="line.517"></a>
+<span class="sourceLineNo">518</span> /**<a name="line.518"></a>
+<span class="sourceLineNo">519</span> * Get FileStatus with filter.<a name="line.519"></a>
+<span class="sourceLineNo">520</span> * Pay attention that FSUtils #listStatusWithStatusFilter would return null,<a name="line.520"></a>
+<span class="sourceLineNo">521</span> * even though status is empty but not null.<a name="line.521"></a>
+<span class="sourceLineNo">522</span> * @param function a filter function<a name="line.522"></a>
+<span class="sourceLineNo">523</span> * @return filtered FileStatus or null if dir doesn't exist<a name="line.523"></a>
+<span class="sourceLineNo">524</span> * @throws IOException if there's an error other than dir not existing<a name="line.524"></a>
+<span class="sourceLineNo">525</span> */<a name="line.525"></a>
+<span class="sourceLineNo">526</span> private List<FileStatus> getFilteredStatus(Predicate<FileStatus> function) throws IOException {<a name="line.526"></a>
+<span class="sourceLineNo">527</span> return FSUtils.listStatusWithStatusFilter(fs, dir, status -> function.test(status));<a name="line.527"></a>
+<span class="sourceLineNo">528</span> }<a name="line.528"></a>
+<span class="sourceLineNo">529</span><a name="line.529"></a>
+<span class="sourceLineNo">530</span> /**<a name="line.530"></a>
+<span class="sourceLineNo">531</span> * Perform a delete on a specified type.<a name="line.531"></a>
+<span class="sourceLineNo">532</span> * @param deletion a delete<a name="line.532"></a>
+<span class="sourceLineNo">533</span> * @param type possible values are 'files', 'subdirs', 'dirs'<a name="line.533"></a>
+<span class="sourceLineNo">534</span> * @return true if it deleted successfully, false otherwise<a name="line.534"></a>
+<span class="sourceLineNo">535</span> */<a name="line.535"></a>
+<span class="sourceLineNo">536</span> private boolean deleteAction(Action<Boolean> deletion, String type) {<a name="line.536"></a>
+<span class="sourceLineNo">537</span> boolean deleted;<a name="line.537"></a>
+<span class="sourceLineNo">538</span> try {<a name="line.538"></a>
+<span class="sourceLineNo">539</span> LOG.trace("Start deleting {} under {}", type, dir);<a name="line.539"></a>
+<span class="sourceLineNo">540</span> deleted = deletion.act();<a name="line.540"></a>
+<span class="sourceLineNo">541</span> } catch (PathIsNotEmptyDirectoryException exception) {<a name="line.541"></a>
+<span class="sourceLineNo">542</span> // N.B. HDFS throws this exception when we try to delete a non-empty directory, but<a name="line.542"></a>
+<span class="sourceLineNo">543</span> // LocalFileSystem throws a bare IOException. So some test code will get the verbose<a name="line.543"></a>
+<span class="sourceLineNo">544</span> // message below.<a name="line.544"></a>
+<span class="sourceLineNo">545</span> LOG.debug("Couldn't delete '{}' yet because it isn't empty. Probably transient. " +<a name="line.545"></a>
+<span class="sourceLineNo">546</span> "exception details at TRACE.", dir);<a name="line.546"></a>
+<span class="sourceLineNo">547</span> LOG.trace("Couldn't delete '{}' yet because it isn't empty w/exception.", dir, exception);<a name="line.547"></a>
+<span class="sourceLineNo">548</span> deleted = false;<a name="line.548"></a>
+<span class="sourceLineNo">549</span> } catch (IOException ioe) {<a name="line.549"></a>
+<span class="sourceLineNo">550</span> LOG.info("Could not delete {} under {}. might be transient; we'll retry. if it keeps " +<a name="line.550"></a>
+<span class="sourceLineNo">551</span> "happening, use following exception when asking on mailing list.",<a name="line.551"></a>
+<span class="sourceLineNo">552</span> type, dir, ioe);<a name="line.552"></a>
+<span class="sourceLineNo">553</span> deleted = false;<a name="line.553"></a>
+<span class="sourceLineNo">554</span> }<a name="line.554"></a>
+<span class="sourceLineNo">555</span> LOG.trace("Finish deleting {} under {}, deleted=", type, dir, deleted);<a name="line.555"></a>
+<span class="sourceLineNo">556</span> return deleted;<a name="line.556"></a>
+<span class="sourceLineNo">557</span> }<a name="line.557"></a>
+<span class="sourceLineNo">558</span><a name="line.558"></a>
+<span class="sourceLineNo">559</span> /**<a name="line.559"></a>
+<span class="sourceLineNo">560</span> * Get cleaner results of subdirs.<a name="line.560"></a>
+<span class="sourceLineNo">561</span> * @param tasks subdirs cleaner tasks<a name="line.561"></a>
+<span class="sourceLineNo">562</span> * @return true if all subdirs deleted successfully, false for patial/all failures<a name="line.562"></a>
+<span class="sourceLineNo">563</span> * @throws IOException something happen during computation<a name="line.563"></a>
+<span class="sourceLineNo">564</span> */<a name="line.564"></a>
+<span class="sourceLineNo">565</span> private boolean getCleanResult(List<CleanerTask> tasks) throws IOException {<a name="line.565"></a>
+<span class="sourceLineNo">566</span> boolean cleaned = true;<a name="line.566"></a>
+<span class="sourceLineNo">567</span> try {<a name="line.567"></a>
+<span class="sourceLineNo">568</span> for (CleanerTask task : tasks) {<a name="line.568"></a>
+<span class="sourceLineNo">569</span> cleaned &= task.get();<a name="line.569"></a>
+<span class="sourceLineNo">570</span> }<a name="line.570"></a>
+<span class="sourceLineNo">571</span> } catch (InterruptedException | ExecutionException e) {<a name="line.571"></a>
+<span class="sourceLineNo">572</span> throw new IOException(e);<a name="line.572"></a>
+<span class="sourceLineNo">573</span> }<a name="line.573"></a>
+<span class="sourceLineNo">574</span> return cleaned;<a name="line.574"></a>
+<span class="sourceLineNo">575</span> }<a name="line.575"></a>
+<span class="sourceLineNo">576</span> }<a name="line.576"></a>
+<span class="sourceLineNo">577</span>}<a name="line.577"></a>