Re: [PR] PostgreSQL: Support PostgreSQL ANALYZE with optional table and column [datafusion-sqlparser-rs]

["iffyio (via GitHub)" <gi...@apache.org>]

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


##########
src/ast/mod.rs:
##########
@@ -3325,19 +3325,24 @@ impl Display for ExceptionWhen {
     }
 }
 
-/// ANALYZE TABLE statement (Hive-specific)
+/// ANALYZE statement
+///
+/// Supported syntax varies by dialect:
+/// - Hive: `ANALYZE TABLE t [PARTITION (...)] COMPUTE STATISTICS [NOSCAN] [FOR COLUMNS [col1, ...]] [CACHE METADATA]`
+/// - PostgreSQL: `ANALYZE [VERBOSE] [t [(col1, ...)]]`
+/// - General: `ANALYZE [TABLE] t`

Review Comment:
   Can we add a link to the postgres docs describing the syntax? maybe this one https://www.postgresql.org/docs/current/sql-analyze.html



-- 
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-unsubscribe@datafusion.apache.org

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


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