[Bugs] [Bug 1476719] glusterd: flow in glusterd_validate_quorum() could be streamlined

bugzilla at redhat.com bugzilla at redhat.com
Tue Aug 1 10:11:41 UTC 2017


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



--- Comment #2 from Worker Ant <bugzilla-bot at gluster.org> ---
COMMIT: https://review.gluster.org/17924 committed in master by Atin Mukherjee
(amukherj at redhat.com) 
------
commit 3023c4bd7651d4503830c36676cd08864e315cdf
Author: Michael Adam <obnox at samba.org>
Date:   Tue Jun 13 14:44:39 2017 +0200

    glusterd: streamline logic flow in glusterd_validate_quorum()

    Make an earlier exit when the volume is not of server quorum.
    Thereby it spares the actual quorum calculation in this case.
    This also increases the overall readability of the function.

    The patch is best seen with the --patience diff option to
    understand what it does (e.g. "git show --patience").

    Change-Id: Ifce50bc3f73d79d3d6226473661a83696d65149a
    BUG: 1476719
    Signed-off-by: Michael Adam <obnox at samba.org>
    Reviewed-on: https://review.gluster.org/17924
    Smoke: Gluster Build System <jenkins at build.gluster.org>
    Tested-by: Atin Mukherjee <amukherj at redhat.com>
    CentOS-regression: Gluster Build System <jenkins at build.gluster.org>
    Reviewed-by: Amar Tumballi <amarts at redhat.com>
    Reviewed-by: Atin Mukherjee <amukherj 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=HZ1XjLt7GL&a=cc_unsubscribe


More information about the Bugs mailing list