yjshen commented on pull request #1782:
URL: 
https://github.com/apache/arrow-datafusion/pull/1782#issuecomment-1035106524


   Thanks @alamb for all the thoughts! 
   
   The PR currently relies on `arrow = { git = 
"https://github.com/apache/arrow-rs.git";, rev = 
"731e132489b99cd688f884642cf20de52aed24d0" }` since the need to access 
ArrayBuilder function. Do you think I should  make this PR behind a non-default 
feature and just remove the `patch.crates-io` section?
   
   Besides, I'd like to incorporate you `bit_util` comments as well the row 
diagram into the PR.


-- 
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-unsubscr...@arrow.apache.org

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


Reply via email to