This is an automated email from the ASF dual-hosted git repository.
yiguolei pushed a commit to branch branch-4.0
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-4.0 by this push:
new 93277bde426 branch-4.0: [fix](cloud) Fix misleading CommittedVersion
display in show partitions #60623 (#60679)
93277bde426 is described below
commit 93277bde4266369385c030af8f500de7bb73aa03
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Fri Feb 13 17:47:13 2026 +0800
branch-4.0: [fix](cloud) Fix misleading CommittedVersion display in show
partitions #60623 (#60679)
Cherry-picked from #60623
Co-authored-by: deardeng <[email protected]>
---
.../java/org/apache/doris/cloud/catalog/CloudPartition.java | 6 ++++++
.../org/apache/doris/cloud/catalog/CloudPartitionTest.java | 10 ++++++++++
2 files changed, 16 insertions(+)
diff --git
a/fe/fe-core/src/main/java/org/apache/doris/cloud/catalog/CloudPartition.java
b/fe/fe-core/src/main/java/org/apache/doris/cloud/catalog/CloudPartition.java
index 788b4c6dbfa..459526e68de 100644
---
a/fe/fe-core/src/main/java/org/apache/doris/cloud/catalog/CloudPartition.java
+++
b/fe/fe-core/src/main/java/org/apache/doris/cloud/catalog/CloudPartition.java
@@ -357,6 +357,12 @@ public class CloudPartition extends Partition {
return news;
}
+ @Override
+ public long getCommittedVersion() {
+ // Cloud partition version is managed by meta-service, not resident in
FE memory.
+ return -1;
+ }
+
@Override
public long getNextVersion() {
// use meta service visibleVersion
diff --git
a/fe/fe-core/src/test/java/org/apache/doris/cloud/catalog/CloudPartitionTest.java
b/fe/fe-core/src/test/java/org/apache/doris/cloud/catalog/CloudPartitionTest.java
index 8213cd6c307..de01b9a02f5 100644
---
a/fe/fe-core/src/test/java/org/apache/doris/cloud/catalog/CloudPartitionTest.java
+++
b/fe/fe-core/src/test/java/org/apache/doris/cloud/catalog/CloudPartitionTest.java
@@ -43,6 +43,16 @@ public class CloudPartitionTest {
return new CloudPartition(pId, "p" + pId, null, null, dbId, tblId);
}
+ @Test
+ public void testGetCommittedVersion() {
+ CloudPartition part = createPartition(1, 2, 3);
+ // Cloud partition's committedVersion should return -1 (unknown),
+ // not the base class's nextVersion - 1 = -2
+ Assertions.assertEquals(-1, part.getCommittedVersion());
+ // Also verify nextVersion returns -1 for consistency
+ Assertions.assertEquals(-1, part.getNextVersion());
+ }
+
@Test
public void testIsCachedVersionExpired() {
// Create ConnectContext with SessionVariable
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
