[GitHub] [arrow-datafusion] alamb commented on pull request #6384: feat: New functions and operations for working with arrays

["alamb (via GitHub)" <gi...@apache.org>]

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

   > What do you think, @alamb?
   
   I think this would be ok -- especially as you have a history of continued contribution. However, there are a few instances where engnaged committers committed in the start of promising features (such as the analysis framework from @isidentical) and then were not able to to finish the work for whatever reason. While this is fine, I think it would be better for datafusion to avoid it.
   
   Thus I would like to suggest an alternate approach which is to break this PR down into several smaller ones (perhaps one for each new function?) That way we can give each function the attention during review it deserves (and maybe even parallelize the work)
   
   We have a much better track record of being able to review and merge smaller PRs quickly than single large PRs. So when the functionality can be split up I think that is the best plan.
   
   What do you think @izveigor ?


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