Skip to content

Add metas to PartiQL Commons #2663

Add metas to PartiQL Commons

Add metas to PartiQL Commons #2663

Triggered via push August 16, 2024 21:01
Status Success
Total duration 6m 23s
Artifacts

coverage.yml

on: push
Matrix: Build and Test
Fit to window
Zoom out
Zoom in

Annotations

41 warnings
doc list item missing indentation: partiql-conformance-tests/src/bin/generate_comparison_report.rs#L23
warning: doc list item missing indentation --> partiql-conformance-tests/src/bin/generate_comparison_report.rs:23:5 | 23 | /// regressed behavior) | ^ | = help: if this is supposed to be its own paragraph, add a blank line = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_lazy_continuation = note: `#[warn(clippy::doc_lazy_continuation)]` on by default help: indent this line | 23 | /// regressed behavior) | ++
unexpected `cfg` condition value: `serde`: partiql-eval/src/eval/mod.rs#L149
warning: unexpected `cfg` condition value: `serde` --> partiql-eval/src/eval/mod.rs:149:12 | 149 | #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))] | ^^^^^^^^^^^^^^^^^ | = note: expected values for `feature` are: `default` = help: consider adding `serde` as a feature in `Cargo.toml` = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> for more information about checking conditional configuration = note: `#[warn(unexpected_cfgs)]` on by default
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L164
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:164:13 | 163 | pub struct EvaluationFailAssertion { | ----------------------- field in this struct 164 | pub result: String, | ^^^^^^ | = note: `EvaluationFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L157
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:157:13 | 156 | pub struct EvaluationSuccessAssertion { | -------------------------- field in this struct 157 | pub result: String, | ^^^^^^ | = note: `EvaluationSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L152
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:152:13 | 151 | pub struct StaticAnalysisFailAssertion { | --------------------------- field in this struct 152 | pub result: String, | ^^^^^^ | = note: `StaticAnalysisFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L147
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:147:13 | 146 | pub struct SyntaxFailAssertion { | ------------------- field in this struct 147 | pub result: String, | ^^^^^^ | = note: `SyntaxFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L142
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:142:13 | 141 | pub struct SyntaxSuccessAssertion { | ---------------------- field in this struct 142 | pub result: String, | ^^^^^^ | = note: `SyntaxSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L75
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:75:28 | 75 | StaticAnalysisFail(StaticAnalysisFailAssertion), | ------------------ ^^^^^^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 75 | StaticAnalysisFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L74
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:74:20 | 74 | SyntaxFail(SyntaxFailAssertion), | ---------- ^^^^^^^^^^^^^^^^^^^ | | | field in this variant | help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 74 | SyntaxFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L73
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:73:23 | 73 | SyntaxSuccess(SyntaxSuccessAssertion), | ------------- ^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `#[warn(dead_code)]` on by default help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 73 | SyntaxSuccess(()), | ~~
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L164
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:164:13 | 163 | pub struct EvaluationFailAssertion { | ----------------------- field in this struct 164 | pub result: String, | ^^^^^^ | = note: `EvaluationFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L157
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:157:13 | 156 | pub struct EvaluationSuccessAssertion { | -------------------------- field in this struct 157 | pub result: String, | ^^^^^^ | = note: `EvaluationSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L152
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:152:13 | 151 | pub struct StaticAnalysisFailAssertion { | --------------------------- field in this struct 152 | pub result: String, | ^^^^^^ | = note: `StaticAnalysisFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L147
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:147:13 | 146 | pub struct SyntaxFailAssertion { | ------------------- field in this struct 147 | pub result: String, | ^^^^^^ | = note: `SyntaxFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L142
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:142:13 | 141 | pub struct SyntaxSuccessAssertion { | ---------------------- field in this struct 142 | pub result: String, | ^^^^^^ | = note: `SyntaxSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L75
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:75:28 | 75 | StaticAnalysisFail(StaticAnalysisFailAssertion), | ------------------ ^^^^^^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 75 | StaticAnalysisFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L74
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:74:20 | 74 | SyntaxFail(SyntaxFailAssertion), | ---------- ^^^^^^^^^^^^^^^^^^^ | | | field in this variant | help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 74 | SyntaxFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L73
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:73:23 | 73 | SyntaxSuccess(SyntaxSuccessAssertion), | ------------- ^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `#[warn(dead_code)]` on by default help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 73 | SyntaxSuccess(()), | ~~
variants `Float32` and `Decimal` are never constructed: partiql-common/src/metadata.rs#L201
warning: variants `Float32` and `Decimal` are never constructed --> partiql-common/src/metadata.rs:201:5 | 195 | pub enum PartiqlMetaValue<T> | ---------------- variants in this enum ... 201 | Float32(f32), | ^^^^^^^ 202 | Float64(f64), 203 | Decimal(Decimal), | ^^^^^^^ | = note: `PartiqlMetaValue` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis
multiple associated items are never used: partiql-common/src/metadata.rs#L21
warning: multiple associated items are never used --> partiql-common/src/metadata.rs:21:12 | 17 | / impl<T> PartiqlMetadata<T> 18 | | where 19 | | T: Eq + Clone + Hash + Borrow<str>, | |_______________________________________- associated items in this implementation 20 | { 21 | pub fn new() -> Self { | ^^^ ... 27 | pub fn insert(&mut self, key: T, value: PartiqlMetaValue<T>) { | ^^^^^^ ... 31 | pub fn get(&self, key: &T) -> Option<&PartiqlMetaValue<T>> { | ^^^ ... 35 | pub fn get_mut(&mut self, key: &T) -> Option<&mut PartiqlMetaValue<T>> { | ^^^^^^^ ... 39 | pub fn contains_key(&self, key: &T) -> bool { | ^^^^^^^^^^^^ ... 43 | pub fn keys(&self) -> impl Iterator<Item = &T> { | ^^^^ ... 47 | pub fn values(&self) -> impl Iterator<Item = &PartiqlMetaValue<T>> { | ^^^^^^ ... 51 | pub fn values_mut(&mut self) -> impl Iterator<Item = &mut PartiqlMetaValue<T>> { | ^^^^^^^^^^ ... 55 | pub fn entry(&mut self, key: T) -> Entry<'_, T, PartiqlMetaValue<T>> { | ^^^^^ ... 59 | pub fn clear(&mut self) { | ^^^^^ ... 63 | pub fn remove(&mut self, key: &T) -> Option<PartiqlMetaValue<T>> { | ^^^^^^ ... 67 | pub fn len(&self) -> usize { | ^^^ ... 75 | pub fn iter_mut(&mut self) -> impl Iterator<Item = (&T, &mut PartiqlMetaValue<T>)> { | ^^^^^^^^ ... 79 | pub fn vec_value(&self, key: &str) -> Option<Vec<PartiqlMetaValue<T>>> { | ^^^^^^^^^ ... 88 | pub fn bool_value(&self, key: &str) -> Option<bool> { | ^^^^^^^^^^ ... 97 | pub fn f32_value(&self, key: &str) -> Option<f32> { | ^^^^^^^^^ ... 106 | pub fn f64_value(&self, key: &str) -> Option<f64> { | ^^^^^^^^^ ... 115 | pub fn decimal_value(&self, key: &str) -> Option<Decimal> { | ^^^^^^^^^^^^^ ... 124 | pub fn i32_value(&self, key: &str) -> Option<i32> { | ^^^^^^^^^ ... 133 | pub fn i64_value(&self, key: &str) -> Option<i64> { | ^^^^^^^^^ ... 142 | pub fn map_value(&self, key: &str) -> Option<PartiqlMetadata<T>> { | ^^^^^^^^^ ... 151 | pub fn string_value(&self, key: &str) -> Option<String> { | ^^^^^^^^^^^^ | = note: `#[warn(dead_code)]` on by default
doc list item missing indentation: partiql-conformance-tests/src/bin/generate_comparison_report.rs#L23
warning: doc list item missing indentation --> partiql-conformance-tests/src/bin/generate_comparison_report.rs:23:5 | 23 | /// regressed behavior) | ^ | = help: if this is supposed to be its own paragraph, add a blank line = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_lazy_continuation = note: `#[warn(clippy::doc_lazy_continuation)]` on by default help: indent this line | 23 | /// regressed behavior) | ++
unexpected `cfg` condition value: `serde`: partiql-eval/src/eval/mod.rs#L149
warning: unexpected `cfg` condition value: `serde` --> partiql-eval/src/eval/mod.rs:149:12 | 149 | #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))] | ^^^^^^^^^^^^^^^^^ | = note: expected values for `feature` are: `default` = help: consider adding `serde` as a feature in `Cargo.toml` = note: see <https://doc.rust-lang.org/nightly/rustc/check-cfg/cargo-specifics.html> for more information about checking conditional configuration = note: `#[warn(unexpected_cfgs)]` on by default
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L164
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:164:13 | 163 | pub struct EvaluationFailAssertion { | ----------------------- field in this struct 164 | pub result: String, | ^^^^^^ | = note: `EvaluationFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L157
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:157:13 | 156 | pub struct EvaluationSuccessAssertion { | -------------------------- field in this struct 157 | pub result: String, | ^^^^^^ | = note: `EvaluationSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L152
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:152:13 | 151 | pub struct StaticAnalysisFailAssertion { | --------------------------- field in this struct 152 | pub result: String, | ^^^^^^ | = note: `StaticAnalysisFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L147
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:147:13 | 146 | pub struct SyntaxFailAssertion { | ------------------- field in this struct 147 | pub result: String, | ^^^^^^ | = note: `SyntaxFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L142
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:142:13 | 141 | pub struct SyntaxSuccessAssertion { | ---------------------- field in this struct 142 | pub result: String, | ^^^^^^ | = note: `SyntaxSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L75
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:75:28 | 75 | StaticAnalysisFail(StaticAnalysisFailAssertion), | ------------------ ^^^^^^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 75 | StaticAnalysisFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L74
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:74:20 | 74 | SyntaxFail(SyntaxFailAssertion), | ---------- ^^^^^^^^^^^^^^^^^^^ | | | field in this variant | help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 74 | SyntaxFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L73
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:73:23 | 73 | SyntaxSuccess(SyntaxSuccessAssertion), | ------------- ^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `#[warn(dead_code)]` on by default help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 73 | SyntaxSuccess(()), | ~~
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L164
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:164:13 | 163 | pub struct EvaluationFailAssertion { | ----------------------- field in this struct 164 | pub result: String, | ^^^^^^ | = note: `EvaluationFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L157
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:157:13 | 156 | pub struct EvaluationSuccessAssertion { | -------------------------- field in this struct 157 | pub result: String, | ^^^^^^ | = note: `EvaluationSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L152
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:152:13 | 151 | pub struct StaticAnalysisFailAssertion { | --------------------------- field in this struct 152 | pub result: String, | ^^^^^^ | = note: `StaticAnalysisFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L147
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:147:13 | 146 | pub struct SyntaxFailAssertion { | ------------------- field in this struct 147 | pub result: String, | ^^^^^^ | = note: `SyntaxFailAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `result` is never read: partiql-conformance-test-generator/src/schema.rs#L142
warning: field `result` is never read --> partiql-conformance-test-generator/src/schema.rs:142:13 | 141 | pub struct SyntaxSuccessAssertion { | ---------------------- field in this struct 142 | pub result: String, | ^^^^^^ | = note: `SyntaxSuccessAssertion` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L75
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:75:28 | 75 | StaticAnalysisFail(StaticAnalysisFailAssertion), | ------------------ ^^^^^^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 75 | StaticAnalysisFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L74
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:74:20 | 74 | SyntaxFail(SyntaxFailAssertion), | ---------- ^^^^^^^^^^^^^^^^^^^ | | | field in this variant | help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 74 | SyntaxFail(()), | ~~
field `0` is never read: partiql-conformance-test-generator/src/schema.rs#L73
warning: field `0` is never read --> partiql-conformance-test-generator/src/schema.rs:73:23 | 73 | SyntaxSuccess(SyntaxSuccessAssertion), | ------------- ^^^^^^^^^^^^^^^^^^^^^^ | | | field in this variant | = note: `#[warn(dead_code)]` on by default help: consider changing the field to be of unit type to suppress this warning while preserving the field numbering, or remove the field | 73 | SyntaxSuccess(()), | ~~
variants `Float32` and `Decimal` are never constructed: partiql-common/src/metadata.rs#L201
warning: variants `Float32` and `Decimal` are never constructed --> partiql-common/src/metadata.rs:201:5 | 195 | pub enum PartiqlMetaValue<T> | ---------------- variants in this enum ... 201 | Float32(f32), | ^^^^^^^ 202 | Float64(f64), 203 | Decimal(Decimal), | ^^^^^^^ | = note: `PartiqlMetaValue` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis
multiple associated items are never used: partiql-common/src/metadata.rs#L21
warning: multiple associated items are never used --> partiql-common/src/metadata.rs:21:12 | 17 | / impl<T> PartiqlMetadata<T> 18 | | where 19 | | T: Eq + Clone + Hash + Borrow<str>, | |_______________________________________- associated items in this implementation 20 | { 21 | pub fn new() -> Self { | ^^^ ... 27 | pub fn insert(&mut self, key: T, value: PartiqlMetaValue<T>) { | ^^^^^^ ... 31 | pub fn get(&self, key: &T) -> Option<&PartiqlMetaValue<T>> { | ^^^ ... 35 | pub fn get_mut(&mut self, key: &T) -> Option<&mut PartiqlMetaValue<T>> { | ^^^^^^^ ... 39 | pub fn contains_key(&self, key: &T) -> bool { | ^^^^^^^^^^^^ ... 43 | pub fn keys(&self) -> impl Iterator<Item = &T> { | ^^^^ ... 47 | pub fn values(&self) -> impl Iterator<Item = &PartiqlMetaValue<T>> { | ^^^^^^ ... 51 | pub fn values_mut(&mut self) -> impl Iterator<Item = &mut PartiqlMetaValue<T>> { | ^^^^^^^^^^ ... 55 | pub fn entry(&mut self, key: T) -> Entry<'_, T, PartiqlMetaValue<T>> { | ^^^^^ ... 59 | pub fn clear(&mut self) { | ^^^^^ ... 63 | pub fn remove(&mut self, key: &T) -> Option<PartiqlMetaValue<T>> { | ^^^^^^ ... 67 | pub fn len(&self) -> usize { | ^^^ ... 75 | pub fn iter_mut(&mut self) -> impl Iterator<Item = (&T, &mut PartiqlMetaValue<T>)> { | ^^^^^^^^ ... 79 | pub fn vec_value(&self, key: &str) -> Option<Vec<PartiqlMetaValue<T>>> { | ^^^^^^^^^ ... 88 | pub fn bool_value(&self, key: &str) -> Option<bool> { | ^^^^^^^^^^ ... 97 | pub fn f32_value(&self, key: &str) -> Option<f32> { | ^^^^^^^^^ ... 106 | pub fn f64_value(&self, key: &str) -> Option<f64> { | ^^^^^^^^^ ... 115 | pub fn decimal_value(&self, key: &str) -> Option<Decimal> { | ^^^^^^^^^^^^^ ... 124 | pub fn i32_value(&self, key: &str) -> Option<i32> { | ^^^^^^^^^ ... 133 | pub fn i64_value(&self, key: &str) -> Option<i64> { | ^^^^^^^^^ ... 142 | pub fn map_value(&self, key: &str) -> Option<PartiqlMetadata<T>> { | ^^^^^^^^^ ... 151 | pub fn string_value(&self, key: &str) -> Option<String> { | ^^^^^^^^^^^^ | = note: `#[warn(dead_code)]` on by default
Build and Test (ubuntu-20.04)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, codecov/codecov-action@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/