[
https://issues.apache.org/jira/browse/HADOOP-8230?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13266700#comment-13266700
]
Suresh Srinivas commented on HADOOP-8230:
-----------------------------------------
bq. Wrt #2 personally I don't think we should allow people to disable durable
sync as that can result in data loss for people running HBase. See HADOOP-8230
for more info. I'm open to having an option to disable durable sync if you
think that use case is important.
There are installations where HBase is not used and sync was disabled. Now this
patch has removed that option. When an installation upgrades to a release with
this patch, suddenly sync is enabled and there is no way to disable it.
bq. (1) there are tests that are using append not to test append per se but for
the side effects and we'd lose sync test coverage by removing those tests and
(2) per the description we're keeping the append code path in case someone
wants to fix the data loss issues in which case it makes sense to keep the test
coverage as well.
For testing sync, with this patch, since it is enabled by default, you do not
need the flag right?
> Enable sync by default and disable append
> -----------------------------------------
>
> Key: HADOOP-8230
> URL: https://issues.apache.org/jira/browse/HADOOP-8230
> Project: Hadoop Common
> Issue Type: Improvement
> Affects Versions: 1.0.0
> Reporter: Eli Collins
> Assignee: Eli Collins
> Fix For: 1.1.0
>
> Attachments: hadoop-8230.txt
>
>
> Per HDFS-3120 for 1.x let's:
> - Always enable the sync path, which is currently only enabled if
> dfs.support.append is set
> - Remove the dfs.support.append configuration option. We'll keep the code
> paths though in case we ever fix append on branch-1, in which case we can add
> the config option back
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
