Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run all datafusion-examples in CI tests #3761

Merged
merged 9 commits into from
Oct 12, 2022
18 changes: 15 additions & 3 deletions .github/workflows/rust.yml
Original file line number Diff line number Diff line change
Expand Up @@ -104,13 +104,25 @@ jobs:
run: |
export PATH=$PATH:$HOME/d/protoc/bin
cargo test --features avro,jit,scheduler,json
- name: Run examples
run: |
export PATH=$PATH:$HOME/d/protoc/bin
# test datafusion-sql examples
cargo run --example sql
# test datafusion examples
cd datafusion-examples
# test datafusion-examples
cargo run --example avro_sql --features=datafusion/avro
cargo run --example csv_sql
cargo run --example custom_datasource
cargo run --example dataframe
cargo run --example dataframe_in_memory
cargo run --example deserialize_to_struct
cargo run --example expr_api
cargo run --example parquet_sql
cargo run --example avro_sql --features=datafusion/avro
cargo run --example parquet_sql_multiple_files
cargo run --example memtable
cargo run --example simple_udf
cargo run --example simple_udaf


integration-test:
name: "Compare to postgres"
Expand Down
6 changes: 3 additions & 3 deletions datafusion-examples/examples/parquet_sql_multiple_files.rs
Original file line number Diff line number Diff line change
Expand Up @@ -56,9 +56,9 @@ async fn main() -> Result<()> {
// execute the query
let df = ctx
.sql(
"SELECT int_col, double_col, CAST(date_string_col as VARCHAR) \
FROM alltypes_plain \
WHERE id > 1 AND tinyint_col < double_col",
"SELECT * \
FROM my_table \
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The table was called "my_table" but the query referred to something else 😆

LIMIT 1",
)
.await?;

Expand Down