RFR 8235475: Add additional timestamp to test java/nio/channels/Selector/SelectWithConsumer.java

[Michael McMahon]

On 06/12/2019 12:12, Alan Bateman wrote:
> On 06/12/2019 11:56, Michael McMahon wrote:
>>
>> :
>>
>> I changed to use nanoTime in this test only, which should guarantee 
>> the highest millisecond resolution
>>
> Okay although I think we will need a follow-on to change the other 
> usages in this file so that we don't have a mix - can you create a 
> follow-on issue?
>
JDK-8235481 - Change java/nio/channels/Selector/SelectWithConsumer.java 
to use System.nanoTime

> The patch creates inconsistencies in the naming. I think "pre" need to 
> renamed to "before". It looks like "duration" needs to renamed too, 
> probably to "selectDuration" so that we don't have a mix of naming 
> conventions in this one test.
>
Okay, I'll change those.

Thanks,

Michael

> -Alan.
>