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

Avoid rewriting unsharded queries and split semantic analysis in two #15217

Merged
merged 12 commits into from
Feb 14, 2024

add test that shows that recursive CTEs are not a problem

f44bfce
Select commit
Loading
Failed to load commit list.
Merged

Avoid rewriting unsharded queries and split semantic analysis in two #15217

add test that shows that recursive CTEs are not a problem
f44bfce
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Feb 14, 2024 in 0s

96.68% of diff hit (target 67.43%)

View this Pull Request on Codecov

96.68% of diff hit (target 67.43%)

Annotations

Check warning on line 109 in go/vt/vtgate/planbuilder/operators/hash_join.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/planbuilder/operators/hash_join.go#L109 

Added line #L109 was not covered by tests

Check warning on line 342 in go/vt/vtgate/semantics/analyzer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/semantics/analyzer.go#L342 

Added line #L342 was not covered by tests

Check warning on line 377 in go/vt/vtgate/semantics/analyzer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/semantics/analyzer.go#L376-L377 

Added lines #L376 - L377 were not covered by tests

Check warning on line 309 in go/vt/vtgate/semantics/semantic_state.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/semantics/semantic_state.go#L309 

Added line #L309 was not covered by tests
View more details on Codecov