kylebarron opened a new pull request, #489:
URL: https://github.com/apache/arrow-js/pull/489

   ## What's Changed
   
   - We now always slice a `Data`'s `valueOffsets` to be exactly the length 
promised by the Arrow spec. Previously, a `valueOffsets` was often longer than 
what the spec defined.
   - Added tests to ensure that the length of offsets are correct.
   
   The change in `data.ts` I manually edited/rewrote from a suggestion by 
Claude.
   
   The new tests in `value-offsets-tests.ts` I reviewed and condensed from 
suggestions by Claude.
   
   Closes #484.
   


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