You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@datafusion.apache.org by "alamb (via GitHub)" <gi...@apache.org> on 2025/03/30 11:05:52 UTC

Re: [PR] Migrate datafusion/sql tests to insta, part1 [datafusion]

alamb commented on PR #15484:
URL: https://github.com/apache/datafusion/pull/15484#issuecomment-2764504770

   > The code changes is now done, please review carefully as the code changes is LARGE.
   
   
   
   Thank you so much @qstommyshu  -- this looks epic
   
   Is there any chance you can break this PR up into several smaller ones for easier review? For example, maybe take the first 500 lines or so of diffs and make a seaparate PR for it?
   
   It is very hard for me to find enough contiguous time to review 1000s of lines carefully.
   
   


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

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscribe@datafusion.apache.org
For additional commands, e-mail: github-help@datafusion.apache.org