deepakpanda93 commented on issue #14513: URL: https://github.com/apache/hudi/issues/14513#issuecomment-5114587808
This issue was reviewed as part of the JIRA-migrated backlog triage. **Findings:** this appears to be fixed. The request (from the [#1009 review comment](https://github.com/apache/incubator-hudi/pull/1009#discussion_r357181383)) was to make layout-version filter handling in the active timeline explicit - ideally via factory methods whose names say whether instants are filtered. That now exists: - `TimelineFactory.createActiveTimeline(HoodieTableMetaClient, boolean applyLayoutFilter)` - `hudi-common/src/main/java/org/apache/hudi/common/table/timeline/TimelineFactory.java:42`, alongside the unfiltered `createActiveTimeline()` / `createActiveTimeline(metaClient)` overloads. - The filtering itself is encapsulated per layout in `TimelineLayout.filterHoodieInstants()`, with `TimelineLayout.fromVersion(...)` selecting `V0`/`V1`/`V2` (`TimelineLayout.java:47-79`). This landed with the 1.x timeline abstraction in [HUDI-8076] (commit [df9c4d67ec7b](https://github.com/apache/hudi/commit/df9c4d67ec7b), #11923) and was first released in **1.0.0**. If you're still hitting this on a recent version, please reopen with details. Closing as fixed. -- 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]
