JingsongLi commented on PR #10153: URL: https://github.com/apache/paimon/pull/10153#issuecomment-5805434677
Requirement fit: SUPPORTED. Avoiding eager reads of all ADD manifests for an append-only LIMIT scan has a direct planning benefit, and this uses the existing bounded reader without adding an option. Implementation: CLEAN in the changed path. I checked normal scan and iterator callers, the ADD/DELETE merge path, and fallbacks for data predicates, deletion vectors, and data evolution. I found no actionable regression. Verification: the 9 new real-file/manifest tests and the core reactor package build passed locally (JDK 8, offline). In a broader TableScanTest run, 20/23 passed; three unrelated postpone-merge cases hit a local AvroSchemaConverter NoSuchMethodError from a preinstalled dependency mismatch. CI was still running when reviewed. The local base predates this PR base by about two days, so green CI on the current base remains necessary. -- 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]
