Upgrade to sqlparser-rs 0.56.0

Notes:

In v0.55 of sqlparser-rs, the `ObjectName` structure has been changed as shown below. Here is now to migrate.

```diff
- pub struct ObjectName(pub Vec<Ident>);
+ pub struct ObjectName(pub Vec<ObjectNamePart>)
```

Therefore, when using the `parse_sql` function, the data structure of the table name in the return value will change.

Previously:

```json
{
    "value": "employee",
    "quote_style": null,
    "span":
    {
        "start":
        {
            "line": 4,
            "column": 10
        },
        "end":
        {
            "line": 4,
            "column": 18
        }
    }
}
```

Now:

```json
{
    "Identifier":
    {
        "value": "employee",
        "quote_style": null,
        "span":
        {
            "start":
            {
                "line": 4,
                "column": 10
            },
            "end":
            {
                "line": 4,
                "column": 18
            }
        }
    }
}
```

Author: vimko

CHANGELOG.md

@@ -1,3 +1,63 @@
+# 0.1.56
+
+- Upgrade to sqlparser-rs 0.56.0
+
+In v0.55 of sqlparser-rs, the `ObjectName` structure has been changed as shown below. Here is now to migrate.
+
+```diff
+- pub struct ObjectName(pub Vec<Ident>);
++ pub struct ObjectName(pub Vec<ObjectNamePart>)
+```
+
+Therefore, when using the `parse_sql` function, the data structure of the table name in the return value will change.
+
+Previously:
+
+```json
+{
+    "value": "employee",
+    "quote_style": null,
+    "span":
+    {
+        "start":
+        {
+            "line": 4,
+            "column": 10
+        },
+        "end":
+        {
+            "line": 4,
+            "column": 18
+        }
+    }
+}
+```
+
+Now:
+
+
+```json
+{
+    "Identifier":
+    {
+        "value": "employee",
+        "quote_style": null,
+        "span":
+        {
+            "start":
+            {
+                "line": 4,
+                "column": 10
+            },
+            "end":
+            {
+                "line": 4,
+                "column": 18
+            }
+        }
+    }
+}
+```
 
 # 0.1.36
 

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "sqloxide"
-version = "0.1.54"
+version = "0.1.56"
 authors = ["Will Eaton <[email protected]>"]
 edition = "2021"
 
@@ -17,5 +17,5 @@ version = "0.22"
 features = ["extension-module"]
 
 [dependencies.sqlparser]
-version = "0.54.0"
+version = "0.56.0"
 features = ["serde", "visitor"]

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "sqloxide"
-version = "0.1.54rc2"
+version = "0.1.56"
 authors = [{ name = "Will Eaton", email= "<[email protected]>" }]
 repository = "https://github.com/wseaton/sqloxide"
 license = "MIT"

src/visitor.rs

@@ -6,7 +6,7 @@ use pyo3::prelude::*;
 use serde::Serialize;
 
 use sqlparser::ast::{
-    Statement, {visit_expressions, visit_expressions_mut, visit_relations, visit_relations_mut},
+    Statement, ObjectNamePart, {visit_expressions, visit_expressions_mut, visit_relations, visit_relations_mut},
 };
 
 // Refactored function for handling depythonization
@@ -61,7 +61,8 @@ pub fn mutate_relations(_py: Python, parsed_query: &Bound<'_, PyAny>, func: &Bou
     for statement in &mut statements {
         visit_relations_mut(statement, |table| {
             for section in &mut table.0 {
-                let val = match func.call1((section.value.clone(),)) {
+                let ObjectNamePart::Identifier(ident) = section;
+                let val = match func.call1((ident.value.clone(),)) {
                     Ok(val) => val,
                     Err(e) => {
                         let msg = e.to_string();
@@ -71,7 +72,7 @@ pub fn mutate_relations(_py: Python, parsed_query: &Bound<'_, PyAny>, func: &Bou
                     }
                 };
 
-                section.value = val.to_string();
+                ident.value = val.to_string();
             }
             ControlFlow::Continue(())
         });

tests/test_sqloxide.py

@@ -43,8 +43,14 @@ def test_extract_relations():
     ast = parse_sql(sql=SQL, dialect="ansi")
 
     assert extract_relations(parsed_query=ast)[0][0] == {
-        "value": "employee",
-        "quote_style": None,
+        "Identifier": {
+            "value": "employee",
+            "quote_style": None,
+            "span": {
+                "start": {"line": 4, "column": 10},
+                "end": {"line": 4, "column": 18},
+            },
+        }
     }
 
 
@@ -54,7 +60,7 @@ def func(x):
 
     ast = parse_sql(sql=SQL, dialect="ansi")
     assert mutate_relations(parsed_query=ast, func=func) == [
-        'SELECT employee.first_name, employee.last_name, c.start_time, c.end_time, call_outcome.outcome_text FROM employee JOIN "call2"."call2"."call2" AS c ON c.employee_id = employee.id JOIN call2_outcome ON c.call_outcome_id = call_outcome.id ORDER BY c.start_time ASC'
+        'SELECT employee.first_name, employee.last_name, c.start_time, c.end_time, call_outcome.outcome_text FROM employee INNER JOIN "call2"."call2"."call2" AS c ON c.employee_id = employee.id INNER JOIN call2_outcome ON c.call_outcome_id = call_outcome.id ORDER BY c.start_time ASC'
     ]
 
 
@@ -81,7 +87,7 @@ def func(x):
     ast = parse_sql(sql=SQL, dialect="ansi")
     result = mutate_expressions(parsed_query=ast, func=func)
     assert result == [
-        'SELECT EMPLOYEE.FIRST_NAME, EMPLOYEE.LAST_NAME, C.START_TIME, C.END_TIME, CALL_OUTCOME.OUTCOME_TEXT FROM employee JOIN "call"."call"."call" AS c ON C.EMPLOYEE_ID = EMPLOYEE.ID JOIN call_outcome ON C.CALL_OUTCOME_ID = CALL_OUTCOME.ID ORDER BY C.START_TIME ASC'
+        'SELECT EMPLOYEE.FIRST_NAME, EMPLOYEE.LAST_NAME, C.START_TIME, C.END_TIME, CALL_OUTCOME.OUTCOME_TEXT FROM employee INNER JOIN "call"."call"."call" AS c ON C.EMPLOYEE_ID = EMPLOYEE.ID INNER JOIN call_outcome ON C.CALL_OUTCOME_ID = CALL_OUTCOME.ID ORDER BY C.START_TIME ASC'
     ]
 
 
@@ -93,7 +99,21 @@ def test_extract_expressions():
 
     assert exprs[0] == {
         "CompoundIdentifier": [
-            {"value": "employee", "quote_style": None},
-            {"value": "first_name", "quote_style": None},
+            {
+                "value": "employee",
+                "quote_style": None,
+                "span": {
+                    "end": {"column": 20, "line": 2},
+                    "start": {"column": 12, "line": 2},
+                },
+            },
+            {
+                "value": "first_name",
+                "quote_style": None,
+                "span": {
+                    "end": {"column": 31, "line": 2},
+                    "start": {"column": 21, "line": 2},
+                },
+            },
         ]
     }