Rafferty97 opened a new pull request, #9557:
URL: https://github.com/apache/arrow-rs/pull/9557

   # Which issue does this PR close?
   
   Another smaller PR extracted from #9494.
   
   # Rationale for this change
   
   I've moved `ValueIter` into its own module because it's already 
self-contained, and because that will make it easier to review the changes I 
have made to `arrow-json/src/reader/schema.rs`.
   
   I've also added a public `record_count` function to `ValueIter` - which can 
be used to simplify consuming code in Datafusion which is currently tracking it 
separately.
   
   # What changes are included in this PR?
   
   * Moved `ValueIter` into own module
   * Added `record_count` method to `ValueIter`
   
   # Are these changes tested?
   
   Yes.
   
   # Are there any user-facing changes?
   
   Addition of one new public method, `ValueIter::record_count`.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to