gjacoby126 commented on PR #1455:
URL: https://github.com/apache/phoenix/pull/1455#issuecomment-1160722256
Also, what will happen to user tables who upgrade from Phoenix 5.1 -> 5.2
and have the Provider set to Tephra? Would that cause the region to crash from
trying to load an enum value that doesn't exist? If so, what sort of upgrade
logic should we include? I can think of three options:
1. Switch tables to non-transactional
2. Switch tables to Omid
3. Keep the Tephra enum and just WARN or ERROR on region load that it
doesn't do anything
1 and 2 are both tricky to coordinate on a rolling restart, because the
normal upgrade logic runs when SYSTEM.CATALOG is upgraded, but the crashing
would probably come before that, when a region hosting a TEPHRA-enabled table
has its region server upgraded. But I worry that some operators would miss the
warning in their logs.
@stoty @jpisaac @kadirozde @virajjasani , curious on your opinions.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
