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

Improve sharded query routing for tuple list #14892

Merged
merged 6 commits into from
Jan 15, 2024
+727 −48

fix unit test expectation

825ce76
Select commit
Loading
Failed to load commit list.
Merged

Improve sharded query routing for tuple list #14892

fix unit test expectation
825ce76
Select commit
Loading
Failed to load commit list.
This check has been archived and is scheduled for deletion. Learn more about checks retention
Codecov / codecov/patch failed Jan 15, 2024 in 0s

0.00% of diff hit (target 47.29%)

View this Pull Request on Codecov

0.00% of diff hit (target 47.29%)

Annotations

Check warning on line 373 in go/vt/vtgate/evalengine/cached_size.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/cached_size.go#L366-L373 

Added lines #L366 - L373 were not covered by tests

Check warning on line 376 in go/vt/vtgate/evalengine/cached_size.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/cached_size.go#L375-L376 

Added lines #L375 - L376 were not covered by tests

Check warning on line 68 in go/vt/vtgate/evalengine/expr_bvar.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/expr_bvar.go#L68 

Added line #L68 was not covered by tests

Check warning on line 83 in go/vt/vtgate/evalengine/expr_bvar.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/expr_bvar.go#L83 

Added line #L83 was not covered by tests

Check warning on line 46 in go/vt/vtgate/evalengine/expr_tuple_bvar.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/expr_tuple_bvar.go#L45-L46 

Added lines #L45 - L46 were not covered by tests

Check warning on line 49 in go/vt/vtgate/evalengine/expr_tuple_bvar.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/expr_tuple_bvar.go#L49 

Added line #L49 was not covered by tests

Check warning on line 56 in go/vt/vtgate/evalengine/expr_tuple_bvar.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/expr_tuple_bvar.go#L52-L56 

Added lines #L52 - L56 were not covered by tests

Check warning on line 60 in go/vt/vtgate/evalengine/expr_tuple_bvar.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/expr_tuple_bvar.go#L58-L60 

Added lines #L58 - L60 were not covered by tests

Check warning on line 82 in go/vt/vtgate/evalengine/expr_tuple_bvar.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/expr_tuple_bvar.go#L62-L82 

Added lines #L62 - L82 were not covered by tests

Check warning on line 93 in go/vt/vtgate/evalengine/expr_tuple_bvar.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/expr_tuple_bvar.go#L85-L93 

Added lines #L85 - L93 were not covered by tests

Check warning on line 95 in go/vt/vtgate/evalengine/expr_tuple_bvar.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/expr_tuple_bvar.go#L95 

Added line #L95 was not covered by tests

Check warning on line 103 in go/vt/vtgate/evalengine/expr_tuple_bvar.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/expr_tuple_bvar.go#L99-L103 

Added lines #L99 - L103 were not covered by tests

Check warning on line 105 in go/vt/vtgate/evalengine/expr_tuple_bvar.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/expr_tuple_bvar.go#L105 

Added line #L105 was not covered by tests

Check warning on line 109 in go/vt/vtgate/evalengine/expr_tuple_bvar.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/expr_tuple_bvar.go#L108-L109 

Added lines #L108 - L109 were not covered by tests

Check warning on line 156 in go/vt/vtgate/evalengine/format.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/format.go#L155-L156 

Added lines #L155 - L156 were not covered by tests

Check warning on line 160 in go/vt/vtgate/evalengine/format.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/format.go#L159-L160 

Added lines #L159 - L160 were not covered by tests

Check warning on line 164 in go/vt/vtgate/evalengine/format.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/format.go#L163-L164 

Added lines #L163 - L164 were not covered by tests

Check warning on line 30 in go/vt/vtgate/evalengine/translate_simplify.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/translate_simplify.go#L29-L30 

Added lines #L29 - L30 were not covered by tests

Check warning on line 63 in go/vt/vtgate/evalengine/translate_simplify.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/evalengine/translate_simplify.go#L62-L63 

Added lines #L62 - L63 were not covered by tests

Check warning on line 266 in go/vt/vtgate/planbuilder/operators/route.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/planbuilder/operators/route.go#L264-L266 

Added lines #L264 - L266 were not covered by tests

Check warning on line 266 in go/vt/vtgate/planbuilder/operators/sharded_routing.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/planbuilder/operators/sharded_routing.go#L262-L266 

Added lines #L262 - L266 were not covered by tests

Check warning on line 380 in go/vt/vtgate/planbuilder/operators/sharded_routing.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/planbuilder/operators/sharded_routing.go#L380 

Added line #L380 was not covered by tests

Check warning on line 392 in go/vt/vtgate/planbuilder/operators/sharded_routing.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/planbuilder/operators/sharded_routing.go#L387-L392 

Added lines #L387 - L392 were not covered by tests

Check warning on line 531 in go/vt/vtgate/planbuilder/operators/sharded_routing.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/planbuilder/operators/sharded_routing.go#L526-L531 

Added lines #L526 - L531 were not covered by tests

Check warning on line 536 in go/vt/vtgate/planbuilder/operators/sharded_routing.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

go/vt/vtgate/planbuilder/operators/sharded_routing.go#L535-L536 

Added lines #L535 - L536 were not covered by tests
View more details on Codecov