Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Failing Test]: Java IO Precommit failing due to KafkaIOTranslation being out of sync #30941

Closed
2 of 16 tasks
ahmedabu98 opened this issue Apr 12, 2024 · 2 comments
Closed
2 of 16 tasks

[Failing Test]: Java IO Precommit failing due to KafkaIOTranslation being out of sync #30941

ahmedabu98 opened this issue Apr 12, 2024 · 2 comments
Labels
bug failing test io java P1 permared tests
Milestone
2.57.0 Release

Comments

@ahmedabu98
Copy link
Contributor

What happened?

Error:

java.lang.AssertionError: Method getConsumerPollingTimeout will not be tracked when upgrading the 'KafkaIO.Read' transform. Please update 'KafkaIOTranslation.KafkaIOReadWithMetadataTranslator' to track the new method and update this test.
	at org.junit.Assert.fail(Assert.java:89)
	at org.junit.Assert.assertTrue(Assert.java:42)
	at org.apache.beam.sdk.io.kafka.upgrade.KafkaIOTranslationTest.testReadTransformRowIncludesAllFields(KafkaIOTranslationTest.java:170)

A new builder method was added for KafkaIO in #30877, which now needs to be added in KafkaIOTranslation.

Issue Failure

Failure: Test is continually failing

Issue Priority

Priority: 2 (backlog / disabled test but we think the product is healthy)

Issue Components

  • Component: Python SDK
  • Component: Java SDK
  • Component: Go SDK
  • Component: Typescript SDK
  • Component: IO connector
  • Component: Beam YAML
  • Component: Beam examples
  • Component: Beam playground
  • Component: Beam katas
  • Component: Website
  • Component: Spark Runner
  • Component: Flink Runner
  • Component: Samza Runner
  • Component: Twister2 Runner
  • Component: Hazelcast Jet Runner
  • Component: Google Cloud Dataflow Runner
@ahmedabu98 ahmedabu98 mentioned this issue Apr 12, 2024
Closed
@JayajP JayajP mentioned this issue Apr 12, 2024
Merged
3 tasks
@Abacn
Copy link
Contributor

Abacn commented Apr 15, 2024

This affects PreCommit so bump to P1

@Abacn Abacn mentioned this issue Apr 15, 2024
Merged
xianhualiu added a commit to xianhualiu/beam that referenced this issue Apr 16, 2024
@xianhualiu
[apache#30941]fix upgrade test deu to missed config ConsumerPollingTi…
3d2154f 
…meout in KafkaIOTranslation
@xianhualiu xianhualiu mentioned this issue Apr 16, 2024
Merged
@Abacn Abacn mentioned this issue Apr 16, 2024
Closed
3 tasks
xianhualiu added a commit to xianhualiu/beam that referenced this issue Apr 16, 2024
@xianhualiu
[apache#30941]fix upgrade test due to missed config ConsumerPollingTi…
7907c78 
…meout in KafkaIOTranslation
xianhualiu added a commit to xianhualiu/beam that referenced this issue Apr 16, 2024
@xianhualiu
[apache#30941]fix upgrade test due to missed config ConsumerPollingTi…
95e46c0 
…meout in KafkaIOTranslation
xianhualiu added a commit to xianhualiu/beam that referenced this issue Apr 16, 2024
@xianhualiu
[apache#30941]fix upgrade test due to missed config ConsumerPollingTi…
d2dfdc3 
…meout in KafkaIOTranslation
xianhualiu added a commit to xianhualiu/beam that referenced this issue Apr 16, 2024
@xianhualiu
[apache#30941]fix upgrade test due to missed config ConsumerPollingTi…
adbd83e 
…meout in KafkaIOTranslation
damccorm pushed a commit that referenced this issue Apr 17, 2024
@xianhualiu
[#30941]fix upgrade test due to missed config ConsumerPollingTimeout (#…
8092932 
…30998)

* [#30941]fix upgrade test deu to  missed config ConsumerPollingTimeout in  KafkaIOTranslation

* [#30941]fix upgrade test due to  missed config ConsumerPollingTimeout in KafkaIOTranslation

* [#30941]fix upgrade test due to  missed config ConsumerPollingTimeout in KafkaIOTranslation

* [#30941]fix upgrade test due to  missed config ConsumerPollingTimeout in KafkaIOTranslation

* [#30941]fix upgrade test due to  missed config ConsumerPollingTimeout in KafkaIOTranslation

* fixed upgrade test and changed consumer timeout to long

* fixed spotless issues

* fixed test
@chamikaramj
Copy link
Contributor

Fixed by #30877.

@github-actions github-actions bot added this to the 2.57.0 Release milestone Apr 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug failing test io java P1 permared tests
Projects
None yet
Milestone
2.57.0 Release
Development

Successfully merging a pull request may close this issue.

Revert #30877 and #30915 to fix Java IOs PreCommit - Kafka upgrade test Abacn/beam
3 participants
@Abacn @chamikaramj @ahmedabu98