[Bugs] [Bug 1188196] Change order of translators in brick

[bugzilla at redhat.com]

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



--- Comment #3 from Anand Avati <aavati at redhat.com> ---
COMMIT: http://review.gluster.org/9751 committed in master by Krishnan
Parthasarathi (kparthas at redhat.com) 
------
commit d1397dbd7d6cdbd2d81d5d36d608b6175d449db4
Author: Vijay Bellur <vbellur at redhat.com>
Date:   Wed Feb 25 20:34:41 2015 +0530

    mgmt/glusterd: Refactor brick graph generation in volgen.

    This commit does the following:

    1. Adds several new functions for generation of brick xlator units
       in a volgen. Each such function takes care of generation of only
       one xlator in volgen.

    2. A new table, server_graph_table, links all individual graph generation
       functions together.  The order of xlator function generators in the
       table determines the topology of the brick graph.

    3. server_graph_builder() invokes individual graph generators by walking
       through server_graph_table. Addition of debug xlators into the brick
       graph is also handled by this walk. As a result, a lot of cruft that
       is present in the exisiting implementation of this function gets
       cleaned up.

    4. get_server_xlator() now makes use of server_graph_table to determine
       whether a xlator key corresponds to a server xlator or not.

    Change-Id: I46bb6e331544150302eb5b33c4007917aff2586d
    BUG: 1188196
    Signed-off-by: Vijay Bellur <vbellur at redhat.com>
    Reviewed-on: http://review.gluster.org/9751
    Tested-by: Gluster Build System <jenkins at build.gluster.com>
    Reviewed-by: Krishnan Parthasarathi <kparthas at redhat.com>
    Tested-by: Krishnan Parthasarathi <kparthas 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=AtxJJlg0jV&a=cc_unsubscribe