<div dir="ltr">I saw the benchmark data provided on the issue and it seems reasonable to me.<div><br></div><div>Also regarding the usage of unicode a good practice is to use the unicode sandwich [1] "<span style="color:rgb(0,0,0);font-family:verdana,sans-serif;font-size:12.928px;line-height:19.392px">bytes on the outside, Unicode on the inside". Importing unicode_literals may be dangerous, but it is a really good step into porting/supporting Python 3.</span></div><div><span style="color:rgb(0,0,0);font-family:verdana,sans-serif;font-size:12.928px;line-height:19.392px"><br></span></div><div><span style="color:rgb(0,0,0);font-family:verdana,sans-serif;font-size:12.928px;line-height:19.392px">The feedback I am providing is as a pure Python developer, since I don't know much about Pulp internals I may not be considering all the points around the proposed improvement. Please take with a grain of salt.</span></div><div><span style="color:rgb(0,0,0);font-family:verdana,sans-serif;font-size:12.928px;line-height:19.392px"><br></span></div><div><span style="color:rgb(0,0,0);font-family:verdana,sans-serif;font-size:12.928px;line-height:19.392px">[1] </span><font color="#000000" face="verdana, sans-serif"><span style="font-size:12.928px;line-height:19.392px"><a href="http://nedbatchelder.com/text/unipain.html">http://nedbatchelder.com/text/unipain.html</a></span></font></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 26, 2016 at 12:03 PM, Filip Dobrovolny <span dir="ltr"><<a href="mailto:fdobrovo@redhat.com" target="_blank">fdobrovo@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi all, I have an idea for enhancing the performance of pulp_rpm publish which would require simpler escaping of special characters and is a lot faster.<div><br><div>I would love to hear what do you think about this.</div><div><br></div><div><a href="https://pulp.plan.io/issues/2169" target="_blank">https://pulp.plan.io/issues/<wbr>2169</a><br clear="all"><div><div data-smartmail="gmail_signature"><div dir="ltr"><div><br></div><div>Kind Regards,<br><br></div>Filip Dobrovolny.<br></div></div></div>
</div></div></div>
<br>______________________________<wbr>_________________<br>
Pulp-dev mailing list<br>
<a href="mailto:Pulp-dev@redhat.com">Pulp-dev@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/pulp-dev" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/pulp-dev</a><br>
<br></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Elyézer Rezende<div>Senior Quality Engineer<br><div>irc: elyezer</div></div></div></div>
</div>