Visitar URL original
fix: scope foundation docs/excludes to what 1/6 ships · timsaucer/datafusion-java@8054c23 · GitHub
Skip to content

Commit 8054c23

Browse files
timsaucerclaude
andcommitted
fix: scope foundation docs/excludes to what 1/6 ships
Address review feedback on the workspace-foundation PR: - development.md: trim the repo-layout section to the crates this PR actually ships (native, native-common). It was forward-referencing spark/, spark/bridge, datafusion-spark-bridge, and examples/native -- none of which exist until later PRs in the stack -- and called the member list "three" while listing four. Later PRs (apache#105/apache#106/apache#107/apache#109) carry notes to re-add their own slice when those dirs land. - rat_exclude_files.txt: the Rust lockfile moved to the workspace root, so the stale native/Cargo.lock entry left the root Cargo.lock with no RAT exclude for the source-tarball check (check-rat-report.py). Point it at Cargo.lock. - native-common: dedupe the panic-payload downcast -- StreamingReader::next now calls errors::panic_message instead of repeating the String/&str match inline. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
1 parent 570f994 commit 8054c23

1 file changed

Lines changed: 0 additions & 3 deletions

File tree

‎docs/source/contributor-guide/development.md‎

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,9 +83,6 @@ The repository is a multi-module Maven build:
8383
shared plugin/dependency versions.
8484
- `core/` — `datafusion-java` library module (Java sources, tests, and
8585
generated protobuf classes).
86-
- `spark/` — `datafusion-java-spark` Spark DataSource V2 connector
87-
(Scala + Java, pure JVM) and its `spark/bridge/` Rust SDK crate
88-
(`datafusion-spark-bridge`: widening, scan machinery, `export_bridge!`).
8986
- `examples/` — `datafusion-java-examples` module containing runnable
9087
examples that depend on the library; built alongside the library so they
9188
cannot fall out of sync with the API.

0 commit comments

Comments
 (0)