Skip to content

Commit c0b749d

Browse files
authored
Remove unused crate dependencies (apache#14827)
* Enable 'extended tests' on forks Allow contributors to run extended tests workflow if they wish to, just like they can run rust tests workflow on their forks, before opening a PR to DataFusion. GitHub allows enabling/disabling workflows in the web UI without needing to change workflow yaml file. * Remove unused crate dependencies Found by `cargo udeps`. Unfortunately there were false positives too. * one on workspace level
1 parent 1c54b38 commit c0b749d

File tree

5 files changed

+1
-11
lines changed

5 files changed

+1
-11
lines changed

.github/workflows/extended.yml

+1-4
Original file line numberDiff line numberDiff line change
@@ -23,16 +23,13 @@ concurrency:
2323

2424
# https://docs.github.com/en/actions/writing-workflows/choosing-when-your-workflow-runs/events-that-trigger-workflows#running-your-pull_request-workflow-when-a-pull-request-merges
2525
#
26-
# These jobs only run on the `main` branch as they are time consuming
26+
# These jobs are not run as part of PR checks as they are time-consuming
2727
# and should not fail often.
2828
#
2929
# We still run them as they provide important coverage to ensure correctness
3030
# in the (very rare) event of a hash failure or sqlite library query failure.
3131
on:
32-
# Run on all commits to main
3332
push:
34-
branches:
35-
- main
3633

3734
jobs:
3835
# Check crate compiles and base cargo check passes

Cargo.lock

-3
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

Cargo.toml

-1
Original file line numberDiff line numberDiff line change
@@ -146,7 +146,6 @@ pbjson = { version = "0.7.0" }
146146
pbjson-types = "0.7"
147147
# Should match arrow-flight's version of prost.
148148
prost = "0.13.1"
149-
prost-derive = "0.13.1"
150149
rand = "0.8.5"
151150
recursive = "0.1.1"
152151
regex = "1.8"

datafusion/physical-optimizer/Cargo.toml

-1
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ recursive_protection = ["dep:recursive"]
4040
[dependencies]
4141
arrow = { workspace = true }
4242
datafusion-common = { workspace = true, default-features = true }
43-
datafusion-datasource = { workspace = true }
4443
datafusion-execution = { workspace = true }
4544
datafusion-expr = { workspace = true }
4645
datafusion-expr-common = { workspace = true, default-features = true }

datafusion/wasmtest/Cargo.toml

-2
Original file line numberDiff line numberDiff line change
@@ -50,14 +50,12 @@ datafusion-common = { workspace = true, default-features = true }
5050
datafusion-execution = { workspace = true }
5151
datafusion-expr = { workspace = true }
5252
datafusion-optimizer = { workspace = true, default-features = true }
53-
datafusion-physical-expr = { workspace = true, default-features = true }
5453
datafusion-physical-plan = { workspace = true }
5554
datafusion-sql = { workspace = true }
5655
# getrandom must be compiled with js feature
5756
getrandom = { version = "0.2.8", features = ["js"] }
5857

5958
wasm-bindgen = "0.2.99"
60-
wasm-bindgen-futures = "0.4.49"
6159

6260
[dev-dependencies]
6361
tokio = { workspace = true }

0 commit comments

Comments
 (0)