saintstack commented on pull request #2274: URL: https://github.com/apache/hbase/pull/2274#issuecomment-677764605
`To me, I think this is a mistake, it should not be placed in RegionInfo, or at least should be marked as IA.Private? We have lots of IA.Private fields and methods in RegionInfo.` And I think the opposite. RegionInfo seems natural home for static defines such as UNDEFINED and first Region of meta. Thanks for the other suggestions. Let me add IA.Private and deprecation. Let me add that here to this last patch. Shout if you -1 on this last approach. Otherwise, please remove changes requested. Thanks. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
