- 
                Notifications
    
You must be signed in to change notification settings  - Fork 3.4k
 
HBASE-28802 Log the IP when hbase.server.useip.enabled is set to true #6175
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
| 
           💔 -1 overall 
 
 This message was automatically generated.  | 
    
| 
           🎊 +1 overall 
 
 This message was automatically generated.  | 
    
| 
           🎊 +1 overall 
 
 This message was automatically generated.  | 
    
| 
           💔 -1 overall 
 
 This message was automatically generated.  | 
    
| 
           🎊 +1 overall 
 
 This message was automatically generated.  | 
    
| 
           🎊 +1 overall 
 
 This message was automatically generated.  | 
    
| 
           💔 -1 overall 
 
 This message was automatically generated.  | 
    
| 
           🎊 +1 overall 
 
 This message was automatically generated.  | 
    
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM +1.
| 
           UT failure is not related to this PR change. Let me retrigger the QA.  | 
    
| 
           🎊 +1 overall 
 
 This message was automatically generated.  | 
    
| 
           🎊 +1 overall 
 
 This message was automatically generated.  | 
    
| 
           🎊 +1 overall 
 
 This message was automatically generated.  | 
    
| 
           🎊 +1 overall 
 
 This message was automatically generated.  | 
    
01fbe22    to
    779c582      
    Compare
  
    | 
           🎊 +1 overall 
 
 This message was automatically generated.  | 
    
| 
           🎊 +1 overall 
 
 This message was automatically generated.  | 
    
| 
           🎊 +1 overall 
 
 This message was automatically generated.  | 
    
| 
           🎊 +1 overall 
 
 This message was automatically generated.  | 
    
…#6175) Signed-off-by: Pankaj Kumar<[email protected]> Reviewed-by: Vineet Kumar Maheshwari <[email protected]> (cherry picked from commit 9292059)
| 
           Thanks for review the changes @vineet4008 @pankaj72981  | 
    
For HRegionServer#handleReportForDutyResponse, when the hostname is different from the regionserver and master side, both the two conditions should abort RS error message is corrected.