RFR 8245302: Upgrade LogRecord to support long thread ids and remove its usage of ThreadLocal

[Rahul Yadav]

Hello,

Deprecated tag in LogRecord getter and setter for threadID was missing 
"since" and so i have added it.

webrev : http://cr.openjdk.java.net/~ryadav/webrev_8245302/webrev.00/

- rahul

On 22/06/2020 19:38, Daniel Fuchs wrote:
> Hi Rahul,
>
> Looks good to me as well.
>
> Reviewed.
>
> best regards,
>
> -- daniel
>
> On 19/06/2020 10:00, Rahul Yadav wrote:
>> Thank you Alan, updated webrev.
>>
>> webrev : 
>> http://cr.openjdk.java.net/~ryadav/webrev_8245302/webrev.00/index.html
>>
>> - rahul
>>
>> On 19/06/2020 08:43, Alan Bateman wrote:
>>> On 18/06/2020 23:37, Rahul Yadav wrote:
>>>> Hi Alan,
>>>>
>>>> Thank you for the feedback.I have updated the webrev.
>>>>
>>>> webrev : 
>>>> http://cr.openjdk.java.net/~ryadav/webrev_8245302/webrev.00/index.html
>>> This looks quite good.
>>>
>>> The comment in shortShortID has "any positive long less than 
>>> Integer.MAX_VALUE" but it's actually <= MAX_VALUE.
>>>
>>> I don't think MIN_SEQUENTIAL_THREAD_ID is used so I assume it can be 
>>> removed.
>>>
>>> The @return for setLongThreadID has a description "Log Record" but 
>>> this should "this LogRecord".
>>>
>>> Can you update SerializeLogRecordTest with clear instructions on how 
>>> to generate the stream? This will help future maintainers that may 
>>> have to update this test.
>>>
>>> -Alan
>>
>