Skip to content

test: select the active FileSource actor in dispatcher checks#3002

Merged
He-Pin merged 1 commit into
apache:mainfrom
He-Pin:fix/file-source-dispatcher-actor-selection
Jun 19, 2026
Merged

test: select the active FileSource actor in dispatcher checks#3002
He-Pin merged 1 commit into
apache:mainfrom
He-Pin:fix/file-source-dispatcher-actor-selection

Conversation

@He-Pin

@He-Pin He-Pin commented May 28, 2026

Copy link
Copy Markdown
Member

Motivation

FileSourceSpec dispatcher checks can select a stale fileSource child left briefly by a previous materialization, which makes the dispatcher override assertion flaky.

Modification

Capture existing fileSource children before starting each dispatcher-check stream, wait for exactly the newly materialized child, and watch it terminate after cancellation.

Result

The checks assert the dispatcher of the stream started by the current test instead of a leftover child from an earlier materialization.

Tests

  • JDK 17 Scala 3.3.6: stream-tests / Test / testOnly org.apache.pekko.stream.io.FileSourceSpec
  • scalafmt --mode diff-ref=origin/main --quiet
  • scalafmt --list --mode diff-ref=origin/main
  • git diff --check

References

Refs #2994

Motivation:
FileSourceSpec dispatcher checks can select a stale fileSource child left briefly by a previous materialization, which makes the dispatcher override assertion flaky.

Modification:
Capture existing fileSource children before starting each dispatcher-check stream, wait for exactly the newly materialized child, and watch it terminate after cancellation.

Result:
The checks assert the dispatcher of the stream started by the current test instead of a leftover child from an earlier materialization.

Tests:
- JDK 17 Scala 3.3.6: stream-tests / Test / testOnly org.apache.pekko.stream.io.FileSourceSpec
- scalafmt --mode diff-ref=origin/main --quiet
- scalafmt --list --mode diff-ref=origin/main
- git diff --check

References:
Refs apache#2994

@pjfanning pjfanning left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@He-Pin He-Pin merged commit 048c5f4 into apache:main Jun 19, 2026
9 checks passed
@He-Pin He-Pin deleted the fix/file-source-dispatcher-actor-selection branch June 19, 2026 22:44
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.

2 participants