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.

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

Array filtering support (Part #3): Postgres relational filter refactoring #189

Merged
merged 28 commits into from
Jan 16, 2024

Addressed review comments

19cbb1a
Select commit
Loading
Failed to load commit list.
Merged

Array filtering support (Part #3): Postgres relational filter refactoring #189

Addressed review comments
19cbb1a
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jan 16, 2024 in 1s

82.75% of diff hit (target 80.48%)

View this Pull Request on Codecov

82.75% of diff hit (target 80.48%)

Annotations

Check warning on line 36 in document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresContainsRelationalFilterParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresContainsRelationalFilterParser.java#L36

Added line #L36 was not covered by tests

Check warning on line 41 in document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresContainsRelationalFilterParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresContainsRelationalFilterParser.java#L40-L41

Added lines #L40 - L41 were not covered by tests

Check warning on line 48 in document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresContainsRelationalFilterParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresContainsRelationalFilterParser.java#L48

Added line #L48 was not covered by tests

Check warning on line 53 in document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresContainsRelationalFilterParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresContainsRelationalFilterParser.java#L52-L53

Added lines #L52 - L53 were not covered by tests

Check warning on line 14 in document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresExistsRelationalFilterParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresExistsRelationalFilterParser.java#L14

Added line #L14 was not covered by tests

Check warning on line 11 in document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresLikeRelationalFilterParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresLikeRelationalFilterParser.java#L10-L11

Added lines #L10 - L11 were not covered by tests

Check warning on line 16 in document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresLikeRelationalFilterParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresLikeRelationalFilterParser.java#L15-L16

Added lines #L15 - L16 were not covered by tests

Check warning on line 19 in document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresLikeRelationalFilterParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresLikeRelationalFilterParser.java#L19

Added line #L19 was not covered by tests

Check warning on line 13 in document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresNotExistsRelationalFilterParser.java

See this annotation in the file changed.

@codecov codecov / codecov/patch

document-store/src/main/java/org/hypertrace/core/documentstore/postgres/query/v1/parser/filter/PostgresNotExistsRelationalFilterParser.java#L13

Added line #L13 was not covered by tests