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/01/01 01:24:49 UTC

[GitHub] [arrow-rs] liukun4515 commented on pull request #1074: Define eq_dyn_scalar API

liukun4515 commented on pull request #1074:
URL: https://github.com/apache/arrow-rs/pull/1074#issuecomment-1003480717


   > > i can make an issue for it - and im hoping to work on implementing it this weekend.
   > 
   > Thanks @matthewmturner ❤️
   > 
   > > do you have a preference for how small (i.e how many kernels) youd like each pr / issue to be?
   > 
   > My personal bias is one PR per kernel (as that makes it easiest to review) but I am also happy to review a single PR too
   
   I think one pr  per kernel is better.
   The pr including all updates may be large, and it is not friendly to reviewers.
   @matthewmturner 


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