[11u] RFR (S): 8151678: com/sun/jndi/ldap/LdapTimeoutTest.java failed due to timeout on DeadServerNoTimeoutTest is incorrect

Langer, Christoph christoph.langer at sap.com
Mon Aug 31 17:32:26 UTC 2020


Hi Severin,

> I had a look at this for 11u approval and this jumped out at me:
> 
> src/jdk.naming.ldap/share/classes/com/sun/jndi/ldap/dns/LdapDnsProvider
> Service.java
> 
>  107         synchronized (LOCK) {
>  108             Iterator<LdapDnsProvider> iterator = providers.iterator();
>  109             while (result == null && iterator.hasNext()) {
>  110                 result = iterator.next().lookupEndpoints(url, envCopy)
>  111                         .filter(r -> r.getEndpoints().isEmpty())
>  112                         .orElse(null);
>  113             }
> 
> Line 111 inverses the filter from the original change[1]. This looks
> wrong.

Oops, you're right. I believe the nightlies would have shown this regression, but anyway, thanks for spotting it.

Here's the fix: http://cr.openjdk.java.net/~clanger/webrevs/8151678.11u.2/

Thanks
Christoph



More information about the jdk-updates-dev mailing list