-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[Enhancement] files()
csv.trim_space
error report
#44740
Conversation
Quality Gate passedIssues Measures |
fe/fe-core/src/main/java/com/starrocks/catalog/TableFunctionTable.java
Outdated
Show resolved
Hide resolved
fe/fe-core/src/test/java/com/starrocks/catalog/TableFunctionTableTest.java
Outdated
Show resolved
Hide resolved
Signed-off-by: ricky <[email protected]>
3ef9327
to
f9b589a
Compare
fe/fe-core/src/main/java/com/starrocks/catalog/TableFunctionTable.java
Outdated
Show resolved
Hide resolved
Signed-off-by: ricky <[email protected]>
Head branch was pushed to by a user without write access
fe/fe-core/src/test/java/com/starrocks/catalog/TableFunctionTableTest.java
Outdated
Show resolved
Hide resolved
Signed-off-by: ricky <[email protected]>
Quality Gate passedIssues Measures |
[BE Incremental Coverage Report]✅ pass : 0 / 0 (0%) |
[FE Incremental Coverage Report]✅ pass : 6 / 6 (100.00%) file detail
|
@Mergifyio backport branch-3.3 |
@Mergifyio backport branch-3.2 |
✅ Backports have been created
|
✅ Backports have been created
|
Signed-off-by: ricky <[email protected]> (cherry picked from commit 144b65c)
Signed-off-by: ricky <[email protected]> (cherry picked from commit 144b65c) # Conflicts: # fe/fe-core/src/main/java/com/starrocks/catalog/TableFunctionTable.java # fe/fe-core/src/test/java/com/starrocks/catalog/TableFunctionTableTest.java
… (#46374) Co-authored-by: ricky <[email protected]>
Why I'm doing:
The
csv.trim_space
property offiles()
should be in bool type.Now, the
files()
will not report any error even if we setcsv.trim_space
as any illegal stringa
.What I'm doing:
This PR fixes this issue. When an illegal values is set,
Access storage error. Error message: illegal value of csv.trim_space: a, only true/false allowed
will be reported.Fixes #42800
What type of PR is this:
Does this PR entail a change in behavior?
If yes, please specify the type of change:
Checklist:
Bugfix cherry-pick branch check: