-
Notifications
You must be signed in to change notification settings - Fork 110
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: Type check DML #1727
feat: Type check DML #1727
Conversation
pub values: usize, | ||
pub fields: usize, | ||
} | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will be nice to know which fields
/values
failed here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added a TODO to that effect
@@ -89,6 +89,14 @@ pub enum Unsupported { | |||
UnqualifiedProjectExpr, | |||
} | |||
|
|||
#[derive(Error, Debug)] | |||
#[error("Inserting a row with {values} values into {table} which has {fields} fields")] | |||
pub struct InsertError { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Other errors use the convention of enclosing the error fields with "`" as Inserting a row with `{values}`...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍 I updated the all the other error messages as well.
206c618
to
6233a33
Compare
6af3279
to
9c03e47
Compare
1de71d5
to
e956f98
Compare
Description of Changes
Added type checking and name resolution for SQL DML statements
Expected complexity level and risk
2
Testing