[Gluster-devel] GF_FOP_IPC changes
Niels de Vos
ndevos at redhat.com
Mon Jun 29 14:48:20 UTC 2015
On Wed, Jun 24, 2015 at 07:44:13PM +0530, Soumya Koduri wrote:
>
>
> On 06/24/2015 10:14 AM, Krishnan Parthasarathi wrote:
> >
> >
> >----- Original Message -----
> >>I've been looking at the recent patches to redirect GF_FOP_IPC to an active
> >>subvolume instead of always to the first. Specifically, these:
> >>
> >> http://review.gluster.org/11346 for DHT
> >> http://review.gluster.org/11347 for EC
> >> http://review.gluster.org/11348 for AFR
> >>
> >>I can't help but wonder if there's a simpler and more generic way to do this,
> >>instead of having to do this in a translator-specific way each time - then
> >>again for NSR, or for a separate tiering translator, and so on. For example
> >>what if each translator had a first_active_child callback?
> >>
> >> xlator_t * (*first_active_child) (xlator_t *parent);
> >>
> >>Then default_ipc could invoke this, if it exists, where it currently invokes
> >>FIRST_CHILD. Each translator could implement a bare minimum to select a
> >>child, then "step out of the way" for a fop it really wasn't all that
> >>interested in to begin with. Any thoughts?
> >
> >We should do this right away. This change doesn't affect external interfaces.
> >we should be bold and implement the first solution. Over time we could improve
> >on this.
>
> +1. It would definitely ease the implementation of many such fops which have
> to default to first active child. We need not keep track of all the fops
> which may get affected with new clustering xlators being added.
I think it is a great improvement and makes the code much easier to
understand. Do we have a volunteer that wants to have a go at
implementing this?
Thanks,
Niels
More information about the Gluster-devel
mailing list