hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Gary Helmling (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-16788) Race in compacted file deletion between HStore close() and closeAndArchiveCompactedFiles()
Date Mon, 10 Oct 2016 21:25:20 GMT

    [ https://issues.apache.org/jira/browse/HBASE-16788?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15563563#comment-15563563
] 

Gary Helmling commented on HBASE-16788:
---------------------------------------

{quote}
If I understand the code correctly, it would take longer for the close() to complete when
concurrent CompactedHFilesDischargeHandler operation gets the archiveLock first.
If this is not a concern, I am fine with your patch.
{quote}

It's true that close() may be blocked by the discharge chore thread if it is holding the archiveLock.
 But whether the work for archiving compacted HFiles is being done by the discharge thread
or by close(), the same work needs to be done before close() can complete.  So I don't expect
this to appreciably change the time taken by close().  It just means that if close() is blocked
by the discharger, it should be able to skip over the archive step once it gets to run.

> Race in compacted file deletion between HStore close() and closeAndArchiveCompactedFiles()
> ------------------------------------------------------------------------------------------
>
>                 Key: HBASE-16788
>                 URL: https://issues.apache.org/jira/browse/HBASE-16788
>             Project: HBase
>          Issue Type: Bug
>          Components: regionserver
>    Affects Versions: 1.3.0
>            Reporter: Gary Helmling
>            Assignee: Gary Helmling
>            Priority: Blocker
>         Attachments: 16788-suggest.v2, HBASE-16788.001.patch, HBASE-16788.002.patch,
HBASE-16788_1.patch
>
>
> HBASE-13082 changed the way that compacted files are archived from being done inline
on compaction completion to an async cleanup by the CompactedHFilesDischarger chore.  It looks
like the changes to HStore to support this introduced a race condition in the compacted HFile
archiving.
> In the following sequence, we can wind up with two separate threads trying to archive
the same HFiles, causing a regionserver abort:
> # compaction completes normally and the compacted files are added to {{compactedfiles}}
in HStore's DefaultStoreFileManager
> # *threadA*: CompactedHFilesDischargeHandler runs in a RS executor service, calling closeAndArchiveCompactedFiles()
> ## obtains HStore readlock
> ## gets a copy of compactedfiles
> ## releases readlock
> # *threadB*: calls HStore.close() as part of region close
> ## obtains HStore writelock
> ## calls DefaultStoreFileManager.clearCompactedfiles(), getting a copy of same compactedfiles
> # *threadA*: calls HStore.removeCompactedfiles(compactedfiles)
> ## archives files in {compactedfiles} in HRegionFileSystem.removeStoreFiles()
> ## call HStore.clearCompactedFiles()
> ## waits on write lock
> # *threadB*: continues with close()
> ## calls removeCompactedfiles(compactedfiles)
> ## calls HRegionFIleSystem.removeStoreFiles() -> HFileArchiver.archiveStoreFiles()
> ## receives FileNotFoundException because the files have already been archived by threadA
> ## throws IOException
> # RS aborts
> I think the combination of fetching the compactedfiles list and removing the files needs
to be covered by locking.  Options I see are:
> * Modify HStore.closeAndArchiveCompactedFiles(): use writelock instead of readlock and
move the call to removeCompactedfiles() inside the lock.  This means the read operations will
be blocked while the files are being archived, which is bad.
> * Synchronize closeAndArchiveCompactedFiles() and modify close() to call it instead of
calling removeCompactedfiles() directly
> * Add a separate lock for compacted files removal and use in closeAndArchiveCompactedFiles()
and close()



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message