summaryrefslogtreecommitdiffstats
path: root/xlators/features/changelog/src
diff options
context:
space:
mode:
authorAjeet Jha <ajha@redhat.com>2014-06-02 14:49:07 +0530
committerVenky Shankar <vshankar@redhat.com>2014-06-10 06:37:57 -0700
commit94260c93a144b52964a27ddc9eaef869a3e8f354 (patch)
tree116590542f6b7bcdf42d04e8df89b68dbf12fc19 /xlators/features/changelog/src
parentb12b15a186714ec4d5639b778f15e075c833ce55 (diff)
features/changelog: skipping empty changelogs in history
Empty changelgs are unlinked and htime-update is bypassed while publishing partially written changelog. Change-Id: I1533ad431e4fbf592d2f38837fce782932d01e9c BUG: 1105466 Signed-off-by: Ajeet Jha <ajha@redhat.com> Reviewed-on: http://review.gluster.org/7949 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Venky Shankar <vshankar@redhat.com> Tested-by: Venky Shankar <vshankar@redhat.com>
Diffstat (limited to 'xlators/features/changelog/src')
-rw-r--r--xlators/features/changelog/src/changelog-helpers.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/xlators/features/changelog/src/changelog-helpers.c b/xlators/features/changelog/src/changelog-helpers.c
index 984106b75e6..8efe7dc4695 100644
--- a/xlators/features/changelog/src/changelog-helpers.c
+++ b/xlators/features/changelog/src/changelog-helpers.c
@@ -166,7 +166,8 @@ htime_update (xlator_t *this,
if (sys_fsetxattr (priv->htime_fd, HTIME_KEY, x_value,
strlen (x_value), XATTR_REPLACE)) {
gf_log (this->name, GF_LOG_ERROR,
- "Htime xattr updation failed");
+ "Htime xattr updation failed, "
+ "reason (%s)",strerror (errno));
goto out;
}
@@ -209,6 +210,7 @@ changelog_rollover_changelog (xlator_t *this,
if (ret && (errno == ENOENT)) {
ret = 0;
+ goto out;
}
if (ret) {