[
https://issues.apache.org/jira/browse/HUDI-8627?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sagar Sumit updated HUDI-8627:
------------------------------
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}}
{{{}{}}}{{{}{}}}
was:
# 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.
> Refactor checkpoint translation APIs
> ------------------------------------
>
> Key: HUDI-8627
> URL: https://issues.apache.org/jira/browse/HUDI-8627
> Project: Apache Hudi
> Issue Type: Task
> Reporter: Sagar Sumit
> Priority: Major
> Fix For: 1.0.1
>
>
> # 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}}
> {{{}{}}}{{{}{}}}
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
