Re: [PR] Triggering extended tests through PR comment [datafusion]

2025-03-27 Thread via GitHub
danila-b commented on PR #15101: URL: https://github.com/apache/datafusion/pull/15101#issuecomment-2742784619 > Is this ready for review or is there something outstanding for it to be still in draft? It should be ready for a review, rebased on the latest main and tested it a bit more

Re: [PR] Triggering extended tests through PR comment [datafusion]

2025-03-26 Thread via GitHub
alamb commented on code in PR #15101: URL: https://github.com/apache/datafusion/pull/15101#discussion_r2013788126 ## .github/workflows/extended.yml: ## @@ -127,4 +145,44 @@ jobs: cargo test --features backtrace --profile release-nonlto --test sqllogictests -- --inclu

Re: [PR] Triggering extended tests through PR comment [datafusion]

2025-03-25 Thread via GitHub
alamb commented on PR #15101: URL: https://github.com/apache/datafusion/pull/15101#issuecomment-2752513176 Ok, all tests have passed initially ![Screenshot 2025-03-25 at 4 48 45  PM](https://github.com/user-attachments/assets/d34ac151-4e13-48b1-9731-71005f236c1d) I will now try and

Re: [PR] Triggering extended tests through PR comment [datafusion]

2025-03-25 Thread via GitHub
alamb commented on PR #15101: URL: https://github.com/apache/datafusion/pull/15101#issuecomment-2752311863 Thanks @danila-b -- you are totally right. I started a new test and we'll see how that goes - https://github.com/alamb/datafusion/pull/33 -- This is an automated message from the

Re: [PR] Triggering extended tests through PR comment [datafusion]

2025-03-24 Thread via GitHub
alamb commented on PR #15101: URL: https://github.com/apache/datafusion/pull/15101#issuecomment-2749403724 When I tested this on my fork, it seems to have run extended tests for my PR without any prompting: - https://github.com/alamb/datafusion/pull/32#issuecomment-2749402970 -- This i

Re: [PR] Triggering extended tests through PR comment [datafusion]

2025-03-24 Thread via GitHub
danila-b commented on PR #15101: URL: https://github.com/apache/datafusion/pull/15101#issuecomment-2749439048 > When I tested this on my fork, it seems to have run extended tests for my PR without any prompting: > > * [Test for triggering extended tests from PR comments alamb/datafusi

Re: [PR] Triggering extended tests through PR comment [datafusion]

2025-03-24 Thread via GitHub
alamb commented on PR #15101: URL: https://github.com/apache/datafusion/pull/15101#issuecomment-2749093157 Thanks @danila-b and @Omega359 🙏 -- I started testing this on my fork here: - https://github.com/alamb/datafusion/pull/32 -- This is an automated message from the Apache Git Serv

Re: [PR] Triggering extended tests through PR comment [datafusion]

2025-03-22 Thread via GitHub
Omega359 commented on PR #15101: URL: https://github.com/apache/datafusion/pull/15101#issuecomment-2743066275 @alamb -- 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 uns

Re: [PR] Triggering extended tests through PR comment [datafusion]

2025-03-18 Thread via GitHub
Omega359 commented on PR #15101: URL: https://github.com/apache/datafusion/pull/15101#issuecomment-2733251482 Is this ready for review or is there something outstanding for it to be still in draft? -- This is an automated message from the Apache Git Service. To respond to the message, ple

Re: [PR] Triggering extended tests through PR comment [datafusion]

2025-03-08 Thread via GitHub
Omega359 commented on PR #15101: URL: https://github.com/apache/datafusion/pull/15101#issuecomment-2708410057 Looks like it ran checks on the forked branch, I think that is what we want ![image](https://github.com/user-attachments/assets/de280fc4-b351-4141-920d-b62012b889ff) --

[PR] Triggering extended tests through PR comment [datafusion]

2025-03-08 Thread via GitHub
danila-b opened a new pull request, #15101: URL: https://github.com/apache/datafusion/pull/15101 ## Which issue does this PR close? Closes https://github.com/apache/datafusion/issues/14319 ## Rationale for this change Allows to run extended tests on some PRs if needed, wh