alamb commented on PR #1581: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1581#issuecomment-2529390710
> Thanks for drafting this @davisp! It does sound reasonable to me, I imagine this could help editors as you mention. cc @alamb for thoughts if something like this could be desirable I think breaking the code up into smaller modules sounds like a great idea. Thank you @davisp > I have no illusions that folks will want to merge this directly, but I figured it was a useful enough exercise that I'd show my work in case there's any desire to start doing something of this nature piecemeal with an eye towards having people that actually know this code base help refine some of my rather arbitrary decisions on the reorganization. Indeed, I think doing this refactoring as a series of PRs would be great. -- 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...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
