Bug 772468 - trivial: gluster_pmap_actors should only use GF_PMAP_* procedures
Summary: trivial: gluster_pmap_actors should only use GF_PMAP_* procedures
Keywords:
Status: CLOSED UPSTREAM
Alias: None
Product: GlusterFS
Classification: Community
Component: rpc
Version: pre-release
Hardware: Unspecified
OS: Unspecified
low
low
Target Milestone: ---
Assignee: Niels de Vos
QA Contact:
URL: http://review.gluster.com/#change,2603
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-01-08 16:15 UTC by Niels de Vos
Modified: 2012-02-18 10:57 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-02-18 10:57:45 UTC
Regression: ---
Mount Type: ---
Documentation: ---
CRM:
Verified Versions:


Attachments (Terms of Use)

Description Niels de Vos 2012-01-08 16:15:51 UTC
GF_HNDSK_NULL is not suitable for gluster_pmap_actors, instead use
GF_PMAP_NULL. The actual value (0) is not different for these, so there
is no fuctional change.

This can be found in xlators/mgmt/glusterd/src/glusterd-pmap.c.

Comment 1 Niels de Vos 2012-01-08 16:21:01 UTC
http://review.gluster.com/#change,2603 contains the patch

Comment 2 Anand Avati 2012-01-14 12:52:37 UTC
CHANGE: http://review.gluster.com/2603 (trivial: gluster_pmap_actors should only use GF_PMAP_* procedures) merged in master by Anand Avati (avati)

Comment 3 Amar Tumballi 2012-02-18 10:57:45 UTC
code cleanup type of bug, no test cases to unit test, other than opening in a editor and verifying the patch got applied :-)


Note You need to log in before you can comment on or make changes to this bug.