On Wed, Oct 3, 2018 at 2:15 PM Gary Gregory <[email protected]> wrote:

> AFK ATM. I can do the TimeUnit API when I get back.
>

Hm... should this be done for all APIs that return a time in a scaled named
in the API name (or not) ?

Gary


>
> Gary
>
> On Wed, Oct 3, 2018, 13:18 Oleg Kalnichevski <[email protected]> wrote:
>
>> On Wed, 2018-10-03 at 12:15 -0600, Gary Gregory wrote:
>> > On Wed, Oct 3, 2018 at 11:08 AM Michael Osipov <[email protected]>
>>
>> ...
>>
>> > > I do simply mean that getSocketTimeout() and
>> > > getSocketTimeout(TimeUnit)
>> > > read way better thatn getSocketTimeoutMillis() and
>> > > getSocketTimeout(TimeUnit). It just looks consistent.
>> > >
>> >
>> > Ah, well, that's a matter of opinion ;-)
>> >
>> > Gary
>>
>> I also find `TimeoutMillis` just outright hideous. I would very much
>> rather have #getSocketTimeout(TimeUnit) as Michael proposes.
>>
>> Oleg
>>
>> ---------------------------------------------------------------------
>> To unsubscribe, e-mail: [email protected]
>> For additional commands, e-mail: [email protected]
>>
>>

Reply via email to