Skip to content

MOD-10533: Sync RLookups - tail and upstreams #6612

Merged
Itzikvaknin merged 11 commits intofeature-RRFfrom
ivaknin-sync_rlookups
Aug 11, 2025
Merged

MOD-10533: Sync RLookups - tail and upstreams #6612
Itzikvaknin merged 11 commits intofeature-RRFfrom
ivaknin-sync_rlookups

Conversation

@Itzikvaknin
Copy link
Copy Markdown
Collaborator

@Itzikvaknin Itzikvaknin commented Aug 7, 2025

Key Changes:

RLookup Synchronization:

  • Use new RLookup_CloneInto() to copy upstream lookup into tail pipeline
  • Aligns field indices (dstidx, svidx) between search subquery and tail pipeline

Pipeline Refactoring:

  • Moved Pipeline_BuildAggregationPart case PLN_T_LOAD logic into a separate function
  • Reused this new function in both Pipeline_BuildAggregationPart and HybridRequest_BuildPipeline
  • In HybridRequest_BuildPipeline, used a cloned load step for this logic

Test Changes:

  • Refactor AddLoadStepToPlan to create unprocessed LOAD steps (with ArgsCursor instead of pre-populated keys)
  • Tests now verify that BuildPipeline correctly processes LOAD steps from unprocessed to processed state

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants