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

[vyom tewari]

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
>>