Re: [I] Auto-update mechanism for dataframe test [datafusion]

2025-03-14 Thread via GitHub
alamb closed issue #10373: Auto-update mechanism for dataframe test URL: https://github.com/apache/datafusion/issues/10373 -- 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 un

Re: [I] Auto-update mechanism for dataframe test [datafusion]

2025-03-12 Thread via GitHub
alamb commented on issue #10373: URL: https://github.com/apache/datafusion/issues/10373#issuecomment-2718116201 > what do you think about [#15165](https://github.com/apache/datafusion/pull/15165) It is amazing😍 t- I recommend others check it out! -- This is an automated message fr

Re: [I] Auto-update mechanism for dataframe test [datafusion]

2025-03-11 Thread via GitHub
blaginin commented on issue #10373: URL: https://github.com/apache/datafusion/issues/10373#issuecomment-2715768413 what do you think about https://github.com/apache/datafusion/pull/15165/ -- This is an automated message from the Apache Git Service. To respond to the message, please log on

Re: [I] Auto-update mechanism for dataframe test [datafusion]

2025-03-11 Thread via GitHub
alamb commented on issue #10373: URL: https://github.com/apache/datafusion/issues/10373#issuecomment-2714659988 I just merged - https://github.com/apache/datafusion/pull/13672 I think it would be pretty cool to migrate a few tests to be `insta` based. Once we had some examples I th

Re: [I] Auto-update mechanism for dataframe test [datafusion]

2025-03-07 Thread via GitHub
Omega359 commented on issue #10373: URL: https://github.com/apache/datafusion/issues/10373#issuecomment-2706307419 I think this could be a good approach for dataframe testing. The existing assert_batches_eq!/assert_batches_sorted_eq! macro is nice but they do how the downside of requiring m

Re: [I] Auto-update mechanism for dataframe test [datafusion]

2025-03-07 Thread via GitHub
blaginin commented on issue #10373: URL: https://github.com/apache/datafusion/issues/10373#issuecomment-2706281206 I agree! And moreover, we'll be able to split "harness" (set up context, register files) from the actual test logic (stored in the small atomic snapshots files). Happy to work

Re: [I] Auto-update mechanism for dataframe test [datafusion]

2025-03-06 Thread via GitHub
alamb commented on issue #10373: URL: https://github.com/apache/datafusion/issues/10373#issuecomment-2704536889 I personally support introducing `insta` to datafusion testing - I think it would make maintaining tests easier and thus lower both the maintenance burden as well as the barrier t

Re: [I] Auto-update mechanism for dataframe test [datafusion]

2025-03-06 Thread via GitHub
wiedld commented on issue #10373: URL: https://github.com/apache/datafusion/issues/10373#issuecomment-2704455110 Outside of sqllogictests which already has an auto-update (see the `--complete` feature), I believe there is still an appetite for insta snapshots which auto-update. I se

Re: [I] Auto-update mechanism for dataframe test [datafusion]

2024-06-21 Thread via GitHub
jayzhan211 commented on issue #10373: URL: https://github.com/apache/datafusion/issues/10373#issuecomment-2180574806 > I think this might be another approach, and we use it in databend. https://crates.io/crates/goldenfile > > ``` > UPDATE_GOLDENFILES=1 cargo test > ``` >

Re: [I] Auto-update mechanism for dataframe test [datafusion]

2024-05-08 Thread via GitHub
alamb commented on issue #10373: URL: https://github.com/apache/datafusion/issues/10373#issuecomment-2101100092 Something we have used to great effect in influxdb is https://insta.rs/ You can then do the equivalent of `sqllogictest --complete` (even for results within files) with a co