Fix version bug of cpt file (#21924)

[LeftHandCold]

User description

Fix version bug of cpt file

Approved by: @XuPeng-SH

What type of PR is this?

• [ ] API-change
• [x] BUG
• [ ] Improvement
• [ ] Documentation
• [ ] Feature
• [ ] Test and CI
• [ ] Code Refactoring

Which issue(s) this PR fixes:

issue https://github.com/matrixorigin/MO-Cloud/issues/5490

What this PR does / why we need it:

Fix version bug of cpt file

---

PR Type

Bug fix

---

Description

• Fix checkpoint version assignment in merge operation

• Replace dynamic version retrieval with constant current version

---

Changes walkthrough 📝

	Relevant files
Bug fix	merge.go Fix checkpoint version assignment                                                pkg/vm/engine/tae/db/gc/v3/merge.go Replace ckpEntries[len(ckpEntries)-1].GetVersion() with logtail.CheckpointCurrentVersion Fix version assignment in checkpoint batch creation during merge +1/-1

---

Need help?

Type /help how to ... in the comments thread for any questions about Qodo Merge usage.

Check out the documentation for more information.

[qodo-code-review[bot]]

You are nearing your monthly Qodo Merge usage quota. For more information, please visit here.

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Version Consistency The change replaces dynamic version retrieval from the last checkpoint entry with a constant current version. This could potentially cause version inconsistencies if the merged checkpoint entries have different versions than the current version, which might affect checkpoint compatibility and recovery processes. bat.GetVectorByName(checkpoint.CheckpointAttr_Version).Append(logtail.CheckpointCurrentVersion, false) bat.GetVectorByName(checkpoint.CheckpointAttr_AllLocations).Append([]byte(location), false)

[qodo-code-review[bot]]

You are nearing your monthly Qodo Merge usage quota. For more information, please visit here.

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Validate checkpoint version compatibility Using a constant version instead of the actual version from checkpoint entries may cause version inconsistency issues. Consider validating that all checkpoint entries have compatible versions before using the constant. pkg/vm/engine/tae/db/gc/v3/merge.go [157] +if err := validateCheckpointVersions(ckpEntries); err != nil { + return nil, err +} bat.GetVectorByName(checkpoint.CheckpointAttr_Version).Append(logtail.CheckpointCurrentVersion, false) [ ] Apply / Chat <!-- /improve --apply_suggestion=0 --> Suggestion importance[1-10]: 7 __ Why: The suggestion correctly points out a potential issue. The PR changes the version source from a dynamic entry to a constant logtail.CheckpointCurrentVersion. It is a valid concern that the versions of the entries in ckpEntries might not be compatible with this constant, which could lead to inconsistency. Adding validation is a good practice to improve the robustness of the code.	Medium
[ ] More <!-- /improve --more_suggestions=true -->