From 2ae1f0c9c0f29e8003861c5af7cf9eacf3dbbf95 Mon Sep 17 00:00:00 2001 From: Gavin Chou Date: Mon, 7 Jul 2025 10:34:21 +0800 Subject: [PATCH] [opt](auditlog) Use varchar(1024) for column frontend_ip of audit log table (#52762) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit length 128 of column frontend_ip of audit log table may be not large enough for k8s env. ``` Reason:column_name [frontend_ip], the length of input is too long than schema. first 32 bytes of input str: [ abcedf-xxxxxx-yyyyyyy] schema length: 128; actual length:159; . src line []; ``` --- .../src/main/java/org/apache/doris/catalog/InternalSchema.java | 2 +- regression-test/suites/manager/test_manager_interface_1.groovy | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/fe/fe-core/src/main/java/org/apache/doris/catalog/InternalSchema.java b/fe/fe-core/src/main/java/org/apache/doris/catalog/InternalSchema.java index 9940bedbe49465..477aaf02d01cf2 100644 --- a/fe/fe-core/src/main/java/org/apache/doris/catalog/InternalSchema.java +++ b/fe/fe-core/src/main/java/org/apache/doris/catalog/InternalSchema.java @@ -133,7 +133,7 @@ public class InternalSchema { AUDIT_SCHEMA.add(new ColumnDef("user", TypeDef.createVarchar(128), ColumnNullableType.NULLABLE)); AUDIT_SCHEMA.add(new ColumnDef("frontend_ip", - TypeDef.createVarchar(128), ColumnNullableType.NULLABLE)); + TypeDef.createVarchar(1024), ColumnNullableType.NULLABLE)); // default ctl and db AUDIT_SCHEMA.add(new ColumnDef("catalog", TypeDef.createVarchar(128), ColumnNullableType.NULLABLE)); diff --git a/regression-test/suites/manager/test_manager_interface_1.groovy b/regression-test/suites/manager/test_manager_interface_1.groovy index ec57261b147dcb..929d3b33db446a 100644 --- a/regression-test/suites/manager/test_manager_interface_1.groovy +++ b/regression-test/suites/manager/test_manager_interface_1.groovy @@ -568,7 +568,7 @@ suite('test_manager_interface_1',"p0") { assertTrue(result[0][1].contains("`return_rows` bigint NULL,")) assertTrue(result[0][1].contains("`stmt_id` bigint NULL,")) assertTrue(result[0][1].contains("`is_query` tinyint NULL,")) - assertTrue(result[0][1].contains("`frontend_ip` varchar(128) NULL,")) + assertTrue(result[0][1].contains("`frontend_ip` varchar(1024) NULL,")) assertTrue(result[0][1].contains("`cpu_time_ms` bigint NULL,")) assertTrue(result[0][1].contains("`sql_hash` varchar(128) NULL,")) assertTrue(result[0][1].contains("`sql_digest` varchar(128) NULL,"))