Hi Pavel,

The change looks good.  Nice job on the tests :-)


> On Aug 7, 2019, at 8:47 AM, Pavel Rappo <pavel.ra...@oracle.com> wrote:
> 
> Hello,
> 
> Please review the following change:
> 
>   http://cr.openjdk.java.net/~prappo/8217606/webrev.00/
> 
> The original issue seems to be an unintended consequence of JDK-8059009 [1].
> The proposed change is based on the private patch by Chris Yin. I skimmed
> through the history of the `com.sun.jndi.ldap.Connection.useable` flag which
> dates back to JDK-4532408 [2]
> 
>    - Maintains a flag, 'useable', to indicate whether underlying network
>    connection is useable. A connection starts out useable and becomes
>    unuseable when an IO error is encountered while reading from/writing
>    to it.
> 
> and I believe that this field is a good fit for what Chris has proposed.
> 
> Not only does the proposed change fix the issue, it also introduces a reusable
> test component for mocking/stubbing/faking an LDAP server. Chris and myself 
> took
> some time in our private correspondence to shape that component such that it
> would allow to refactor some of the existing LDAP tests, should we decide to 
> do so.
> 
> As far as mach5 is concerned, the change looks good.
> 
> Thanks,
> -Pavel
> 
> -------------------------------------------------------------------------------
> [1] https://bugs.openjdk.java.net/browse/JDK-8059009
> [2] https://bugs.openjdk.java.net/browse/JDK-4532408
> 

 <http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif> 
<http://oracle.com/us/design/oracle-email-sig-198324.gif>
 <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| 
Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering 
1 Network Drive 
Burlington, MA 01803
lance.ander...@oracle.com <mailto:lance.ander...@oracle.com>



Reply via email to