[RFR] 8160768: Add capability to custom resolve host/domain names within the default JDNI LDAP provider

vyom tewari vyom.tewari at oracle.com
Tue Oct 30 09:22:47 UTC 2018


Hi Rob,

Overall your code looks OK to me, In LdapDnsProviderService.java please 
make 'service' as volatile otherwise 'getInstance()' may not work as 
expected. I can see you implemented DCL but it may not work if 'service' 
is not declare volatile.

If you like, you can us the initialization 'On Demand Holder idiom' to 
implement the lazy initialization.

Thanks,

Vyom

On 26/10/18 8:31 PM, Rob McKenna wrote:
> Yes, thanks a lot Alan.
>
> Vyom and Martin both had some very helpful feedback so I'm hoping to
> hear from both!
>
>      -Rob
>
> On 26/10/18 15:34, Alan Bateman wrote:
>> On 25/10/2018 17:34, Rob McKenna wrote:
>>> This recently received CSR approval, so it seems like a good time to pick
>>> the codereview up again:
>>>
>>> http://cr.openjdk.java.net/~robm/8160768/webrev.08/
>>>
>> I went through a number of iterations with Rob on the API/javadoc so I think
>> the API parts in webrev.08 are good. I have not had time to go through the
>> implementation and test changes so hopefully that someone else has cycles to
>> help on that.
>>
>> -Alan
>>


More information about the core-libs-dev mailing list