Skip to content

[branch-50] Set update = none for datafusion-testing git submodule - #53

Merged
bcmyers merged 2 commits into
DataDog:branch-50from
bcmyers:bcmyers/datafusion-testing
Oct 14, 2025
Merged

[branch-50] Set update = none for datafusion-testing git submodule#53
bcmyers merged 2 commits into
DataDog:branch-50from
bcmyers:bcmyers/datafusion-testing

Conversation

@bcmyers

@bcmyers bcmyers commented Oct 14, 2025

Copy link
Copy Markdown

We've been seeing issues with some users in dd-source failing to be able to (via cargo) clone the datafusion repo due to cargo failing to clone the datafusion-test submodule. Let's make it so cargo doesn't try to do that.

@gabotechs gabotechs changed the title Set update = none for datafusion-testing git submodule [branch-50] Set update = none for datafusion-testing git submodule Oct 14, 2025
Comment thread .gitmodules Outdated
@bcmyers
bcmyers force-pushed the bcmyers/datafusion-testing branch from 1fe9d38 to 8c5ce44 Compare October 14, 2025 11:05
@bcmyers

bcmyers commented Oct 14, 2025

Copy link
Copy Markdown
Author

/merge

@dd-devflow-routing-codex

dd-devflow-routing-codex Bot commented Oct 14, 2025

Copy link
Copy Markdown

View all feedbacks in Devflow UI.

2025-10-14 11:31:00 UTC ℹ️ Start processing command /merge


2025-10-14 11:31:05 UTC ℹ️ MergeQueue: pull request added to the queue

The expected merge time in branch-50 is approximately 0s (p90).


2025-10-14 11:59:44 UTCMergeQueue: The checks failed on this merge request

Tests failed on this commit 5d0092c:

What to do next?

  • Investigate the failures and when ready, re-add your pull request to the queue!
  • If your PR checks are green, try to rebase/merge. It might be because the CI run is a bit old.
  • Any question, go check the FAQ.

@bcmyers
bcmyers merged commit ff2533c into DataDog:branch-50 Oct 14, 2025
29 of 30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants