3604df
From b52044163b8e971b1f9c2e7bd2d78a9dfcce1fe0 Mon Sep 17 00:00:00 2001
3604df
From: Pranith Kumar K <pkarampu@redhat.com>
3604df
Date: Tue, 7 Mar 2017 14:32:18 +0530
3604df
Subject: [PATCH 303/303] cluster/ec: Don't mark dirty on entry/meta ops in
3604df
 query-info
3604df
3604df
	Backport of https://review.gluster.org/16865
3604df
3604df
We wanted to mark dirty for metadata/entry operations
3604df
whenever query-info is set and info is not yet there because we
3604df
are anyway sending xattrop over the network. But this is causing
3604df
25% regression from 3.8.8 so removing this optimization
3604df
3604df
BUG: 1408655
3604df
Change-Id: Iabb0b64bd4d3623688790e4b67e5c20b4da977a1
3604df
Signed-off-by: Pranith Kumar K <pkarampu@redhat.com>
3604df
Reviewed-on: https://code.engineering.redhat.com/gerrit/99547
3604df
---
3604df
 xlators/cluster/ec/src/ec-common.c | 6 ------
3604df
 1 file changed, 6 deletions(-)
3604df
3604df
diff --git a/xlators/cluster/ec/src/ec-common.c b/xlators/cluster/ec/src/ec-common.c
3604df
index 647e750..f54084d 100644
3604df
--- a/xlators/cluster/ec/src/ec-common.c
3604df
+++ b/xlators/cluster/ec/src/ec-common.c
3604df
@@ -1130,12 +1130,6 @@ void ec_get_size_version(ec_lock_link_t *link)
3604df
         !(ec->node_mask & ~link->lock->good_mask) && !ec_is_data_fop (fop->id))
3604df
             link->optimistic_changelog = _gf_true;
3604df
 
3604df
-    /* If ctx->have_info is false and lock->query is true, it means that we'll
3604df
-     * send the xattrop anyway, so we can use it to update dirty counts, even
3604df
-     * if it's not necessary to do it right now. */
3604df
-    if (!ctx->have_info && lock->query)
3604df
-            link->optimistic_changelog = _gf_false;
3604df
-
3604df
     set_dirty = ec_set_dirty_flag (link, ctx, dirty);
3604df
 
3604df
     /* If ec metadata has already been retrieved, do not try again. */
3604df
-- 
3604df
2.9.3
3604df