dajiaohuang opened a new pull request, #2646: URL: https://github.com/apache/phoenix/pull/2646
### What changes were proposed in this pull request? Skip setting `CHECK_VERIFY_COLUMN` for covered global-index scans that evaluate `ROW_SIZE()` or `RAW_ROW_SIZE()`. Keep the existing index-rebuild exclusion and leave read repair enabled for ordinary scans. ### Why are the changes needed? These size functions should measure the stored index state without triggering a read-repair mutation. In particular, `RAW_ROW_SIZE()` reads raw versions and delete markers, which is incompatible with the regular read-repair path. ### Does this PR introduce _any_ user-facing change? Yes. `ROW_SIZE()` and `RAW_ROW_SIZE()` scans no longer repair covered global-index rows as a side effect; they measure the existing stored state. ### How was this patch tested? - `mvn -B -ntp -Dmaven.repo.local=<task-local-cache> -f <checkout>/pom.xml -pl phoenix-core -am -Dtest=ScanUtilTest -Dsurefire.failIfNoSpecifiedTests=false -DskipITs=true -DPhoenixPatchProcess test` - JDK 17.0.20.1 and Maven 3.9.11; `ScanUtilTest`: 23 passed, 0 failures, 0 errors, 0 skipped. - No full build or CI run is claimed. ### Was this patch authored or co-authored using generative AI tooling? Generated-by: Pi 1.4.0 (gpt-6-luna) This PR description was generated by AI and has not been human-reviewed. -- 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]
