<div dir="ltr">Pranith,<div><br></div><div><div class="gmail_extra"><br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><br></div> We found that compound fops is not giving better performance in replicate and I am thinking of removing that code. Sent the patch at <a href="https://review.gluster.org/19655" target="_blank">https://review.gluster.org/196<wbr>55</a><span class="m_-6327896448455170920HOEnZb"><font color="#888888"><br></font></span></div><span class="m_-6327896448455170920HOEnZb"><font color="#888888"><div><div><div><br></div></div></div></font></span></div></blockquote><div><br></div><div>If I understand it right, as of now AFR is the only component which uses Compound FOP. If it stops using that code, should we maintain the compound fop codebase at all in other places, like protocol, volgen (decompounder etc?)</div><div><br></div><div>Because, if AFR as a module decides compound fop is not useful, and other method is better, it is completely a decision of AFR maintainers. I don't see a concern there. </div><div><br></div><div>Only if it had an option and if people are using it, then warning them upfront about this change is better.</div><div><br></div><div>Regards,</div><div>Amar</div><div><br></div></div>
</div></div></div>