Dandandan commented on pull request #8860:
URL: https://github.com/apache/arrow/pull/8860#issuecomment-742653929


   So the proposal would be to change the statistics struct to 
   
   ```rust
   pub struct Statistics {
       /// The number of table rows
       pub num_rows: Option<usize>,
       /// total byte of the table rows
       pub total_byte_size: Option<usize>,
   }
   ```
   
   and the Statistics doesn't need to be wrapped as `Option<Statistics>` 
anymore.


----------------------------------------------------------------
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]


Reply via email to