stoty commented on PR #1604: URL: https://github.com/apache/phoenix/pull/1604#issuecomment-1545284290
However, we also call the function from updateDescriptorForTx(), and that one didn't check for old vs new indexing. It doesn't matter for the current master, as changing transactional status is disabled in both directions, but it may matter for an 5.1 backport, or in the future when it is re-enabled. Please take another look, @gjacoby126 . -- 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]
