<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 24 Mar 2022, 22:16 Xavi Hernandez <<a href="mailto:xhernandez@redhat.com">xhernandez@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div>Hi Strahil,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Mar 24, 2022 at 8:26 PM Strahil Nikolov <<a href="mailto:hunter86_bg@yahoo.com" target="_blank" rel="noreferrer">hunter86_bg@yahoo.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hey Xavi,<div><br></div><div>Did anyone measure performance behavior before and after the changes?</div></blockquote><div><br></div><div>I haven't tested performance for this change, but I don't expect any appreciable variation. The main reason to do it is to provide a simpler way to create and use log messages that makes them more flexible and consistent. It's specially useful when used with an editor that support code completion.</div><div><br></div><div>Given that I've rewritten a significant part of the code, I've taken the opportunity to include some things that could have a minimal performance benefit, but it's not the main reason.</div></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">If we skip the work to create the log string, when eventually we don't need to log at all (as we are not in TRACE or DEBUG levels) that could help a bit. Some of those UUID and lkowner prints for example. </div><div dir="auto">Y. </div><div dir="auto"><br></div><div dir="auto"><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 class="gmail_quote"><div><br></div><div>Best regards,</div><div><br></div><div>Xavi</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><br></div><div>Best Regards,</div><div>Strahil Nikolov<br> <br> <blockquote style="margin:0px 0px 20px"> <div style="font-family:Roboto,sans-serif;color:rgb(109,0,246)"> <div>On Thu, Mar 24, 2022 at 20:33, Xavi Hernandez</div><div><<a href="mailto:xhernandez@redhat.com" target="_blank" rel="noreferrer">xhernandez@redhat.com</a>> wrote:</div> </div> <div style="padding:10px 0px 0px 20px;margin:10px 0px 0px;border-left:1px solid rgb(109,0,246)"> <div id="m_8006331838643996687gmail-m_-3474842985224959619yiv8462860519"><div dir="ltr">Hi all,<div><br></div><div>I've just posted a proposal for a new logging interface here: <a rel="nofollow noopener noreferrer noreferrer" href="https://github.com/gluster/glusterfs/pull/3342" target="_blank">https://github.com/gluster/glusterfs/pull/3342</a></div><div><br></div><div>There are many comments and the documentation is updated in the PR itself, so I won't duplicate all the info here. Please check it if you are interested in the details.</div><div><br></div><div>As a summary, I think that the new interface is easier to use, more powerful, more flexible and more robust.</div><div><br></div><div>Since it affects an interface used by every single component of Gluster I would like to have some more feedback before deciding whether we merge it or not. Feel free to comment here or in the PR itself.</div><div><br></div><div>Thank you very much,</div><div><br></div><div>Xavi</div></div>
</div>-------<br><br>Community Meeting Calendar:<br>Schedule -<br>Every 2nd and 4th Tuesday at 14:30 IST / 09:00 UTC<br>Bridge: <a href="https://meet.google.com/cpu-eiue-hvk" target="_blank" rel="noreferrer">https://meet.google.com/cpu-eiue-hvk</a><br><br>Gluster-devel mailing list<br><a href="mailto:Gluster-devel@gluster.org" target="_blank" rel="noreferrer">Gluster-devel@gluster.org</a><br><a href="https://lists.gluster.org/mailman/listinfo/gluster-devel" target="_blank" rel="noreferrer">https://lists.gluster.org/mailman/listinfo/gluster-devel</a><br><br> </div> </blockquote></div></blockquote></div></div>
-------<br>
<br>
Community Meeting Calendar:<br>
Schedule -<br>
Every 2nd and 4th Tuesday at 14:30 IST / 09:00 UTC<br>
Bridge: <a href="https://meet.google.com/cpu-eiue-hvk" rel="noreferrer noreferrer" target="_blank">https://meet.google.com/cpu-eiue-hvk</a><br>
<br>
Gluster-devel mailing list<br>
<a href="mailto:Gluster-devel@gluster.org" target="_blank" rel="noreferrer">Gluster-devel@gluster.org</a><br>
<a href="https://lists.gluster.org/mailman/listinfo/gluster-devel" rel="noreferrer noreferrer" target="_blank">https://lists.gluster.org/mailman/listinfo/gluster-devel</a><br>
<br>
</blockquote></div></div></div>