<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">Thanks for the update. Changes look good to me.<div class=""><br class=""><div class="">
<div dir="auto" style="caret-color: rgb(0, 0, 0); color: rgb(0, 0, 0); letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><div>- Rajan</div></div>

</div>
<div style=""><br class=""><blockquote type="cite" class=""><div class="">On Mar 25, 2020, at 1:48 AM, <a href="mailto:sha.jiang@oracle.com" class="">sha.jiang@oracle.com</a> wrote:</div><br class="Apple-interchange-newline"><div class="">
  
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8" class="">
  
  <div class=""><p class="">Hi Rajan,<br class="">
      Thanks for your suggestions.<br class="">
      Please review the updated webrev:
      <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~jjiang/8237977/webrev.01/">http://cr.openjdk.java.net/~jjiang/8237977/webrev.01/</a><br class="">
    </p>
    <div class="moz-cite-prefix">On 2020/3/25 11:08, Rajan Halade wrote:<br class="">
    </div>
    <blockquote type="cite" cite="mid:71507D8F-FA7D-4FB3-A171-1FD9810B41EF@ORACLE.COM" class="">
      <meta http-equiv="Content-Type" content="text/html; charset=UTF-8" class="">
      Hi John,
      <div class=""><br class="">
      </div>
      <div class="">Thanks for taking care of this fix. Your changes
        look good me. </div>
      <div class=""><br class="">
      </div>
      <div class="">I have couple of suggestions:</div>
      <div class=""><br class="">
      </div>
      <div class="">- consider changing header for last column from “Why
        negative case” to “Reason”</div>
      <div class="">- for a failed test case (a testcase that succeeds
        when expected to fail or a testcase that fails when expected to
        succeed) code will not have any reason listed and we will need
        to click hyperlink in the report to find out root cause. May I
        suggest to pass in status to TestCase.negativeCaseReason()
        method call and use reason as “Refer to log at test hyperlink
        for details…” for such cases.</div>
    </blockquote>
    A new method TestCase::reason is introduced for the reasons.<br class="">
    <br class="">
    Best regards,<br class="">
    John Jiang
    <blockquote type="cite" cite="mid:71507D8F-FA7D-4FB3-A171-1FD9810B41EF@ORACLE.COM" class="">
      <div class=""><br class="">
        <div class="">
          <div dir="auto" style="caret-color: rgb(0, 0, 0); letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none; word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
            <div class="">Thanks,</div>
            <div class="">Rajan</div>
          </div>
        </div>
        <div class=""><br class="">
          <blockquote type="cite" class="">
            <div class="">On Mar 24, 2020, at 6:50 PM, <a href="mailto:sha.jiang@oracle.com" class="" moz-do-not-send="true">sha.jiang@oracle.com</a> wrote:</div>
            <br class="Apple-interchange-newline">
            <div class="">
              <div class="">Hi,<br class="">
                This patch updates the manual test
                javax/net/ssl/compatibility/Compatibility.java on the
                following points:<br class="">
                1. Covers SSLv3<br class="">
                2. The server side doesn't limit/specify protocols and
                cipher suites anymore. Only the client side specifies
                these parameters.<br class="">
                3. Only focus on the testing JDK specified by jtreg
                option "-jdk", and not run the cases between the JDKs
                builds in jdkList. This would save much execution time.<br class="">
                4. The report has a new column that clarifying why a
                case is a negative case.<br class="">
                <br class="">
                Issue: <a href="https://bugs.openjdk.java.net/browse/JDK-8237977" class="" moz-do-not-send="true">https://bugs.openjdk.java.net/browse/JDK-8237977</a><br class="">
                Webrev: <a href="http://cr.openjdk.java.net/~jjiang/8237977/webrev.00/" class="" moz-do-not-send="true">http://cr.openjdk.java.net/~jjiang/8237977/webrev.00/</a><br class="">
                <br class="">
                Best regards,<br class="">
                John Jiang<br class="">
                <br class="">
              </div>
            </div>
          </blockquote>
        </div>
        <br class="">
      </div>
    </blockquote>
  </div>

</div></blockquote></div><br class=""></div></body></html>