update

Author: forsaken628

.typos.toml

@@ -12,6 +12,7 @@ extend-ignore-re = [
 # Preserved for NdJson
 Nd = "Nd"
 typ = "typ"
+aranges = "aranges"
 
 [default.extend-identifiers]
 

src/common/hashtable/src/utils.rs

@@ -234,7 +234,7 @@ pub struct MergeIntoBlockInfoIndex {
 pub type Interval = (u32, u32);
 
 /// the segment blocks are not sequential,because we do parallel hashtable build.
-/// the block lay out in chunks could be like belows:
+/// the block lay out in chunks could be like bellows:
 /// segment0_block1 |
 /// segment1_block0 |  chunk0
 /// segment0_block0 |

src/meta/raft-store/src/state/raft_state_kv.rs

@@ -31,6 +31,7 @@ pub enum RaftStateKey {
     /// The node id.
     Id,
 
+    // typos:ignore-next-line
     /// Hard state of the raft log, including `current_term` and `voted_vor`.
     HardState,
 

src/meta/types/src/sys_data.rs

@@ -216,7 +216,7 @@ mod tests {
         assert_eq!(want, serialized);
     }
 
-    /// Test serialze will all fields set.
+    /// Test serialize will all fields set.
     #[test]
     fn test_sys_data_serialize() {
         let sys_data = SysData {

src/query/ast/src/parser/error_suggestion.rs

@@ -246,7 +246,7 @@ mod tests {
             Some("Did you mean `SHOW TABLE FUNCTIONS` or `SHOW TABLES`?".to_string())
         );
         assert_eq!(
-            suggest_correction("vacum drop table"),
+            suggest_correction("vacum drop table"), // typos:disable-line
             Some("Did you mean `VACUUM DROP TABLE`?".to_string())
         );
         assert_eq!(

src/query/ast/src/parser/statement.rs

@@ -3298,7 +3298,7 @@ pub fn column_def(i: Input) -> IResult<ColumnDefinition> {
         value(ColumnConstraint::Nullable(true), rule! { NULL }),
         value(ColumnConstraint::Nullable(false), rule! { NOT ~ ^NULL }),
     ));
-    let identity_parmas = alt((
+    let identity_params = alt((
         map(
             rule! {
                 "(" ~ ^#literal_u64 ~ ^"," ~ ^#literal_i64 ~ ^")"
@@ -3341,7 +3341,7 @@ pub fn column_def(i: Input) -> IResult<ColumnDefinition> {
         map(
             rule! {
                 (AUTOINCREMENT | IDENTITY)
-                ~ #identity_parmas?
+                ~ #identity_params?
                 ~ (ORDER | NOORDER)?
             },
             |(_, params, order_token)| {

src/query/ast/tests/it/parser.rs

@@ -1097,6 +1097,7 @@ fn test_statement_error() {
         r#"GRANT OWNERSHIP ON *.* TO ROLE 'd20_0015_owner';"#,
         r#"CREATE FUNCTION IF NOT EXISTS isnotempty AS(p) -> not(is_null(p)"#,
         r#"CREATE FUNCTION my_agg (INT) STATE { s STRING } RETURNS BOOLEAN LANGUAGE javascript HANDLER = 'my_agg' ADDRESS = 'http://0.0.0.0:8815';"#,
+        // typos:ignore-next-line
         r#"CREATE FUNCTION my_agg (INT) STATE { s STRIN } RETURNS BOOLEAN LANGUAGE javascript ADDRESS = 'http://0.0.0.0:8815';"#,
         r#"drop table :a"#,
         r#"drop table IDENTIFIER(a)"#,

src/query/service/src/physical_plans/physical_mutation.rs

@@ -433,7 +433,7 @@ impl PhysicalPlanBuilder {
             DUMMY_COLUMN_INDEX
         };
 
-        // For distributed merge, we shuffle data blocks by block_id (drived from row_id) to avoid
+        // For distributed merge, we shuffle data blocks by block_id (derived from row_id) to avoid
         // different nodes update the same physical block simultaneously, data blocks that are needed
         // to insert just keep in local node.
         if *distributed && *row_id_shuffle && !is_not_matched_only {

src/query/service/src/servers/flight/v1/actions/flight_actions.rs

@@ -95,7 +95,7 @@ impl FlightActions {
                     let future = CatchUnwindFuture::create(future);
                     match future
                         .await
-                        .with_context(|| "failed to do filght action")
+                        .with_context(|| "failed to do flight action")
                         .flatten()
                     {
                         Ok(v) => {

src/query/service/src/sessions/session_privilege_mgr.rs

@@ -171,7 +171,7 @@ impl SessionPrivilegeManager for SessionPrivilegeManagerImpl<'_> {
     /// If secondary_roles is set, it must be ALL or NONE:
     /// 1. ALL: all the roles granted to the current user will have effects on validate_privilege,
     ///    `secondary_roles` will be set to None, which is default.
-    /// 2. NONE: only the current_role has effects on validate_privilge, `secondary_roles`
+    /// 2. NONE: only the current_role has effects on validate_privilege, `secondary_roles`
     ///    will be set to Some([]).
     /// 3. SpecifyRoles: Some([role1, role2, .. etc.]).
     #[async_backtrace::framed]