Bump parser version (#1168)

There's one place we can benefit from mutable lists, and a couple of
things that slipped through resolving a merge conflict

Author: sgrif

Cargo.lock

@@ -80,7 +80,7 @@ smallvec = "1"
 reqwest.workspace = true
 hex = "0.4"
 x509-parser = "0.18"
-pg_raw_parse = { git = "https://github.com/pgdogdev/pg_raw_parse.git", rev = "6c7cee4", optional = true }
+pg_raw_parse = { git = "https://github.com/pgdogdev/pg_raw_parse.git", rev = "48b4944", optional = true }
 itertools = "0.15.0"
 
 [target.'cfg(unix)'.dependencies]

pgdog/Cargo.toml

@@ -139,7 +139,7 @@ mod test {
             unreachable!();
         };
 
-        for node in stmt.targetList() {
+        for node in stmt.target_list() {
             let func = Function::try_from(node.val()).unwrap();
             check(func);
         }

pgdog/src/frontend/router/parser/function.rs

@@ -69,13 +69,9 @@ impl AggregatesRewrite {
         if helper_nodes.is_empty() {
             RewriteOutput::default()
         } else {
-            let mut target_list = select
-                .target_list()
-                .into_iter()
-                .map(|n| mem.make_unique(n))
-                .collect::<Vec<_>>();
-            target_list.extend(helper_nodes);
-            select.set_target_list(mem.make_list(&target_list));
+            select
+                .target_list_mut()
+                .extend(mem, mem.make_list(&helper_nodes));
             RewriteOutput::new(plan)
         }
     }

pgdog/src/frontend/router/parser/rewrite/statement/aggregate/engine.rs

@@ -5,8 +5,10 @@ use pg_raw_parse::{NodeMut, make::MemoryToken};
 #[cfg(not(feature = "new_parser"))]
 use pgdog_config::QueryParserEngine;
 
+#[cfg(not(feature = "new_parser"))]
+use crate::backend::ShardingSchema;
+use crate::frontend::PreparedStatements;
 use crate::net::Parse;
-use crate::{backend::ShardingSchema, frontend::PreparedStatements};
 
 use super::{Error, StatementRewrite};
 
@@ -55,7 +57,7 @@ impl StatementRewrite<'_> {
             return Ok(result);
         }
 
-        match rewrite_single_prepared(node, mem, self.prepared_statements, self.schema)? {
+        match rewrite_single_prepared(node, mem, self.prepared_statements)? {
             SimplePreparedRewrite::Prepared => {
                 result.rewritten = true;
             }

pgdog/src/frontend/router/parser/rewrite/statement/simple_prepared.rs

@@ -622,7 +622,7 @@ impl<'a> StatementRewrite<'a> {
 fn rewrite_params(node: NodeMut<'_, '_>) -> IndexSet<u16> {
     let mut params = IndexSet::new();
     walk::walk_mut(node, |node| match node {
-        NodeMut::ParamRef(mut param) => {
+        NodeMut::ParamRef(param) => {
             params.insert(param.number as _);
             param.set_number(params.get_index_of(&(param.number as u16)).unwrap() as i32 + 1)
         }

pgdog/src/frontend/router/parser/rewrite/statement/update.rs

@@ -281,7 +281,7 @@ mod test {
     fn selected_expr(result: &pg_raw_parse::ParseResult) -> Node<'_> {
         let stmt = result.stmts().exactly_one().ok().unwrap();
         match stmt {
-            Node::SelectStmt(s) => s.targetList().into_iter().exactly_one().unwrap().val(),
+            Node::SelectStmt(s) => s.target_list().into_iter().exactly_one().unwrap().val(),
             _ => unreachable!(),
         }
     }