[PHOENIX-5920] Skip SYSTEM TABLE checks while creating phoenix connection if client has set the DoNotUpgrade config

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 5.0.0, 4.15.0, 4.14.3
• Fix Version/s: 5.1.0, 4.16.0
• old issue number: 4,579
• Assignee: Neha Gupta
• Reporter: Neha Gupta
• Created: 26/May/20 19:48
• Updated: 14/Dec/20 21:20
• Resolved: 25/Nov/20 18:25

Description

Phoenix checks upgrade path even if client has set "DoNotUpgrade" config. When HMaster failover happens and Phoenix client creating phoenix connection, we get exception "org.apache.hadoop.hbase.PleaseHoldException: Master is initializing" because phoenix calls HBase admin apis to check if SYSTEM tables are in place and does not require an upgrade.  CQSI.init() initiates creating systems table and CQSI.ensureTableCreated is checking if SYTEM table requires upgrade and calls hbaseAdminApis to perform this operation which fails if HMaster is not running. Can we honor the "DoNotUpgrade" config by client(By making an assumption that SYSTEM table is already up-to-date and phoenix client shouldn't perform any upgrades) and skip this upgrade checks ?

EnsureTableCreated performs bunch of client-server compatibility checks like:

1.SYSTEM.CATALOG exists and its timestamp < MIN_SYSTEM_TABLE_TIMESTAMP
2.SYSTEM.CATALOG exists, but client and server-side namespace mapping is enabled so we need to migrate SYSTEM tables to the SYSTEM namespace

3.When an end-user uses the vanilla PhoenixDriver to create a connection and a requirement for upgrade is detected. In this case, the user should get a connection on which they are only able to run "EXECUTE UPGRADE".

Issue Links

• links to: GitHub Pull Request #792