apache / apache/hudi

Refactor checkpoint translation APIs

Open
#17,318 1 comment 0 reactions 1 assignee Assigned to @nsivabalan View on GitHub
from-jira priority:critical type:devtask
Dominant language
Java
Stars
6.2k
Forks
2.5k
Avg merge
2d 8h
Merged PRs (30d)
111

Description

# Do we really need completely different relations based on checkpoint version.. I feel just passing in the checkpoint version to use is sufficient.. I worry there are too many V1, V2 classes to maintain here. [https://github.com/apache/hudi/pull/12342#discussion_r1864665107]
# Can we introduce a factory.. {{Checkpoint.create(..)}} that will create the current/latest by default.. then we avoid use of version specific {{CheckpointV2}} and CheckpointV1` classes in regular code.. There should be a few places where we will handle CheckpointV1 and thats it.
#
use {{HoodieTableVersion instead of int writeTableVersion - [https://github.com/apache/hudi/pull/12342#discussion_r1864669100]}}

{{4. Code duplication - https://github.com/apache/hudi/pull/12342#discussion_r1864668618}}

{{{}{}}}{{{}{}}}

## JIRA info

- Link: https://issues.apache.org/jira/browse/HUDI-8627
- Type: Sub-task
- Parent: https://issues.apache.org/jira/browse/HUDI-9105
- Fix version(s):
- 1.1.0

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.