Skip to content

feat(pipeline-void): add timeout and perClass toggle to VoID stage options - #236

Merged
ddeboer merged 1 commit into
mainfrom
feat/void-stage-timeout-perclass
Mar 13, 2026
Merged

feat(pipeline-void): add timeout and perClass toggle to VoID stage options#236
ddeboer merged 1 commit into
mainfrom
feat/void-stage-timeout-perclass

Conversation

@ddeboer

@ddeboer ddeboer commented Mar 13, 2026

Copy link
Copy Markdown
Member

Summary

  • Add timeout option (default 60 000 ms) to all VoID stages, forwarded to SparqlConstructExecutor — down from the pipeline default of 300 s, which is too generous for the many small queries per-class stages issue.
  • Add perClass boolean to per-class stage factories (default true), allowing callers to disable class-by-class iteration and run monolithic queries instead.
  • Split VoidStageOptions into VoidStageOptions (timeout) and PerClassVoidStageOptions (timeout, batchSize, maxConcurrency, perClass), since batchSize/maxConcurrency only affect per-class stages that use an item selector.
  • Forward timeout to domain-specific executor wrappers (UriSpaceExecutor, VocabularyExecutor).
  • Stop passing unused batchSize/maxConcurrency to the Stage constructor for global (non-per-class) stages.
  • Update README with new options and type documentation.

…tions

- Add timeout option (default 60s), passed to SparqlConstructExecutor
- Add perClass boolean to toggle class-by-class iteration
- Split into VoidStageOptions and PerClassVoidStageOptions
- Forward timeout to UriSpaceExecutor and VocabularyExecutor
- Stop passing unused batchSize/maxConcurrency to global stages
- Update README with new options and type documentation
@ddeboer
ddeboer enabled auto-merge (squash) March 13, 2026 17:57
@ddeboer
ddeboer merged commit 7efe290 into main Mar 13, 2026
2 checks passed
@ddeboer
ddeboer deleted the feat/void-stage-timeout-perclass branch March 13, 2026 17:58
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