github-actions[bot] commented on PR #25328: URL: https://github.com/apache/datafusion/pull/25328#issuecomment-5679158023
<!-- 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 v55.1.0 (current) Built [ 63.434s] (current) Parsing datafusion v55.1.0 (current) Parsed [ 0.035s] (current) Building datafusion v55.1.0 (baseline) Built [ 58.031s] (baseline) Parsing datafusion v55.1.0 (baseline) Parsed [ 0.034s] (baseline) Checking datafusion v55.1.0 -> v55.1.0 (no change; assume patch) Checked [ 0.568s] 223 checks: 223 pass, 31 skip Summary no semver update required Finished [ 124.358s] datafusion Building datafusion-catalog-listing v55.1.0 (current) Built [ 40.550s] (current) Parsing datafusion-catalog-listing v55.1.0 (current) Parsed [ 0.011s] (current) Building datafusion-catalog-listing v55.1.0 (baseline) Built [ 41.099s] (baseline) Parsing datafusion-catalog-listing v55.1.0 (baseline) Parsed [ 0.012s] (baseline) Checking datafusion-catalog-listing v55.1.0 -> v55.1.0 (no change; assume patch) Checked [ 0.081s] 223 checks: 223 pass, 31 skip Summary no semver update required Finished [ 82.691s] datafusion-catalog-listing Building datafusion-common v55.1.0 (current) Built [ 34.358s] (current) Parsing datafusion-common v55.1.0 (current) Parsed [ 0.065s] (current) Building datafusion-common v55.1.0 (baseline) Built [ 34.267s] (baseline) Parsing datafusion-common v55.1.0 (baseline) Parsed [ 0.066s] (baseline) Checking datafusion-common v55.1.0 -> v55.1.0 (no change; assume patch) Checked [ 0.671s] 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 ParquetOptions.eager_pruning in /home/runner/work/datafusion/datafusion/datafusion/common/src/config.rs:1371 field ParquetOptions.eager_pruning_file_limit in /home/runner/work/datafusion/datafusion/datafusion/common/src/config.rs:1371 Summary semver requires new major version: 1 major and 0 minor checks failed Finished [ 70.452s] datafusion-common Building datafusion-datasource v55.1.0 (current) Built [ 44.706s] (current) Parsing datafusion-datasource v55.1.0 (current) Parsed [ 0.031s] (current) Building datafusion-datasource v55.1.0 (baseline) Built [ 43.768s] (baseline) Parsing datafusion-datasource v55.1.0 (baseline) Parsed [ 0.033s] (baseline) Checking datafusion-datasource v55.1.0 -> v55.1.0 (no change; assume patch) Checked [ 0.251s] 223 checks: 222 pass, 1 fail, 0 warn, 31 skip --- failure trait_method_parameter_count_changed: pub trait method parameter count changed --- Description: A trait method now takes a different number of parameters. ref: https://doc.rust-lang.org/cargo/reference/semver.html#trait-item-signature impl: https://github.com/obi1kenobi/cargo-semver-checks/tree/v0.50.0/src/lints/trait_method_parameter_count_changed.ron Failed in: FileFormat::create_physical_plan now takes 3 instead of 2 parameters, in file /home/runner/work/datafusion/datafusion/datafusion/datasource/src/file_format.rs:201 Summary semver requires new major version: 1 major and 0 minor checks failed Finished [ 89.815s] datafusion-datasource Building datafusion-datasource-arrow v55.1.0 (current) Built [ 44.168s] (current) Parsing datafusion-datasource-arrow v55.1.0 (current) Parsed [ 0.012s] (current) Building datafusion-datasource-arrow v55.1.0 (baseline) Built [ 43.577s] (baseline) Parsing datafusion-datasource-arrow v55.1.0 (baseline) Parsed [ 0.012s] (baseline) Checking datafusion-datasource-arrow v55.1.0 -> v55.1.0 (no change; assume patch) Checked [ 0.072s] 223 checks: 223 pass, 31 skip Summary no semver update required Finished [ 88.754s] datafusion-datasource-arrow Building datafusion-datasource-avro v55.1.0 (current) Built [ 45.248s] (current) Parsing datafusion-datasource-avro v55.1.0 (current) Parsed [ 0.010s] (current) Building datafusion-datasource-avro v55.1.0 (baseline) Built [ 44.605s] (baseline) Parsing datafusion-datasource-avro v55.1.0 (baseline) Parsed [ 0.011s] (baseline) Checking datafusion-datasource-avro v55.1.0 -> v55.1.0 (no change; assume patch) Checked [ 0.075s] 223 checks: 223 pass, 31 skip Summary no semver update required Finished [ 90.916s] datafusion-datasource-avro Building datafusion-datasource-csv v55.1.0 (current) Built [ 43.610s] (current) Parsing datafusion-datasource-csv v55.1.0 (current) Parsed [ 0.011s] (current) Building datafusion-datasource-csv v55.1.0 (baseline) Built [ 43.824s] (baseline) Parsing datafusion-datasource-csv v55.1.0 (baseline) Parsed [ 0.017s] (baseline) Checking datafusion-datasource-csv v55.1.0 -> v55.1.0 (no change; assume patch) Checked [ 0.100s] 223 checks: 223 pass, 31 skip Summary no semver update required Finished [ 88.533s] datafusion-datasource-csv Building datafusion-datasource-json v55.1.0 (current) Built [ 43.613s] (current) Parsing datafusion-datasource-json v55.1.0 (current) Parsed [ 0.012s] (current) Building datafusion-datasource-json v55.1.0 (baseline) Built [ 44.135s] (baseline) Parsing datafusion-datasource-json v55.1.0 (baseline) Parsed [ 0.013s] (baseline) Checking datafusion-datasource-json v55.1.0 -> v55.1.0 (no change; assume patch) Checked [ 0.090s] 223 checks: 223 pass, 31 skip Summary no semver update required Finished [ 88.803s] datafusion-datasource-json Building datafusion-datasource-parquet v55.1.0 (current) Built [ 49.440s] (current) Parsing datafusion-datasource-parquet v55.1.0 (current) Parsed [ 0.034s] (current) Building datafusion-datasource-parquet v55.1.0 (baseline) Built [ 49.166s] (baseline) Parsing datafusion-datasource-parquet v55.1.0 (baseline) Parsed [ 0.035s] (baseline) Checking datafusion-datasource-parquet v55.1.0 -> v55.1.0 (no change; assume patch) Checked [ 0.141s] 223 checks: 223 pass, 31 skip Summary no semver update required Finished [ 99.883s] datafusion-datasource-parquet Building datafusion-proto-common v55.1.0 (current) Built [ 22.446s] (current) Parsing datafusion-proto-common v55.1.0 (current) Parsed [ 0.046s] (current) Building datafusion-proto-common v55.1.0 (baseline) Built [ 22.387s] (baseline) Parsing datafusion-proto-common v55.1.0 (baseline) Parsed [ 0.050s] (baseline) Checking datafusion-proto-common v55.1.0 -> v55.1.0 (no change; assume patch) Checked [ 1.060s] 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 ParquetOptions.eager_pruning in /home/runner/work/datafusion/datafusion/datafusion/proto-common/src/generated/prost.rs:871 field ParquetOptions.eager_pruning_file_limit_opt in /home/runner/work/datafusion/datafusion/datafusion/proto-common/src/generated/prost.rs:907 field ParquetOptions.eager_pruning in /home/runner/work/datafusion/datafusion/datafusion/proto-common/src/generated/prost.rs:871 field ParquetOptions.eager_pruning_file_limit_opt in /home/runner/work/datafusion/datafusion/datafusion/proto-common/src/generated/prost.rs:907 field ParquetOptions.eager_pruning in /home/runner/work/datafusion/datafusion/datafusion/proto-common/src/generated/prost.rs:871 field ParquetOptions.eager_pruning_file_limit_opt in /home/runner/work/datafusion/datafusion/datafusion/proto-common/src/generated/prost.rs:907 Summary semver requires new major version: 1 major and 0 minor checks failed Finished [ 47.000s] datafusion-proto-common Building datafusion-proto-models v55.1.0 (current) Built [ 25.190s] (current) Parsing datafusion-proto-models v55.1.0 (current) Parsed [ 0.132s] (current) Building datafusion-proto-models v55.1.0 (baseline) Built [ 25.089s] (baseline) Parsing datafusion-proto-models v55.1.0 (baseline) Parsed [ 0.135s] (baseline) Checking datafusion-proto-models v55.1.0 -> v55.1.0 (no change; assume patch) Checked [ 1.723s] 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 ParquetOptions.eager_pruning in /home/runner/work/datafusion/datafusion/datafusion/proto-models/src/generated/datafusion_proto_common.rs:871 field ParquetOptions.eager_pruning_file_limit_opt in /home/runner/work/datafusion/datafusion/datafusion/proto-models/src/generated/datafusion_proto_common.rs:907 field ParquetOptions.eager_pruning in /home/runner/work/datafusion/datafusion/datafusion/proto-models/src/generated/datafusion_proto_common.rs:871 field ParquetOptions.eager_pruning_file_limit_opt in /home/runner/work/datafusion/datafusion/datafusion/proto-models/src/generated/datafusion_proto_common.rs:907 Summary semver requires new major version: 1 major and 0 minor checks failed Finished [ 53.285s] datafusion-proto-models Building datafusion-sqllogictest v55.1.0 (current) Built [ 103.323s] (current) Parsing datafusion-sqllogictest v55.1.0 (current) Parsed [ 0.021s] (current) Building datafusion-sqllogictest v55.1.0 (baseline) Built [ 102.200s] (baseline) Parsing datafusion-sqllogictest v55.1.0 (baseline) Parsed [ 0.022s] (baseline) Checking datafusion-sqllogictest v55.1.0 -> v55.1.0 (no change; assume patch) Checked [ 0.086s] 223 checks: 223 pass, 31 skip Summary no semver update required Finished [ 208.321s] datafusion-sqllogictest ``` </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]
