pgmold-294: bump pgmold-sqlparser to 0.62.0 + adopt table_name rename (#272)

The 0.62.0 release (fmguerreiro/datafusion-sqlparser-rs#32) renames
Statement::Comment.relation to table_name for consistency with sibling
AST nodes (AlterPolicy.table_name, CreatePolicy.table_name,
DropPolicy.table_name, AlterTrigger.table_name).

pgmold-293's audit had concluded the bump was Cargo.toml-only because
pgmold matched Statement::Comment { .. } without field-name destructure.
That audit was stale: pgmold-273 (e860e3d) had since added a destructure
that bound `relation`. This commit therefore also:

- renames pgmold's internal CommentStatement.relation -> table_name to
  mirror the upstream AST.
- aliases the field at the destructure site to `partner_table` so the
  Trigger arm's `let (table_schema, table_name) = extract_qualified_name(...)`
  doesn't shadow it.

Discovered-from: pgmold-293.

Author: fmguerreiro

Cargo.lock

@@ -17,7 +17,7 @@ categories = ["command-line-utilities", "database"]
 clap = { version = "4.5", features = ["derive", "env"] }
 tokio = { version = "1.35", features = ["full"] }
 sqlx = { version = "0.8", features = ["runtime-tokio-rustls", "postgres"] }
-sqlparser = { package = "pgmold-sqlparser", version = "0.61.1", features = ["visitor"] }
+sqlparser = { package = "pgmold-sqlparser", version = "0.62.0", features = ["visitor"] }
 serde = { version = "1.0", features = ["derive"] }
 serde_json = "1.0"
 anyhow = "1.0"
@@ -37,11 +37,11 @@ sqlx = { version = "0.8", features = ["runtime-tokio-rustls", "postgres"] }
 tokio = { version = "1.35", features = ["full"] }
 assert_cmd = "2"
 criterion = { version = "0.5", features = ["html_reports"] }
-sqlparser = { package = "pgmold-sqlparser", version = "0.61.1", features = ["visitor"] }
+sqlparser = { package = "pgmold-sqlparser", version = "0.62.0", features = ["visitor"] }
 
 [[bench]]
 name = "performance"
 harness = false
 
 [build-dependencies]
-sqlparser = { package = "pgmold-sqlparser", version = "0.61.1", features = ["visitor"] }
+sqlparser = { package = "pgmold-sqlparser", version = "0.62.0", features = ["visitor"] }

Cargo.toml

@@ -31,7 +31,7 @@ pub(super) struct CommentStatement<'a> {
     pub object_type: CommentObject,
     pub object_name: &'a ObjectName,
     pub arguments: Option<&'a [DataType]>,
-    pub relation: Option<&'a ObjectName>,
+    pub table_name: Option<&'a ObjectName>,
     pub comment: Option<String>,
 }
 
@@ -47,7 +47,7 @@ pub(super) fn apply_comment_statement(
         object_type,
         object_name,
         arguments,
-        relation,
+        table_name: partner_table,
         comment,
     } = stmt;
 
@@ -155,12 +155,12 @@ pub(super) fn apply_comment_statement(
                 )));
             }
             let trigger_name = trigger_parts.into_iter().next().unwrap();
-            let Some(relation) = relation else {
+            let Some(partner_table) = partner_table else {
                 return Err(SchemaError::ParseError(
                     "COMMENT ON TRIGGER missing ON <table> tail".into(),
                 ));
             };
-            let (table_schema, table_name) = extract_qualified_name(relation);
+            let (table_schema, table_name) = extract_qualified_name(partner_table);
             let key = format!("{table_schema}.{table_name}.{trigger_name}");
             push(schema, PendingCommentObjectType::Trigger, key, comment);
         }
@@ -169,7 +169,7 @@ pub(super) fn apply_comment_statement(
         // these via its preprocess-stage scan and turn them into errors
         // under `--strict`.
         CommentObject::Policy => {
-            let target = match relation {
+            let target = match partner_table {
                 Some(rel) => {
                     let (rs, rn) = extract_qualified_name(rel);
                     format!("{object_name} ON {rs}.{rn}")

src/parser/comments.rs

@@ -1328,7 +1328,7 @@ fn parse_sql_string_inner(sql: &str) -> Result<Schema> {
                 object_type,
                 object_name,
                 arguments,
-                relation,
+                table_name,
                 comment,
                 if_exists: _,
             } => {
@@ -1337,7 +1337,7 @@ fn parse_sql_string_inner(sql: &str) -> Result<Schema> {
                         object_type,
                         object_name: &object_name,
                         arguments: arguments.as_deref(),
-                        relation: relation.as_ref(),
+                        table_name: table_name.as_ref(),
                         comment,
                     },
                     &mut schema,