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.

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

[Failing Test]: Exclude MultimapState tests from Dataflow VR suites #24750

Closed
1 of 15 tasks
tvalentyn opened this issue Dec 21, 2022 · 4 comments
Closed
1 of 15 tasks

[Failing Test]: Exclude MultimapState tests from Dataflow VR suites #24750

tvalentyn opened this issue Dec 21, 2022 · 4 comments
Labels
awaiting triage bug done & done Issue has been reviewed after it was closed for verification, followups, etc. failing test java P2 permared tests

Comments

@tvalentyn
Copy link
Contributor

tvalentyn commented Dec 21, 2022

What happened?

Validates runner and VR Java11/Java17/Streaming test suites are failing after #23491 was merged, example:

https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow_Java11/4492/

org.apache.beam.sdk.transforms.ParDoTest$StateTests.testMultimapStateClear

Failing for the past 1 build (Since #4492 )
Took 5.2 sec.
Error Message
java.lang.UnsupportedOperationException: DataflowRunner does not currently support MultimapState
Stacktrace
java.lang.UnsupportedOperationException: DataflowRunner does not currently support MultimapState
	at org.apache.beam.runners.dataflow.DataflowRunner.verifyDoFnSupported(DataflowRunner.java:2427)
	at org.apache.beam.runners.dataflow.BatchStatefulParDoOverrides$StatefulMultiOutputParDo.expand(BatchStatefulParDoOverrides.java:192)
	at org.apache.beam.runners.dataflow.BatchStatefulParDoOverrides$StatefulMultiOutputParDo.expand(BatchStatefulParDoOverrides.java:177)
	at org.apache.beam.sdk.Pipeline.applyReplacement(Pipeline.java:576)

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 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
@tvalentyn
Copy link
Contributor Author

@zhengbuqian would you mind excluding relevant tests on Dataflow for now? Thanks!
cc: @lukecwik

@lukecwik
Copy link
Member

This should be as simple as excluding the test category

@lukecwik
Copy link
Member

lukecwik commented Dec 22, 2022

Is this now fixed?

Note that you can add fixes #YYY to the body of a PR and it will auto-close the issue when the PR is merged in the future.

@zhengbuqian
Copy link
Contributor

Yes this has been fixed, the newest runs of the failed test are passing https://ci-beam.apache.org/job/beam_PostCommit_Java_ValidatesRunner_Dataflow_Java11/

@github-actions github-actions bot added this to the 2.45.0 Release milestone Dec 27, 2022
@kennknowles kennknowles added the done & done Issue has been reviewed after it was closed for verification, followups, etc. label Jan 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting triage bug done & done Issue has been reviewed after it was closed for verification, followups, etc. failing test java P2 permared tests
Projects
None yet
Development

No branches or pull requests

4 participants