neilconway commented on PR #24682: URL: https://github.com/apache/datafusion/pull/24682#issuecomment-5427708491
@tanejaansh007 Thanks for your contribution! However, I suspect there is very little chance of such a large PR being reviewed, let alone merged, without more work done to motivate why this change is being proposed, the problem that is being solved, the context and design, describe the alternatives considered, and so on. Where possible, breaking changes down into smaller individual PRs makes them easier to review. It would also help if you would follow the PR template's guidelines. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
