GeorgeLeePatterson commented on PR #321: URL: https://github.com/apache/arrow-js/pull/321#issuecomment-3478409364
@kou Yes! After checking it against your comment, this PR does fix issue #15 it looks like. Both issues stem from the same root cause in `toIntervalMonthDayNanoInt32Array()`. The function is called by the integration test JSON loader (`src/visitor/vectorloader.ts:215`), which is where issue #15's test failures occurred. The `>>> 0` fix ensures proper unsigned 32-bit representation and should resolve both the integration test failures and the precision loss I encountered. -- 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]
