iffyio commented on code in PR #1844:
URL:
https://github.com/apache/datafusion-sqlparser-rs/pull/1844#discussion_r2082694788
##########
src/ast/ddl.rs:
##########
@@ -39,6 +39,27 @@ use crate::ast::{
use crate::keywords::Keyword;
use crate::tokenizer::Token;
+#[derive(Debug, Clone, PartialEq, PartialOrd, Eq, Ord, Hash)]
Review Comment:
It could be helpful here to add a description comment for the enum with a
link to the docs
##########
src/ast/ddl.rs:
##########
@@ -208,6 +229,12 @@ pub enum AlterTableOperation {
old_partitions: Vec<Expr>,
new_partitions: Vec<Expr>,
},
+ /// REPLICA IDENTITY { DEFAULT | USING INDEX index_name | FULL | NOTHING }
+ ///
+ /// Note: this is a PostgreSQL-specific operation.
Review Comment:
Can we add a link to the docs ideally so folks can easily find where the
syntax comes from?
--
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
