github-actions[bot] commented on PR #24930: URL: https://github.com/apache/datafusion/pull/24930#issuecomment-5531564554
<!-- semver-check-comment --> Thank you for opening this pull request! Reviewer note: [cargo-semver-checks](https://github.com/obi1kenobi/cargo-semver-checks) reported the current version number is not SemVer-compatible with the changes in this pull request (compared against the base branch). <details> <summary>Details</summary> ``` Cloning apache/main Building datafusion-datasource-parquet v55.0.0 (current) Built [ 69.044s] (current) Parsing datafusion-datasource-parquet v55.0.0 (current) Parsed [ 0.032s] (current) Building datafusion-datasource-parquet v55.0.0 (baseline) Built [ 46.640s] (baseline) Parsing datafusion-datasource-parquet v55.0.0 (baseline) Parsed [ 0.031s] (baseline) Checking datafusion-datasource-parquet v55.0.0 -> v55.0.0 (no change; assume patch) Checked [ 0.223s] 223 checks: 223 pass, 31 skip Summary no semver update required Finished [ 117.803s] datafusion-datasource-parquet Building datafusion-proto v55.0.0 (current) error: running cargo-doc on crate 'datafusion-proto' failed with output: ----- Compiling proc-macro2 v1.0.107 Compiling unicode-ident v1.0.24 Compiling quote v1.0.47 Compiling libc v0.2.189 Checking cfg-if v1.0.4 Checking memchr v2.8.3 Compiling autocfg v1.5.1 Compiling libm v0.2.16 Compiling num-traits v0.2.19 Compiling syn v2.0.119 Compiling syn v3.0.4 Compiling zerocopy v0.8.56 Compiling jobserver v0.1.35 Compiling shlex v2.0.1 Compiling find-msvc-tools v0.1.11 Checking bytes v1.12.1 Compiling cc v1.4.4 Compiling serde_core v1.0.229 Checking itoa v1.0.18 Checking foldhash v0.2.0 Checking allocator-api2 v0.2.21 Compiling getrandom v0.3.4 Checking once_cell v1.21.4 Checking equivalent v1.0.2 Compiling synstructure v0.13.2 Checking hashbrown v0.17.1 Compiling zmij v1.0.23 Checking indexmap v2.14.1 Compiling serde_json v1.0.151 Checking num-integer v0.1.47 Checking siphasher v1.0.3 Compiling version_check v0.9.5 Checking iana-time-zone v0.1.65 Checking chrono v0.4.45 Compiling zerocopy-derive v0.8.56 Compiling zerofrom-derive v0.1.7 Compiling ahash v0.8.12 Compiling yoke-derive v0.8.2 Checking phf_shared v0.12.1 Checking num-bigint v0.5.1 Checking zerofrom v0.1.8 Checking stable_deref_trait v1.2.1 Compiling chrono-tz v0.10.4 Checking phf v0.12.1 Checking arrow-schema v59.3.0 Checking yoke v0.8.3 Compiling pkg-config v0.3.34 Compiling zerovec-derive v0.11.6 Checking num-complex v0.4.6 Compiling displaydoc v0.2.7 Checking zerovec v0.11.8 Checking pin-project-lite v0.2.17 Compiling zstd-sys v2.0.16+zstd.1.5.7 Checking smallvec v1.16.0 Checking tinystr v0.8.4 Checking futures-sink v0.3.34 Checking lexical-util v1.0.7 Checking litemap v0.8.3 Checking writeable v0.6.4 Checking futures-core v0.3.34 Checking icu_locale_core v2.3.0 Checking potential_utf v0.1.6 Checking zerotrie v0.2.5 Compiling icu_normalizer_data v2.3.0 Checking utf8_iter v1.0.4 Compiling icu_properties_data v2.3.0 Checking half v2.7.1 Checking icu_collections v2.3.0 Checking arrow-buffer v59.3.0 Checking icu_provider v2.3.1 Compiling semver v1.0.28 Checking arrow-data v59.3.0 Compiling zstd-safe v7.2.4 Compiling rustc_version v0.4.1 Checking futures-channel v0.3.34 Checking lexical-parse-integer v1.0.6 Checking arrow-array v59.3.0 Checking lexical-write-integer v1.0.6 Compiling tokio-macros v2.7.2 Compiling futures-macro v0.3.34 Compiling parking_lot_core v0.9.12 Checking slab v0.4.12 Checking futures-task v0.3.34 Checking futures-io v0.3.34 Checking bitflags v2.13.1 Checking futures-util v0.3.34 Checking tokio v1.53.1 Checking arrow-select v59.3.0 Checking lexical-write-float v1.0.6 Checking lexical-parse-float v1.0.6 Checking icu_properties v2.3.0 Checking icu_normalizer v2.3.0 Compiling flatbuffers v25.12.19 Checking aho-corasick v1.1.5 Checking regex-syntax v0.8.11 Checking unicode-width v0.2.2 Checking unicode-segmentation v1.13.3 Checking ryu v1.0.23 Checking base64 v0.23.1 Compiling getrandom v0.4.3 Compiling crc32fast v1.5.1 Checking scopeguard v1.2.0 Checking lock_api v0.4.14 Checking comfy-table v7.2.2 Checking idna_adapter v1.2.2 Checking lexical-core v1.0.6 Checking arrow-ord v59.3.0 Checking regex-automata v0.4.18 Checking atoi v2.0.0 Checking simd-adler32 v0.3.10 Checking adler2 v2.0.1 Checking percent-encoding v2.3.2 Checking either v1.18.0 Checking twox-hash v2.1.4 Compiling snap v1.1.2 Compiling thiserror v2.0.20 Checking alloc-no-stdlib v2.0.4 Checking alloc-stdlib v0.2.4 Checking lz4_flex v0.14.0 Checking arrow-cast v59.3.0 Checking form_urlencoded v1.2.2 Checking miniz_oxide v0.9.1 Checking idna v1.1.0 Checking regex v1.13.1 Checking futures-executor v0.3.34 Compiling tracing-attributes v0.1.31 Compiling thiserror-impl v2.0.20 Checking tracing-core v0.1.36 Checking csv-core v0.1.13 Checking zlib-rs v0.6.7 Checking simdutf8 v0.1.5 Checking same-file v1.0.6 Checking walkdir v2.5.0 Checking tracing v0.1.44 Checking csv v1.4.0 Checking futures v0.3.34 Checking url v2.5.8 Checking flate2 v1.1.10 Checking brotli-decompressor v5.0.3 Checking itertools v0.14.0 Checking parking_lot v0.12.5 Compiling async-trait v0.1.92 Checking http v1.5.0 Checking humantime v2.4.0 Compiling serde v1.0.229 Compiling anyhow v1.0.104 Checking object_store v0.13.2 Checking brotli v8.0.4 Checking arrow-csv v59.3.0 Checking arrow-json v59.3.0 Checking arrow-string v59.3.0 Checking zstd v0.13.3 Checking arrow-ipc v59.3.0 Checking uuid v1.26.0 Checking arrow-arith v59.3.0 Checking arrow-row v59.3.0 Compiling serde_derive v1.0.229 Compiling seq-macro v0.3.6 Checking log v0.4.34 Checking arrow v59.3.0 Checking parquet v59.3.0 Checking itertools v0.15.0 Compiling prost-derive v0.14.4 Checking base64 v0.22.1 Compiling pin-project-internal v1.1.13 Checking pin-project v1.1.13 Checking ppv-lite86 v0.2.21 Checking pbjson v0.9.0 Checking prost v0.14.4 Checking rand_core v0.9.5 Compiling crossbeam-utils v0.8.22 Compiling rustix v1.1.4 Checking rand_chacha v0.9.0 Checking datafusion-doc v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/doc) Checking linux-raw-sys v0.12.1 Checking rand v0.9.5 Checking tokio-util v0.7.19 Checking foldhash v0.1.5 Checking hashbrown v0.14.5 Checking fastrand v2.5.0 Checking hashbrown v0.15.5 Checking dashmap v6.2.1 Checking fixedbitset v0.5.7 Checking petgraph v0.8.3 Compiling datafusion-macros v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/macros) Checking tempfile v3.27.0 Checking hex v0.4.3 Checking datafusion-common-runtime v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/common-runtime) Checking datafusion-common v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/common) Checking glob v0.3.4 Compiling object v0.39.1 Compiling liblzma-sys v0.4.8 Compiling stacker v0.1.25 Checking libbz2-rs-sys v0.2.5 Compiling heck v0.5.0 Checking crc-catalog v2.5.0 Checking crc v3.4.0 Compiling strum_macros v0.28.0 Compiling ar_archive_writer v0.5.3 Checking datafusion-proto-common v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/proto-common) Checking datafusion-expr-common v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/expr-common) Compiling psm v0.1.32 Checking bzip2 v0.6.1 Checking tokio-stream v0.1.19 Compiling recursive-proc-macro-impl v0.1.1 Checking recursive v0.1.1 Checking datafusion-proto-models v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/proto-models) Checking liblzma v0.4.8 Checking arrow-avro v59.3.0 Checking datafusion-physical-expr-common v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/physical-expr-common) Checking datafusion-functions-aggregate-common v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/functions-aggregate-common) Checking datafusion-functions-window-common v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/functions-window-common) Checking datafusion-expr v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/expr) Checking datafusion-execution v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/execution) Checking datafusion-physical-expr v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/physical-expr) Checking datafusion-functions v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/functions) Checking datafusion-physical-plan v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/physical-plan) Checking datafusion-physical-expr-adapter v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/physical-expr-adapter) Checking datafusion-session v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/session) Checking datafusion-datasource v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/datasource) Checking datafusion-catalog v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/catalog) Checking datafusion-pruning v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/pruning) Checking datafusion-datasource-arrow v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/datasource-arrow) Checking datafusion-datasource-csv v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/datasource-csv) Checking datafusion-datasource-parquet v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/datasource-parquet) Checking datafusion-datasource-avro v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/datasource-avro) Checking datafusion-datasource-json v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/datasource-json) Checking datafusion-catalog-listing v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/catalog-listing) Checking datafusion-functions-table v55.0.0 (/home/runner/work/datafusion/datafusion/datafusion/functions-table) error[E0282]: type annotations needed --> /home/runner/work/datafusion/datafusion/datafusion/datasource-parquet/src/source.rs:1100:17 | 1100 | Ok(protobuf::PhysicalSortExprNodeCollection { | ^^ cannot infer type of the type parameter `E` declared on the enum `Result` | help: consider specifying the generic arguments | 1100 | Ok::<PhysicalSortExprNodeCollection, E>(protobuf::PhysicalSortExprNodeCollection { | +++++++++++++++++++++++++++++++++++++ error[E0283]: type annotations needed --> /home/runner/work/datafusion/datafusion/datafusion/datasource-parquet/src/source.rs:1100:17 | 1100 | Ok(protobuf::PhysicalSortExprNodeCollection { | ^^ cannot infer type of the type parameter `E` declared on the enum `Result` ... 1104 | )?, | - type must be known at this point | = note: multiple `impl`s satisfying `_: From<DataFusionError>` found in the following crates: `datafusion_common`, `datafusion_proto_common`: - impl From<DataFusionError> for arrow_schema::ArrowError; - impl From<DataFusionError> for datafusion_proto_common::from_proto::Error; - impl From<DataFusionError> for std::io::Error; help: consider specifying the generic arguments | 1100 | Ok::<PhysicalSortExprNodeCollection, E>(protobuf::PhysicalSortExprNodeCollection { | +++++++++++++++++++++++++++++++++++++ error[E0283]: type annotations needed --> /home/runner/work/datafusion/datafusion/datafusion/datasource-parquet/src/source.rs:1100:17 | 1100 | Ok(protobuf::PhysicalSortExprNodeCollection { | ^^ cannot infer type of the type parameter `E` declared on the enum `Result` ... 1107 | .transpose()?; | - type must be known at this point | = note: multiple `impl`s satisfying `DataFusionError: From<_>` found in the following crates: `datafusion_common`, `datafusion_proto_common`: - impl From<Box<(dyn std::error::Error + std::marker::Send + Sync + 'static)>> for DataFusionError; - impl From<arrow_schema::ArrowError> for DataFusionError; - impl From<datafusion_proto_common::from_proto::Error> for DataFusionError; - impl From<datafusion_proto_common::to_proto::Error> for DataFusionError; - impl From<object_store::Error> for DataFusionError; - impl From<object_store::path::Error> for DataFusionError; - impl From<parquet::errors::ParquetError> for DataFusionError; - impl From<std::fmt::Error> for DataFusionError; - impl From<std::io::Error> for DataFusionError; help: consider specifying the generic arguments | 1100 | Ok::<PhysicalSortExprNodeCollection, E>(protobuf::PhysicalSortExprNodeCollection { | +++++++++++++++++++++++++++++++++++++ Some errors have detailed explanations: E0282, E0283. For more information about an error, try `rustc --explain E0282`. error: could not compile `datafusion-datasource-parquet` (lib) due to 3 previous errors ----- error: failed to build rustdoc for crate datafusion-proto v55.0.0 note: this is usually due to a compilation error in the crate, and is unlikely to be a bug in cargo-semver-checks note: the following command can be used to reproduce the error: cargo new --lib example && cd example && echo '[workspace]' >> Cargo.toml && cargo add --path /home/runner/work/datafusion/datafusion/datafusion/proto --features avro,datafusion-datasource-avro,datafusion-datasource-parquet,default,json,parquet,recursive_protection,serde_json && cargo check && cargo doc Building datafusion-proto-models v55.0.0 (current) Built [ 24.554s] (current) Parsing datafusion-proto-models v55.0.0 (current) Parsed [ 0.158s] (current) Building datafusion-proto-models v55.0.0 (baseline) Built [ 24.289s] (baseline) Parsing datafusion-proto-models v55.0.0 (baseline) Parsed [ 0.125s] (baseline) Checking datafusion-proto-models v55.0.0 -> v55.0.0 (no change; assume patch) Checked [ 2.417s] 223 checks: 222 pass, 1 fail, 0 warn, 31 skip --- failure constructible_struct_adds_field: struct exhaustively constructible through public API adds field --- Description: A pub struct that could be exhaustively constructed with a literal using only public API has a new pub field, breaking existing exhaustive literals. ref: https://doc.rust-lang.org/reference/expressions/struct-expr.html impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.50.0/src/lints/constructible_struct_adds_field.ron Failed in: field ParquetScanExecNode.sort_order_for_reorder in /home/runner/work/datafusion/datafusion/datafusion/proto-models/src/generated/prost.rs:1974 field ParquetScanExecNode.reverse_row_groups in /home/runner/work/datafusion/datafusion/datafusion/proto-models/src/generated/prost.rs:1976 field ParquetScanExecNode.sort_order_for_reorder in /home/runner/work/datafusion/datafusion/datafusion/proto-models/src/generated/prost.rs:1974 field ParquetScanExecNode.reverse_row_groups in /home/runner/work/datafusion/datafusion/datafusion/proto-models/src/generated/prost.rs:1976 Summary semver requires new major version: 1 major and 0 minor checks failed Finished [ 52.865s] datafusion-proto-models error: aborting due to failure to build rustdoc for crate datafusion-proto v55.0.0 ``` </details> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
