<!DOCTYPE html><html><head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body>
    On 31/05/2025 20:12, Shruthi . wrote:<br>
    <blockquote type="cite" cite="mid:SJ0PR15MB4663D5C27435718507DBA5C0CC60A@SJ0PR15MB4663.namprd15.prod.outlook.com">
      
      <style type="text/css" style="display:none;">P {margin-top:0;margin-bottom:0;}</style>
      <div class="elementToProof" style="margin-top: 1em; margin-bottom: 1em; font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
        Hi Alan,<br>
        <br>
        I wanted to follow up on the update I shared earlier</div>
      <div class="elementToProof" style="margin-top: 1em; margin-bottom: 1em; font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
        I’ve successfully backported the preClose re-order changes to <b>JDK
          17</b> and ran the
        <code>Race.java</code> test case <b>500 times</b>. It passed
        consistently without any failures. For validation, I also ran
        the test
        <b>without the patch</b>, and it failed on the <b>first
          iteration</b>. I ran the tests under
        <b>java/nio</b> to see if there is any regression and all the
        testcases are passing.</div>
      <div class="elementToProof" style="margin-top: 1em; margin-bottom: 1em; font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
        Currently, we are running the tests under <code>java/net</code>.</div>
      <ul data-end="802" data-start="667">
        <li style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
          <div style="margin-top: 1em; margin-bottom: 1em;">Do you
            recommend running any additional test groups?</div>
        </li>
        <li style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
          <div style="margin-top: 1em; margin-bottom: 1em;">Also, can we
            proceed with porting the preClose re-order fix to
            <b>headstream</b>?</div>
        </li>
      </ul>
      <div class="elementToProof" style="margin-top: 1em; margin-bottom: 1em; font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">
        Please let me know your thoughts.</div>
    </blockquote>
    <br>
    Good to hear that the preClose changes helped. I think the next step
    is to create a PR for JDK main line.<br>
    <br>
    -Alan<br>
  </body>
</html>