<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<div class="moz-cite-prefix">On 12/12/2016 02:10 PM, Pranith Kumar
Karampuri wrote:<br>
</div>
<blockquote
cite="mid:CAOgeEnZSLFtt-T78_HNPcSwdYYgVrjk+83r4QH2DZh=NGMouGg@mail.gmail.com"
type="cite">
<div dir="ltr"><br>
<div class="gmail_extra"><br>
<div class="gmail_quote">On Sun, Dec 11, 2016 at 1:48 PM,
Niels de Vos <span dir="ltr"><<a moz-do-not-send="true"
href="mailto:ndevos@redhat.com" target="_blank">ndevos@redhat.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">We've
mentioned earlier that only (small) patches for bugfixes
and<br>
stabilization should get backported. Yet there is a patch
with +470/-61<br>
lines proposed for inclusion. This is not something that
is normally<br>
suitable for backporting, so I'd like to get the opinion
of several<br>
maintainers before it gets merged.<br>
</blockquote>
</div>
</div>
</div>
</blockquote>
<br>
The patch is entirely in AFR, so in addition to Pranith, which
specific maintainers would you like a review from?<br>
<br>
<blockquote
cite="mid:CAOgeEnZSLFtt-T78_HNPcSwdYYgVrjk+83r4QH2DZh=NGMouGg@mail.gmail.com"
type="cite">
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
afr: allow I/O when favorite-child-policy is enabled<br>
<a moz-do-not-send="true"
href="http://review.gluster.org/16091" rel="noreferrer"
target="_blank">http://review.gluster.org/<wbr>16091</a><br>
<br>
It sounds like a serious bugfix, but I'd like to know if
there is any<br>
risk for users of a version that has this patch. </blockquote>
</div>
</div>
</div>
</blockquote>
<br>
<br>
I don't think there is any risk in taking the patch in. The bug
(<a class="moz-txt-link-freetext" href="https://bugzilla.redhat.com/show_bug.cgi?id=1378547">https://bugzilla.redhat.com/show_bug.cgi?id=1378547</a><br>
) 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?<br>
<br>
Thanks,<br>
Ravi<br>
<br>
<br>
<blockquote
cite="mid:CAOgeEnZSLFtt-T78_HNPcSwdYYgVrjk+83r4QH2DZh=NGMouGg@mail.gmail.com"
type="cite">
<div dir="ltr">
<div class="gmail_extra">
<div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">The
version for the<br>
master branch has only been there for 2 weeks, and the 3.9
version has<br>
not been merged yet either.<br>
</blockquote>
<div><br>
</div>
<div>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.<br>
</div>
<div> </div>
<blockquote class="gmail_quote" style="margin:0 0 0
.8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks,<br>
Niels<br>
<br>
______________________________<wbr>_________________<br>
maintainers mailing list<br>
<a moz-do-not-send="true"
href="mailto:maintainers@gluster.org">maintainers@gluster.org</a><br>
<a moz-do-not-send="true"
href="http://www.gluster.org/mailman/listinfo/maintainers"
rel="noreferrer" target="_blank">http://www.gluster.org/<wbr>mailman/listinfo/maintainers</a><br>
<br>
</blockquote>
</div>
<br>
<br clear="all">
<br>
-- <br>
<div class="gmail_signature" data-smartmail="gmail_signature">
<div dir="ltr">Pranith<br>
</div>
</div>
</div>
</div>
</blockquote>
<p><br>
</p>
</body>
</html>