From fea8bc5ee750a6be81e72e91f0fe92fd632f35d6 Mon Sep 17 00:00:00 2001 From: Copilot <198982749+Copilot@users.noreply.github.com> Date: Thu, 10 Jul 2025 10:24:42 +0800 Subject: [PATCH] chore(comments): fix typo and grammar issues (#6496) * Initial plan * Fix 5 TODO comments: spelling typos and formatting issues Co-authored-by: waynexia <15380403+waynexia@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: waynexia <15380403+waynexia@users.noreply.github.com> --- src/catalog/src/system_schema/pg_catalog/pg_class.rs | 2 +- src/common/meta/src/ddl/create_flow/metadata.rs | 2 +- src/flow/src/adapter.rs | 2 +- src/operator/src/statement/ddl.rs | 2 +- tests-fuzz/src/validator/column.rs | 2 +- 5 files changed, 5 insertions(+), 5 deletions(-) diff --git a/src/catalog/src/system_schema/pg_catalog/pg_class.rs b/src/catalog/src/system_schema/pg_catalog/pg_class.rs index 57f8db882c..d34a6b4c6b 100644 --- a/src/catalog/src/system_schema/pg_catalog/pg_class.rs +++ b/src/catalog/src/system_schema/pg_catalog/pg_class.rs @@ -169,7 +169,7 @@ impl DfPartitionStream for PGClass { } /// Builds the `pg_catalog.pg_class` table row by row -/// TODO(J0HN50N133): `relowner` is always the [`DUMMY_OWNER_ID`] cuz we don't have user. +/// TODO(J0HN50N133): `relowner` is always the [`DUMMY_OWNER_ID`] because we don't have users. /// Once we have user system, make it the actual owner of the table. struct PGClassBuilder { schema: SchemaRef, diff --git a/src/common/meta/src/ddl/create_flow/metadata.rs b/src/common/meta/src/ddl/create_flow/metadata.rs index 1681479d91..27b85b7946 100644 --- a/src/common/meta/src/ddl/create_flow/metadata.rs +++ b/src/common/meta/src/ddl/create_flow/metadata.rs @@ -21,7 +21,7 @@ use crate::key::table_name::TableNameKey; impl CreateFlowProcedure { /// Allocates the [FlowId]. pub(crate) async fn allocate_flow_id(&mut self) -> Result<()> { - //TODO(weny, ruihang): We doesn't support the partitions. It's always be 1, now. + // TODO(weny, ruihang): We don't support the partitions. It's always be 1, now. let partitions = 1; let (flow_id, peers) = self .context diff --git a/src/flow/src/adapter.rs b/src/flow/src/adapter.rs index 1e55a0958f..2acba4979e 100644 --- a/src/flow/src/adapter.rs +++ b/src/flow/src/adapter.rs @@ -497,7 +497,7 @@ impl StreamingEngine { &self, schema: &RelationDesc, ) -> Result<(Vec, Vec, bool), Error> { - // TODO(discord9): condiser remove buggy auto create by schema + // TODO(discord9): consider remove buggy auto create by schema // TODO(discord9): use default key from schema let primary_keys = schema diff --git a/src/operator/src/statement/ddl.rs b/src/operator/src/statement/ddl.rs index 016619c23e..d57f575ce6 100644 --- a/src/operator/src/statement/ddl.rs +++ b/src/operator/src/statement/ddl.rs @@ -654,7 +654,7 @@ impl StatementExecutor { ctx.clone(), )?; - //TODO(dennis): validate the logical plan + // TODO(dennis): validate the logical plan self.create_view_by_expr(expr, ctx).await } diff --git a/tests-fuzz/src/validator/column.rs b/tests-fuzz/src/validator/column.rs index 1bf8274e92..dbd50e25c8 100644 --- a/tests-fuzz/src/validator/column.rs +++ b/tests-fuzz/src/validator/column.rs @@ -126,7 +126,7 @@ impl PartialEq for ColumnEntry { return false; } } - //TODO: Checks `semantic_type` + // TODO: Checks `semantic_type` match semantic_type(&self.semantic_type) { Some(SemanticType::Tag) => { if !other