(unnamed)

text/html

Download

<div>Hi,</div><div><div>I applied the diff file.</div><div>Deleted the $ts variable.</div><div> </div></div><div> </div><div> </div><div>----------------</div><div>Кому: 'ls7777' (<a href="mailto:ls7777@yandex.ru" rel="noopener noreferrer">ls7777@yandex.ru</a>);</div><div>Копия: <a href="mailto:pgsql-hackers@postgresql.org" rel="noopener noreferrer">pgsql-hackers@postgresql.org</a>, <a href="mailto:orlovmg@gmail.com" rel="noopener noreferrer">orlovmg@gmail.com</a>, <a href="mailto:amit.kapila16@gmail.com" rel="noopener noreferrer">amit.kapila16@gmail.com</a>;</div><div>Тема: Patch for migration of the pg_commit_ts directory;</div><div>14.10.2025, 09:27, "Hayato Kuroda (Fujitsu)" &lt;<a href="mailto:kuroda.hayato@fujitsu.com" rel="noopener noreferrer">kuroda.hayato@fujitsu.com</a>&gt;:</div><blockquote><p>Hi,<br /><br />Thanks for updating the patch.<br /><br />```<br />+my ($xid,$ts) = $resold =~ /\s*(\d+)\s*\|(.*)/;<br />```<br /><br />Per my understanding $ts is not used here. Can you remove it?<br /><br />Apart from above, I found some cosmetic issues. Please see attached<br />my fix which can be applied atop HEAD. Can you check and include if<br />it is OK?<br /><br />Best regards,<br />Hayato Kuroda<br />FUJITSU LIMITED<br /> </p></blockquote>