You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@hc.apache.org by Oleg Kalnichevski <ol...@apache.org> on 2011/09/06 20:57:15 UTC

Re: svn commit: r1165635

On Tue, 2011-09-06 at 20:55 +0300, Vasile Alin wrote:
> On 6 September 2011 19:02, sebb <se...@gmail.com> wrote:
> 
> >>     /**
> >>      * Creates a new client connection operator for the given scheme registry.
> >>      *
> >> @@ -100,6 +105,26 @@ public class DefaultClientConnectionOper
> >>             throw new IllegalArgumentException("Scheme registry amy not be null");
> >>         }
> >>         this.schemeRegistry = schemes;
> >> +        this.dnsResolver = null;
> >
> > Could replace this with a default resolver that did
> >
> >     return InetAddress.getAllByName(host);
> >
> >> +    }
> 
> Good idea.
> 
> 

+1. Would you have time to do a follow-up patch?

Oleg


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org


Re: svn commit: r1165635

Posted by Vasile Alin <al...@gmail.com>.
Yes, I'll attach the patch to the original issue.

On 6 September 2011 21:57, Oleg Kalnichevski <ol...@apache.org> wrote:
> On Tue, 2011-09-06 at 20:55 +0300, Vasile Alin wrote:
>> On 6 September 2011 19:02, sebb <se...@gmail.com> wrote:
>>
>> >>     /**
>> >>      * Creates a new client connection operator for the given scheme registry.
>> >>      *
>> >> @@ -100,6 +105,26 @@ public class DefaultClientConnectionOper
>> >>             throw new IllegalArgumentException("Scheme registry amy not be null");
>> >>         }
>> >>         this.schemeRegistry = schemes;
>> >> +        this.dnsResolver = null;
>> >
>> > Could replace this with a default resolver that did
>> >
>> >     return InetAddress.getAllByName(host);
>> >
>> >> +    }
>>
>> Good idea.
>>
>>
>
> +1. Would you have time to do a follow-up patch?
>
> Oleg
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
> For additional commands, e-mail: dev-help@hc.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org