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.

Sign up for GitHub

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

Validator consolidation #467

Merged
merged 2 commits into from
Jan 9, 2024

Remove unused class

6179889
Select commit
Loading
Failed to load commit list.
Merged

Validator consolidation #467

Remove unused class
6179889
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jan 9, 2024 in 1s

85.41% of diff hit (target 52.45%)

View this Pull Request on Codecov

85.41% of diff hit (target 52.45%)

Annotations

Check warning on line 143 in sqrl-planner/sqrl-common/src/main/java/com/datasqrl/calcite/QueryPlanner.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

sqrl-planner/sqrl-common/src/main/java/com/datasqrl/calcite/QueryPlanner.java#L143 

Added line #L143 was not covered by tests

Check warning on line 145 in sqrl-planner/sqrl-common/src/main/java/com/datasqrl/calcite/QueryPlanner.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

sqrl-planner/sqrl-common/src/main/java/com/datasqrl/calcite/QueryPlanner.java#L145 

Added line #L145 was not covered by tests

Check warning on line 117 in sqrl-planner/sqrl-common/src/main/java/com/datasqrl/plan/validate/ScriptPlanner.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

sqrl-planner/sqrl-common/src/main/java/com/datasqrl/plan/validate/ScriptPlanner.java#L117 

Added line #L117 was not covered by tests

Check warning on line 458 in sqrl-planner/sqrl-common/src/main/java/com/datasqrl/plan/validate/ScriptPlanner.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

sqrl-planner/sqrl-common/src/main/java/com/datasqrl/plan/validate/ScriptPlanner.java#L457-L458 

Added lines #L457 - L458 were not covered by tests

Check warning on line 460 in sqrl-planner/sqrl-common/src/main/java/com/datasqrl/plan/validate/ScriptPlanner.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

sqrl-planner/sqrl-common/src/main/java/com/datasqrl/plan/validate/ScriptPlanner.java#L460 

Added line #L460 was not covered by tests

Check warning on line 839 in sqrl-planner/sqrl-common/src/main/java/com/datasqrl/plan/validate/ScriptPlanner.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

sqrl-planner/sqrl-common/src/main/java/com/datasqrl/plan/validate/ScriptPlanner.java#L839 

Added line #L839 was not covered by tests

Check warning on line 892 in sqrl-planner/sqrl-common/src/main/java/com/datasqrl/plan/validate/ScriptPlanner.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

sqrl-planner/sqrl-common/src/main/java/com/datasqrl/plan/validate/ScriptPlanner.java#L892 

Added line #L892 was not covered by tests

Check warning on line 921 in sqrl-planner/sqrl-common/src/main/java/com/datasqrl/plan/validate/ScriptPlanner.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

sqrl-planner/sqrl-common/src/main/java/com/datasqrl/plan/validate/ScriptPlanner.java#L921 

Added line #L921 was not covered by tests

Check warning on line 45 in sqrl-planner/sqrl-planner-local/src/main/java/com/datasqrl/plan/ScriptPlanner.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

sqrl-planner/sqrl-planner-local/src/main/java/com/datasqrl/plan/ScriptPlanner.java#L42-L45 

Added lines #L42 - L45 were not covered by tests
View more details on Codecov