Skip to content
This repository has been archived by the owner on Sep 28, 2021. It is now read-only.

initial expression tree validation logic #600

Closed
wants to merge 3 commits into from

Conversation

alex-dukhno
Copy link
Owner

linked with #598

@alex-dukhno alex-dukhno force-pushed the initial-expression-tree-validation branch from f542abf to 5ced652 Compare June 4, 2021 18:17
@coveralls
Copy link

coveralls commented Jun 4, 2021

Pull Request Test Coverage Report for Build 912154682

  • 374 of 637 (58.71%) changed or added relevant lines in 26 files are covered.
  • 5 unchanged lines in 3 files lost coverage.
  • Overall coverage decreased (-1.5%) to 66.451%

Changes Missing Coverage Covered Lines Changed/Added Lines %
sql_engine/catalog/src/lib.rs 4 5 80.0%
sql_engine/data_definition/execution_plan/src/lib.rs 0 1 0.0%
data_repr/scalar/src/lib.rs 4 8 50.0%
expression_validator/src/lib.rs 41 45 91.11%
node_engine/src/transaction_manager.rs 2 6 33.33%
sql_engine/query_processing/type_inference/src/lib.rs 15 24 62.5%
inter_repr/typed_tree/src/lib.rs 14 26 53.85%
sql_engine/data_manipulation/untyped_tree_old/src/lib.rs 0 14 0.0%
inter_repr/untyped_tree/src/lib.rs 7 22 31.82%
sql_engine/data_manipulation/typed_tree_old/src/lib.rs 29 48 60.42%
Files with Coverage Reduction New Missed Lines %
data_repr/scalar/src/lib.rs 1 51.72%
sql_engine/data_definition/execution_plan/src/lib.rs 1 64.71%
types/src/lib.rs 3 65.12%
Totals Coverage Status
Change from base Build 877821299: -1.5%
Covered Lines: 2460
Relevant Lines: 3702

💛 - Coveralls

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants