Skip to content

test: cover RDF bloom remote dynamic filter e2e - #4

Draft
discord9 wants to merge 2 commits into
omos/rdf-bloom-dn-exactfrom
omos/rdf-bloom-e2e
Draft

test: cover RDF bloom remote dynamic filter e2e#4
discord9 wants to merge 2 commits into
omos/rdf-bloom-dn-exactfrom
omos/rdf-bloom-e2e

Conversation

@discord9

@discord9 discord9 commented Jun 24, 2026

Copy link
Copy Markdown
Owner

I hereby agree to the terms of the GreptimeDB CLA.

Refer to a related PR or issue link (optional)

Fork-only stack PR. Stacked on PR3: #3

Earlier stack:

Depends on:

Part of original umbrella draft PR: GreptimeTeam#8342

What's changed and what's your intention?

This is PR 4 of the RDF Bloom stack. It adds integration coverage and implementation notes:

  • adds an E2E test that builds sparse probe data and 302 build keys
  • verifies SeqScan dyn_filters do not contain bloom_probe
  • verifies non-pushdown Bloom FilterExec reduces rows versus its child scan
  • records concise Simplified Chinese gotchas in impl-note.md

Validation run locally:

  • cargo fmt --all -- --check
  • git diff --check
  • CARGO_PROFILE_DEV_DEBUG=0 CARGO_PROFILE_TEST_DEBUG=0 cargo check -p tests-integration
  • CARGO_PROFILE_DEV_DEBUG=0 CARGO_PROFILE_TEST_DEBUG=0 cargo test -p tests-integration test_remote_dyn_filter_bloom_typed_payload_e2e --lib -- --nocapture

PR Checklist

Please convert it to a draft if some of the following conditions are not met.

  • I have written the necessary rustdoc comments.
  • I have added the necessary unit tests and integration tests.
  • This PR requires documentation updates.
  • API changes are backward compatible.
  • Schema or data changes are backward compatible.

@discord9
discord9 force-pushed the omos/rdf-bloom-e2e branch from 56bfdcb to ce04137 Compare June 25, 2026 04:06
@discord9
discord9 force-pushed the omos/rdf-bloom-dn-exact branch from 9d0d955 to 42e0ba0 Compare June 25, 2026 04:06
@discord9
discord9 force-pushed the omos/rdf-bloom-e2e branch 2 times, most recently from 2023719 to 4d26863 Compare June 25, 2026 07:38
@discord9
discord9 force-pushed the omos/rdf-bloom-dn-exact branch from 16b6af5 to 0e90855 Compare June 25, 2026 09:16
@discord9
discord9 force-pushed the omos/rdf-bloom-e2e branch 2 times, most recently from 1305af3 to cdaef0e Compare June 25, 2026 10:24
@discord9
discord9 force-pushed the omos/rdf-bloom-dn-exact branch from 0e90855 to 05f9888 Compare June 25, 2026 10:24
@discord9
discord9 force-pushed the omos/rdf-bloom-e2e branch from cdaef0e to 97a0ead Compare June 25, 2026 12:59
@discord9
discord9 force-pushed the omos/rdf-bloom-dn-exact branch from 05f9888 to 71a935f Compare June 25, 2026 12:59
@discord9
discord9 force-pushed the omos/rdf-bloom-e2e branch from 97a0ead to f21f263 Compare June 30, 2026 04:01
@discord9
discord9 force-pushed the omos/rdf-bloom-dn-exact branch from 71a935f to 797e90e Compare June 30, 2026 04:01
discord9 added 2 commits July 6, 2026 16:46
Signed-off-by: discord9 <discord9@163.com>
Signed-off-by: discord9 <discord9@163.com>
@discord9
discord9 force-pushed the omos/rdf-bloom-dn-exact branch from 797e90e to 8a1110d Compare July 6, 2026 09:46
@discord9
discord9 force-pushed the omos/rdf-bloom-e2e branch from f21f263 to 37ff154 Compare July 6, 2026 09:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant