RFR of JDK-8232446: logging enhancement for rmi when socket closed

Hamlin Li huaming.li at oracle.com
Fri Nov 22 07:54:14 UTC 2019


Hi Roger,

Thank you for reviewing, I have updated as you suggested: 
http://cr.openjdk.java.net/~mli/8232446/webrev.01/

Thank you

-Hamlin

On 2019/11/18 11:48 PM, Roger Riggs wrote:
> Hi Hamlin,
>
> TCPConnection.java:212:
>
> Keep the "close connection" logging and add the socket to the same log 
> message:
>
> If anyone is scraping the log, they won't loose this message. 
> TCPTransport.tcpLog.log(Log.BRIEF, "close connection, socket: " + 
> socket);
>
> TCPTransport.java
>
> 277-278:  combine the message to be one logging call.
> server socket
> 289: use Log.BRIEF, avoid creating mixture of and too many log levels.
>
> Reword the log messages so they each begin with "server socket...",
> or "server socket close"...
> it makes it easier to grep for and coorelate related messages
>
> Thanks, Roger
>
>
> On 11/6/19 7:02 AM, Hamlin Li wrote:
>>
>> On 2019/11/6 5:36 PM, Peter Levart wrote:
>>> Hi Hamlin,
>>>
>>> in TCPTransport.decrementExportCount():
>>>
>>>  283             try {
>>>  284                 if (tcpLog.isLoggable(Log.BRIEF)) {
>>>  285                     tcpLog.log(Log.BRIEF, "close server socket 
>>> on " + ss);
>>>  286                 }
>>>  287                 ss.close();
>>>  288             } catch (IOException e) {
>>>  289             }
>>>
>>> ...you could add a log statement to the catch block too. Or even 
>>> better, rearrange for IOException to be thrown from that method and 
>>> deal with it in two places:
>>>
>>> - in exportObject() - add it as suppressed exception to exception 
>>> thrown from super.exportObject().
>>> - in targetUnexported() - log it or wrap it into 
>>> UncheckedIOException (depending on what are the callers of 
>>> targetUnexported())
>>>
>>> What do you think?
>> Thanks Peter.
>>
>> I agree. I adopt your first suggestion: add log statement to catch 
>> block, as I think it's simple/straight and sufficient to help diagnose.
>>
>> And I also add log for catch blocks in other close places.
>>
>> The change is updated in place at: 
>> http://cr.openjdk.java.net/~mli/8232446/webrev.00/
>>
>>
>> Thank you
>>
>> -Hamlin
>>
>>>
>>> Regards, Peter
>>>
>>>
>>>
>>> On 11/6/19 3:07 AM, Hamlin Li wrote:
>>>> Would you please review the patch?
>>>>
>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8232446
>>>>
>>>> webrev: http://cr.openjdk.java.net/~mli/8232446/webrev.00/
>>>>
>>>>
>>>> We have some intermittent failures in rmi related to socket 
>>>> closing, this is to add more logging to help diagnose the issues.
>>>>
>>>>
>>>> Thanks you
>>>>
>>>> -Hamlin
>>>>
>>>
>


More information about the core-libs-dev mailing list