-
Notifications
You must be signed in to change notification settings - Fork 44
feat: CSV Reader Configs #466
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
Conversation
Spark Test ReportCommit Information
Test Summary
Test DetailsError Counts
Passed Tests Diff(empty) Failed Tests
|
Ok(output) | ||
(Some(null_value), _) if !null_value.is_empty() => { | ||
// Convert null_value to regex by escaping special characters | ||
Ok(Some(regex::escape(&null_value))) |
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.
Nice!
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.
🚀
_ => Ok(None), | ||
}?; | ||
|
||
let file_format = CsvFormat::default() |
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.
Let's add a TODO here to return error if the char
is not an ASCII character, otherwise as u8
only returns the last byte.
No description provided.