You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/09/01 12:12:42 UTC

[GitHub] [arrow-datafusion] kmitchener commented on pull request #3309: add SQL support for tinyint and all unsigned INTs

kmitchener commented on PR #3309:
URL: https://github.com/apache/arrow-datafusion/pull/3309#issuecomment-1234195415

   > > I've added the unsigned ints to this PR as well, matching what's in the issue
   > 
   > I have the opposite opinion for supporting the unsigned value in SQL Level now.
   > 
   > You can fill a new issue to discuss about this, I think this is big change for API and interface which is not compatible with other database or SQL system.
   
   Sure thing -- it's in #3325.
    


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org