QuakeWang commented on PR #9141: URL: https://github.com/apache/paimon/pull/9141#issuecomment-5282105374
> I feel the changes are a bit too extensive; hardly anyone uses 64-bit bitmaps in a production environment—they are primarily used to generate Iceberg-compatible tables from primary key tables. @JingsongLi Thanks, I agree that the current scope has grown beyond the original correctness issue. I’ll simplify the PR to use a conservative fallback whenever Bitmap64 would require position-based file-index pushdown, instead of adding Bitmap64-specific projection and optimization logic. I’ll retain only the minimal post-DV LIMIT safeguard needed for correctness and focused regression tests for the original under-return issue and large positions. This should also remove the cardinality heuristic and the Bitmap64-specific filter/TopN optimizations. Would this reduced scope match your expectation? -- 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]
