[ https://issues.apache.org/jira/browse/HBASE-17437?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15850585#comment-15850585 ]
Umesh Agashe commented on HBASE-17437: -------------------------------------- +1 for the branch-1 patch. Changes lgtm. Found a few variable names which can me changed to wal, but as the patch is already into master, no need to change. If tests pass, ship it! Thank you [~zyork] for backporting the patch! > Support specifying a WAL directory outside of the root directory > ---------------------------------------------------------------- > > Key: HBASE-17437 > URL: https://issues.apache.org/jira/browse/HBASE-17437 > Project: HBase > Issue Type: Improvement > Components: Filesystem Integration, wal > Affects Versions: 1.2.4 > Reporter: Yishan Yang > Assignee: Zach York > Labels: patch > Fix For: 2.0.0, 1.4.0 > > Attachments: HBASE-17437.branch-1.001.patch, > hbase-17437-branch-1.2.patch, HBASE-17437.master.001.patch, > HBASE-17437.master.002.patch, HBASE-17437.master.003.patch, > HBASE-17437.master.004.patch, HBASE-17437.master.005.patch, > HBASE-17437.master.006.patch, HBASE-17437.master.007.patch, > HBASE-17437.master.008.patch, HBASE-17437.master.009.patch, > HBASE-17437.master.010.patch, HBASE-17437.master.011.patch, > HBASE-17437.master.012.patch, hbase-17437-master.patch > > > Currently, the WAL and the StoreFiles need to be on the same FileSystem. Some > FileSystems (such as Amazon S3) don’t support append or consistent writes. > These two properties are imperative for the WAL in order to avoid loss of > writes. However, StoreFiles don’t necessarily need the same consistency > guarantees (since writes are cached locally and if writes fail, they can > always be replayed from the WAL). > > This JIRA aims to allow users to configure a log directory (for WALs) that is > outside of the root directory or even in a different FileSystem. The default > value will still put the log directory under the root directory. -- This message was sent by Atlassian JIRA (v6.3.15#6346)