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

[Rahul Yadav]

Hello,

I have added comments @Serial based on csr feedback for read/writeObject 
methods.

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

- rahul

On 25/06/2020 10:22, Rahul Yadav wrote:
> 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
>>>
>>
>