Skip to content

Commit 42c8743

Browse files
committed
add more insightful suggestions
1 parent fad038e commit 42c8743

File tree

2 files changed

+19
-10
lines changed

2 files changed

+19
-10
lines changed

src/parser/mod.rs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12218,7 +12218,7 @@ impl<'a> Parser<'a> {
1221812218
})
1221912219
} else {
1222012220
self.expected(
12221-
" DATA` or an extension name after `LOAD`",
12221+
"`DATA` or an extension name after `LOAD`",
1222212222
self.peek_token(),
1222312223
)
1222412224
}

tests/sqlparser_common.rs

Lines changed: 18 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -11564,8 +11564,13 @@ fn parse_load_data() {
1156411564
.unwrap_err(),
1156511565
ParserError::ParserError("Expected: end of statement, found: INPATH".to_string())
1156611566
);
11567-
assert_eq!( not_supports_load_dialects.parse_sql_statements(sql).unwrap_err(),
11568-
ParserError::ParserError("Expected: dialect supports `LOAD DATA` or `LOAD extension` to parse `LOAD` statements, found: INPATH".to_string())
11567+
assert_eq!(
11568+
not_supports_load_dialects
11569+
.parse_sql_statements(sql)
11570+
.unwrap_err(),
11571+
ParserError::ParserError(
11572+
"Expected: `DATA` or an extension name after `LOAD`, found: INPATH".to_string()
11573+
)
1156911574
);
1157011575

1157111576
// with LOCAL keyword
@@ -11599,8 +11604,12 @@ fn parse_load_data() {
1159911604
ParserError::ParserError("Expected: end of statement, found: LOCAL".to_string())
1160011605
);
1160111606
assert_eq!(
11602-
not_supports_load_dialects.parse_sql_statements(sql).unwrap_err(),
11603-
ParserError::ParserError("Expected: dialect supports `LOAD DATA` or `LOAD extension` to parse `LOAD` statements, found: LOCAL".to_string())
11607+
not_supports_load_dialects
11608+
.parse_sql_statements(sql)
11609+
.unwrap_err(),
11610+
ParserError::ParserError(
11611+
"Expected: `DATA` or an extension name after `LOAD`, found: LOCAL".to_string()
11612+
)
1160411613
);
1160511614

1160611615
// with PARTITION clause
@@ -11689,11 +11698,11 @@ fn parse_load_data() {
1168911698
// negative test case
1169011699
let sql = "LOAD DATA2 LOCAL INPATH '/local/path/to/data.txt' INTO TABLE test.my_table";
1169111700
assert_eq!(
11692-
dialects
11693-
.parse_sql_statements(sql)
11694-
.unwrap_err(),
11695-
ParserError::ParserError("Expected: dialect supports `LOAD DATA` or `LOAD extension` to parse `LOAD` statements, found: DATA2".to_string())
11696-
);
11701+
dialects.parse_sql_statements(sql).unwrap_err(),
11702+
ParserError::ParserError(
11703+
"Expected: `DATA` or an extension name after `LOAD`, found: DATA2".to_string()
11704+
)
11705+
);
1169711706
}
1169811707

1169911708
#[test]

0 commit comments

Comments
 (0)