apurtell commented on PR #2428: URL: https://github.com/apache/phoenix/pull/2428#issuecomment-6067073147
@kadirozde I updated the merge commit where it failed to address some explain plan changes. Now it does and all relevant tests pass. While performing the work Opus 5.5 claimed to discover two bugs. They seem legitimate to me so I have included fixes for them on the branch and would like your review if you have time: - [3e1caf6](https://github.com/apache/phoenix/pull/2428/commits/3e1caf67559ff5d1a07e5bc6b305d69b55fa4729) : When an `IN` list on a leading primary key dimension exceeds the KeySpaceList budget (65,536 spaces), `widenToBudget` completely drops that dimension, turning the scan into an unbounded full table scan. - [05fed39](https://github.com/apache/phoenix/pull/2428/commits/05fed398df1d87069204e184aeba73e3aced9750): The V2 where optimizer can produce incorrect scan ranges and drop matching rows when filtering on `DESC` variable-length columns (e.g., `VARCHAR DESC`) with prefix-related bounds (such as `k1 = '2' OR k1 > '23'`). -- 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]
