yjhjstz commented on code in PR #1761:
URL: https://github.com/apache/cloudberry/pull/1761#discussion_r3298979534
##########
src/backend/optimizer/path/indxpath.c:
##########
@@ -813,7 +813,14 @@ get_index_paths(PlannerInfo *root, RelOptInfo *rel,
*/
if (index->amhasgettuple)
{
- if (!AMHandlerIsAO(rel->amhandler) ||
+ /*
+ * PAX shares AO's planner concern: random TID fetch
+ * re-decompresses whole stripes and bypasses the
buffer pool.
+ * Treat it as AO-like so only Bitmap paths survive
add_path
+ * here.
+ */
+ if ((!AMHandlerIsAO(rel->amhandler) &&
+ rel->relam != PAX_AM_OID) ||
Review Comment:
fixed. And keep gp_enable_ao_indexscan path for some purpose.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
