saintstack commented on pull request #2274:
URL: https://github.com/apache/hbase/pull/2274#issuecomment-675774822
Your suggestion is better @bharathv .... more code change but looks safe to
me. RegionInfoBuilder, the host for FIRST_META_REGIONINFO is
@InterfaceAudience.Private so moving it is 'allowed'. Thought of adding back
something like the below to RegionInfoBuilder in case any downstream
references...
public static final RegionInfo FIRST_META_REGIONINFO =
RegionInfo.FIRST_META_REGIONINFO;
... but that might bring back the statics load deadlock in a new guise.
----------------------------------------------------------------
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:
us...@infra.apache.org
