[Gluster-Maintainers] Huge AFR change for 3.8 proposed, needs careful review and explicit approval

[Ravishankar N]

On 12/12/2016 02:10 PM, Pranith Kumar Karampuri wrote:
>
>
> On Sun, Dec 11, 2016 at 1:48 PM, Niels de Vos <ndevos at redhat.com 
> <mailto:ndevos at redhat.com>> wrote:
>
>     We've mentioned earlier that only (small) patches for bugfixes and
>     stabilization should get backported. Yet there is a patch with
>     +470/-61
>     lines proposed for inclusion. This is not something that is normally
>     suitable for backporting, so I'd like to get the opinion of several
>     maintainers before it gets merged.
>

The patch is entirely in AFR, so in addition to Pranith, which specific 
maintainers would you like a review from?

>
>       afr: allow I/O when favorite-child-policy is enabled
>     http://review.gluster.org/16091 <http://review.gluster.org/16091>
>
>     It sounds like a serious bugfix, but I'd like to know if there is any
>     risk for users of a version that has this patch. 
>


I don't think there is any risk in taking the patch in. The bug 
(https://bugzilla.redhat.com/show_bug.cgi?id=1378547
) was raised by Ubisoft on 3.8. I can set a need-info on the BZ to see 
if they would be willing to test it out. If they can, and find the fix 
to be working, can you take the patch in?

Thanks,
Ravi


>     The version for the
>     master branch has only been there for 2 weeks, and the 3.9 version has
>     not been merged yet either.
>
>
> We are awaiting netbsd regression to pass for merging it in 3.9. I 
> will be planning to get this in for 3.9.1.
>
>
>     Thanks,
>     Niels
>
>     _______________________________________________
>     maintainers mailing list
>     maintainers at gluster.org <mailto:maintainers at gluster.org>
>     http://www.gluster.org/mailman/listinfo/maintainers
>     <http://www.gluster.org/mailman/listinfo/maintainers>
>
>
>
>
> -- 
> Pranith


-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://www.gluster.org/pipermail/maintainers/attachments/20161212/8bbae7d0/attachment.html>