[Gluster-devel] Fwd: Change in glusterfs[release-3.3]: client/protocol : Changes in client3_1_getxattr()

Rajesh Amaravathi rajesh at redhat.com
Thu May 17 06:42:56 UTC 2012


+1

Regards, 
Rajesh Amaravathi, 
Software Engineer, GlusterFS 
RedHat Inc. 

----- Original Message -----
From: "John Mark Walker" <johnmark at redhat.com>
To: gluster-devel at nongnu.org
Sent: Thursday, May 17, 2012 12:05:10 PM
Subject: [Gluster-devel] Fwd: Change in glusterfs[release-3.3]:	client/protocol : Changes in client3_1_getxattr()

I was thinking about sending these gerritt notifications to gluster-devel by default - what do y'all think? 

-JM


----- Forwarded Message -----
From: "Kaushal M (Code Review)" <root at dev.gluster.com>
Sent: Wednesday, May 16, 2012 11:32:26 PM
Subject: Change in glusterfs[release-3.3]: client/protocol : Changes in client3_1_getxattr()

Kaushal M has uploaded a new change for review.

Change subject: client/protocol : Changes in client3_1_getxattr()
......................................................................

client/protocol : Changes in client3_1_getxattr()

Backporting change 1d02db63ae from master.

Copy args->loc to local->loc in client3_1_getxattr(). This prevents logs with
"(null) (--)" in client3_1_getxattr_cbk().
Also save args->name in local->name and print it in the log as well.

BUG: 812199
Change-Id: I5419f6a244de93dd1a96ac8e229be3ecdc9f456e
Signed-off-by: Kaushal M <kaushal at redhat.com>
---
M xlators/protocol/client/src/client-helpers.c
M xlators/protocol/client/src/client.h
M xlators/protocol/client/src/client3_1-fops.c
3 files changed, 11 insertions(+), 2 deletions(-)


  git pull ssh://*/glusterfs refs/changes/50/3350/1
--
To view, visit http://review.gluster.com/3350
To unsubscribe, visit http://review.gluster.com/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I5419f6a244de93dd1a96ac8e229be3ecdc9f456e
Gerrit-PatchSet: 1
Gerrit-Project: glusterfs
Gerrit-Branch: release-3.3
Gerrit-Owner: Kaushal M <kaushal at redhat.com>

_______________________________________________
Gluster-devel mailing list
Gluster-devel at nongnu.org
https://lists.nongnu.org/mailman/listinfo/gluster-devel




More information about the Gluster-devel mailing list