houqp commented on a change in pull request #9534:
URL: https://github.com/apache/arrow/pull/9534#discussion_r579630159
##########
File path: rust/arrow/src/csv/reader.rs
##########
@@ -121,18 +141,12 @@ fn infer_file_schema<R: Read + Seek>(
.collect()
};
- // save the csv reader position after reading headers
- let position = csv_reader.position().clone();
-
let header_length = headers.len();
// keep track of inferred field types
let mut column_types: Vec<HashSet<DataType>> = vec![HashSet::new();
header_length];
// keep track of columns with nulls
let mut nulls: Vec<bool> = vec![false; header_length];
- // return csv reader position to after headers
- csv_reader.seek(position)?;
Review comment:
this is a no op.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
