aljaz/turso
1
0
Fork 0
mirror of https://github.com/aljazceru/turso.git synced 2026-01-03 16:34:19 +01:00
Code Issues Packages Projects Releases Wiki Activity

don't allow duplicate columns and get column type more precisely

Browse Source
This commit is contained in:
Pavan-Nambi
2025-09-22 16:24:22 +05:30
parent 25754cb0e4
commit 8dc485e5f7
2 changed files with 8 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
core/schema.rs
View File

@@ -1294,7 +1294,7 @@ impl From<&ColumnDefinition> for Column {
let ty_str = value
.col_type
.as_ref()
.map(|t| t.name.to_string())
.map(|t| t.to_string())
.unwrap_or_default();
let hidden = ty_str.contains("HIDDEN");

7
core/translate/alter.rs
View File

@@ -207,6 +207,13 @@ pub fn translate_alter_table(
}
}
let new_column_name = column.name.as_ref().unwrap();
if btree.get_column(new_column_name).is_some() {
return Err(LimboError::ParseError(
"duplicate column name: ".to_string() + new_column_name,
));
}
// TODO: All quoted ids will be quoted with `[]`, we should store some info from the parsed AST
btree.columns.push(column.clone());