iffyio commented on code in PR #1616:
URL: 
https://github.com/apache/datafusion-sqlparser-rs/pull/1616#discussion_r1898902087


##########
src/ast/mod.rs:
##########
@@ -3401,6 +3401,13 @@ pub enum Statement {
         partitioned: Option<Vec<Expr>>,
         table_format: Option<HiveLoadDataFormat>,
     },
+    /// ```sql
+    /// Rename TABLE tbl_name TO new_tbl_name[, tbl_name2 TO new_tbl_name2] ...
+    /// ```
+    /// Renames one or more tables
+    ///
+    /// See Mysql <https://dev.mysql.com/doc/refman/9.1/en/rename-table.html>
+    RenameTable(Vec<RenameObjectDef>),

Review Comment:
   ```suggestion
       RenameTable(Vec<RenameTable>),
   ```
   Could we call `RenameTable`? that would match the variant name, and the 
object def terminology I think would become too specific or no longer 
descriptive if other fields are added to the struct



##########
src/dialect/mod.rs:
##########
@@ -715,6 +715,11 @@ pub trait Dialect: Debug + Any {
         true
     }
 
+    // Returns true if the dialect supports the `RENAME TABLE` statement
+    fn supports_rename_table(&self) -> bool {
+        false
+    }

Review Comment:
   Ah so its not only a matter of whether the dialect supports the syntax in 
this case, I also meant that in order to avoid introducing the 
`supports_rename_table` flag, we can allow the parser always accept the `RENAME 
TABLE` syntax. Yes it will indeed mean that all dialects accept the syntax but 
that is fine given that it does not conflict with any other syntax the dialect 
supports



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to