ZENOTME commented on PR #135: URL: https://github.com/apache/iceberg-rust/pull/135#issuecomment-1893177443
Hi, I find that we can modify ``` /// The current file status of iceberg writer. It implement for the writer which write a single /// file. pub trait CurrentFileStatus { /// Get the current file path. fn current_file_path(&self) -> String; /// Get the current file row number. fn current_row_num(&self) -> usize; /// Get the current file written size. fn current_written_size(&self) -> usize; } ``` as ``` pub trait CurrentStatus<T> { fn current_status(&self) -> T } impl CurrentStatus<CurrentFileStatus> for DataFileWriter { ... } impl CurrentStatus<DataFileMetrics> for DataFileWriter { ... } ``` The following traits can also help export metrics in the future. -- 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. To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org