Skip to content

Commit 6f285d6

Browse files
authored
Fix wasm32 build on version 46 (#15102)
* fix build * try to install clang in ci
1 parent 04d823b commit 6f285d6

File tree

4 files changed

+6
-4
lines changed

4 files changed

+6
-4
lines changed

.github/workflows/rust.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -259,6 +259,10 @@ jobs:
259259
uses: ./.github/actions/setup-builder
260260
with:
261261
rust-version: stable
262+
- name: Install dependencies
263+
run: |
264+
apt-get update -qq
265+
apt-get install -y -qq clang
262266
- name: Install wasm-pack
263267
run: curl https://rustwasm.github.io/wasm-pack/installer/init.sh -sSf | sh
264268
- name: Build with wasm-pack

datafusion/datasource-parquet/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ object_store = { workspace = true }
5252
parking_lot = { workspace = true }
5353
parquet = { workspace = true }
5454
rand = { workspace = true }
55-
tokio = { workspace = true, features = ["fs"] }
55+
tokio = { workspace = true }
5656

5757
[dev-dependencies]
5858
chrono = { workspace = true }

datafusion/datasource-parquet/src/file_format.rs

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -466,7 +466,6 @@ impl FileFormat for ParquetFormat {
466466
}
467467

468468
/// Coerces the file schema if the table schema uses a view type.
469-
#[cfg(not(target_arch = "wasm32"))]
470469
pub fn coerce_file_schema_to_view_type(
471470
table_schema: &Schema,
472471
file_schema: &Schema,
@@ -516,7 +515,6 @@ pub fn coerce_file_schema_to_view_type(
516515
/// If the table schema uses a string type, coerce the file schema to use a string type.
517516
///
518517
/// See [ParquetFormat::binary_as_string] for details
519-
#[cfg(not(target_arch = "wasm32"))]
520518
pub fn coerce_file_schema_to_string_type(
521519
table_schema: &Schema,
522520
file_schema: &Schema,

datafusion/wasmtest/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ chrono = { version = "0.4", features = ["wasmbind"] }
4545
# all the `std::fmt` and `std::panicking` infrastructure, so isn't great for
4646
# code size when deploying.
4747
console_error_panic_hook = { version = "0.1.1", optional = true }
48-
datafusion = { workspace = true }
48+
datafusion = { workspace = true, features = ["parquet"] }
4949
datafusion-common = { workspace = true, default-features = true }
5050
datafusion-execution = { workspace = true }
5151
datafusion-expr = { workspace = true }

0 commit comments

Comments
 (0)