Skip to content

Conversation

@tomstepp
Copy link
Contributor

Move Kafka read with redistribute override to Kafka IO package.


See the Contributor Guide for more tips on how to make review process smoother.

To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md

GitHub Actions Tests Status (on master branch)

Build python source distribution and wheels
Python tests
Java tests
Go tests

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

@tomstepp tomstepp marked this pull request as ready for review November 24, 2025 17:11
@tomstepp
Copy link
Contributor Author

R: @Abacn

@tomstepp
Copy link
Contributor Author

R: @scwhittle

@github-actions
Copy link
Contributor

Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control. If you'd like to restart, comment assign set of reviewers

1 similar comment
@github-actions
Copy link
Contributor

Stopping reviewer notifications for this pull request: review requested by someone other than the bot, ceding control. If you'd like to restart, comment assign set of reviewers

@Abacn
Copy link
Contributor

Abacn commented Nov 24, 2025

testOverrideAppliedWhenRedistributeEnabled (org.apache.beam.sdk.io.kafka.KafkaReadWithRedistributeOverrideTest) failed

sdks/java/io/kafka/build/test-results/test/TEST-org.apache.beam.sdk.io.kafka.KafkaReadWithRedistributeOverrideTest.xml [took 0s]

org.apache.beam.sdk.testing.TestPipeline$PipelineRunMissingException: The pipeline has not been run.
	at app//org.apache.beam.sdk.testing.TestPipeline$PipelineAbandonedNodeEnforcement.verifyPipelineExecution(TestPipeline.java:215)

Need to set p.enableAbandonedNodeEnforcement(false)

@tomstepp tomstepp requested a review from Abacn November 24, 2025 19:28
@Abacn
Copy link
Contributor

Abacn commented Nov 24, 2025

1 out of 8 runs failed: testUnreachableKafkaBrokers (org.apache.beam.sdk.io.kafka.KafkaIOTest)

not related to this change. Merging for now

@Abacn Abacn merged commit 6a82448 into apache:master Nov 24, 2025
21 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants