alamb commented on PR #4547: URL: https://github.com/apache/arrow-datafusion/pull/4547#issuecomment-1344180626
FWIW if no one beats me to it I plan to try and polish the conversion to `Vec<Vec<String>>` this weekend, but I would be happy to have this one merged in first. I suspect it has been accumulating both actual conflicts (as github says) as well as logical conflicts with newly added SLT files. I poked around in the sqllogictest runner, and it looks like you already have some code that will run / update the test files with new results that looks super awesome (and relevant to updating results) https://github.com/risinglightdb/sqllogictest-rs/blob/d753e4c77fbf00cdaab69477df2e9c3dba3f5fcc/sqllogictest-bin/src/lib.rs#L505 -- 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]
