alamb commented on PR #9770:
URL: 
https://github.com/apache/arrow-datafusion/pull/9770#issuecomment-2028460541

   > Thank you very much @progval 🙏
   > 
   > Can we please add tests for this code (so that we don't accidentally break 
it in some future refactoring?)
   
   Hi @progval  -- do you think you will have time to add some tests briefly
   
   I noticed that https://github.com/apache/arrow-datafusion/pull/9778 only 
adds tests for the signed (`Int8`/`Int16`/etc variants). I'll go get that PR 
ready to merge per our discussion as maybe you wrote it to add tests for this 
one 🤔 


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