[Bugs] [Bug 1591580] Remove code duplication in protocol/client

bugzilla at redhat.com bugzilla at redhat.com
Mon Jun 18 03:45:48 UTC 2018


https://bugzilla.redhat.com/show_bug.cgi?id=1591580

Worker Ant <bugzilla-bot at gluster.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|POST                        |MODIFIED



--- Comment #2 from Worker Ant <bugzilla-bot at gluster.org> ---
COMMIT: https://review.gluster.org/20276 committed in master by "Krutika
Dhananjay" <kdhananj at redhat.com> with a commit message- protocol/client: Remove
code duplication

client_submit_vec_request() which is used by WRITEV, and PUT
and client_submit_request() used by the rest of the fops have
almost similar code. However, there have been some more checks
- such as whether setvolume was successful or not, and one more
that is send-gid-specific - that have been missed out in the vectored
version of the function.

This patch fixes this code duplication.

Change-Id: I363a28eeead6219cb1009dc831538153e8bd7d40
fixes: bz#1591580
Signed-off-by: Krutika Dhananjay <kdhananj at redhat.com>

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=C0PQVXUZhg&a=cc_unsubscribe


More information about the Bugs mailing list