Skip to content

feat: expose virtual-forward flag in event CTEs#177

Open
o-az wants to merge 5 commits intomainfrom
o-az/expose-virtual-forward-in-ctes
Open

feat: expose virtual-forward flag in event CTEs#177
o-az wants to merge 5 commits intomainfrom
o-az/expose-virtual-forward-in-ctes

Conversation

@o-az
Copy link
Copy Markdown
Member

@o-az o-az commented Apr 27, 2026

Summary

  • Expose logs.is_virtual_forward on event CTEs for PostgreSQL and ClickHouse
  • Allow raw predicate pushdown for is_virtual_forward filters
  • Keep /status fast/reliable by removing synchronous block gap detection from the HTTP request path
  • Update query skill docs, changelog entries, tests, and CTE snapshots

Why

TIP-1022 virtual-address deposits emit both sender -> virtual and virtual -> master Transfer logs. Event CTE consumers need is_virtual_forward to avoid surfacing or counting the forwarding hop as a normal transfer.

Production also showed /status returning DB errors because the endpoint ran an expensive gap-detection window scan over blocks on every request. Exact gap discovery now stays in the background gap-fill path instead of the API status path.

Test

  • cargo test query::parser::tests
  • cargo check
  • cargo test --test status_test --no-run

Note: cargo emits the existing unused synced_num warning in src/cli/status.rs.

@o-az o-az requested a review from kamsz April 28, 2026 00:05
@o-az o-az force-pushed the o-az/expose-virtual-forward-in-ctes branch from 17cb681 to 7dae8aa Compare May 5, 2026 06:25
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