Update base on comments

Author: romanoff

src/ast/query.rs

@@ -2578,7 +2578,7 @@ pub enum TableVersion {
     /// When the table version is defined using a function.
     /// For example: `SELECT * FROM tbl AT(TIMESTAMP => '2020-08-14 09:30:00')`
     Function(Expr),
-    /// Snowflake CHANGES clause for change tracking queries.
+    /// Snowflake `CHANGES` clause for change tracking queries.
     /// For example:
     /// ```sql
     /// SELECT * FROM t

src/parser/mod.rs

@@ -16276,19 +16276,7 @@ impl<'a> Parser<'a> {
                 let expr = self.parse_expr()?;
                 return Ok(Some(TableVersion::ForSystemTimeAsOf(expr)));
             } else if self.peek_keyword(Keyword::CHANGES) {
-                // Snowflake CHANGES clause:
-                // CHANGES(INFORMATION => ...) AT(...) [END(...)]
-                let changes_name = self.parse_object_name(true)?;
-                let changes = self.parse_function(changes_name)?;
-                let at_name = self.parse_object_name(true)?;
-                let at = self.parse_function(at_name)?;
-                let end = if self.peek_keyword(Keyword::END) {
-                    let end_name = self.parse_object_name(true)?;
-                    Some(self.parse_function(end_name)?)
-                } else {
-                    None
-                };
-                return Ok(Some(TableVersion::Changes { changes, at, end }));
+                return self.parse_table_version_changes().map(Some);
             } else if self.peek_keyword(Keyword::AT) || self.peek_keyword(Keyword::BEFORE) {
                 let func_name = self.parse_object_name(true)?;
                 let func = self.parse_function(func_name)?;
@@ -16304,6 +16292,30 @@ impl<'a> Parser<'a> {
         Ok(None)
     }
 
+    /// Parses the Snowflake `CHANGES` clause for change tracking queries.
+    ///
+    /// Syntax:
+    /// ```sql
+    /// CHANGES (INFORMATION => DEFAULT)
+    ///   AT (TIMESTAMP => <expr>)
+    ///   [END (TIMESTAMP => <expr>)]
+    /// ```
+    ///
+    /// <https://docs.snowflake.com/en/sql-reference/constructs/changes>
+    fn parse_table_version_changes(&mut self) -> Result<TableVersion, ParserError> {
+        let changes_name = self.parse_object_name(true)?;
+        let changes = self.parse_function(changes_name)?;
+        let at_name = self.parse_object_name(true)?;
+        let at = self.parse_function(at_name)?;
+        let end = if self.peek_keyword(Keyword::END) {
+            let end_name = self.parse_object_name(true)?;
+            Some(self.parse_function(end_name)?)
+        } else {
+            None
+        };
+        Ok(TableVersion::Changes { changes, at, end })
+    }
+
     /// Parses MySQL's JSON_TABLE column definition.
     /// For example: `id INT EXISTS PATH '$' DEFAULT '0' ON EMPTY ERROR ON ERROR`
     pub fn parse_json_table_column_def(&mut self) -> Result<JsonTableColumn, ParserError> {

tests/sqlparser_snowflake.rs

@@ -3967,7 +3967,7 @@ fn test_timetravel_at_before() {
 fn test_changes_clause() {
     // CHANGES with AT and END
     snowflake().verified_stmt(
-        "SELECT a FROM \"PCH_ODS_FIDELIO\".\"SRC_VW_SYS_ACC_MASTER\" CHANGES(INFORMATION => DEFAULT) AT(TIMESTAMP => TO_TIMESTAMP_TZ('2026-02-18 11:23:19.660000000')) END(TIMESTAMP => TO_TIMESTAMP_TZ('2026-02-18 11:38:30.211000000'))",
+        r#"SELECT a FROM "PCH_ODS_FIDELIO"."SRC_VW_SYS_ACC_MASTER" CHANGES(INFORMATION => DEFAULT) AT(TIMESTAMP => TO_TIMESTAMP_TZ('2026-02-18 11:23:19.660000000')) END(TIMESTAMP => TO_TIMESTAMP_TZ('2026-02-18 11:38:30.211000000'))"#,
     );
 
     // CHANGES with AT only (no END)