alamb commented on pull request #1810: URL: https://github.com/apache/arrow-datafusion/pull/1810#issuecomment-1041388425
Given how large this PR is getting (and thus its potential for accumulating conflicts) I am going to merge it to main and we can keep iterating in future PRs. I looked at the error handling commit at https://github.com/apache/arrow-datafusion/pull/1810/commits/12b1c738a5754c62efea05cfc2d0b84e8180355e and it looked better than `unwrap()` to me (though there is likely still significant room for UX improvement with error handling). -- 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
