This is an automated email from the ASF dual-hosted git repository.
airborne pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/master by this push:
new 6c573d6b443 [fix](store) Fix the issue where storage_page_size equals
0 in multi-FE scenarios (#44306)
6c573d6b443 is described below
commit 6c573d6b4430c3fe73a26761b465d2e641a980a8
Author: zzzxl <[email protected]>
AuthorDate: Wed Nov 20 13:53:04 2024 +0800
[fix](store) Fix the issue where storage_page_size equals 0 in multi-FE
scenarios (#44306)
### What problem does this PR solve?
Related PR:
https://github.com/apache/doris/pull/43690
https://github.com/apache/doris/pull/44080
Problem Summary:
1. Creating a table on a Follower will result in storage_page_size being
equal to 0.
---
fe/fe-core/src/main/java/org/apache/doris/catalog/TableProperty.java | 1 +
regression-test/suites/query_p0/system/test_storage_page_size.groovy | 2 +-
2 files changed, 2 insertions(+), 1 deletion(-)
diff --git
a/fe/fe-core/src/main/java/org/apache/doris/catalog/TableProperty.java
b/fe/fe-core/src/main/java/org/apache/doris/catalog/TableProperty.java
index 25cf2f4287e..1ac556c6846 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/catalog/TableProperty.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/catalog/TableProperty.java
@@ -737,6 +737,7 @@ public class TableProperty implements Writable,
GsonPostProcessable {
buildStoreRowColumn();
buildRowStoreColumns();
buildRowStorePageSize();
+ buildStoragePageSize();
buildSkipWriteIndexOnLoad();
buildCompactionPolicy();
buildTimeSeriesCompactionGoalSizeMbytes();
diff --git
a/regression-test/suites/query_p0/system/test_storage_page_size.groovy
b/regression-test/suites/query_p0/system/test_storage_page_size.groovy
index 9d7b129226d..491d702938e 100644
--- a/regression-test/suites/query_p0/system/test_storage_page_size.groovy
+++ b/regression-test/suites/query_p0/system/test_storage_page_size.groovy
@@ -33,7 +33,7 @@ suite ("test_storage_page_size") {
test {
sql "show create table table_1;"
check { result, exception, startTime, endTime ->
- assertFalse(result[0][1].contains("\"storage_page_size\" =
\"65536\""))
+ assertFalse(result[0][1].contains("storage_page_size"))
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
