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

[Bug]: The ParDo#validateStateApplicableForInput the catch of this method discards the original Exception message #25841

Closed
1 of 15 tasks
bigduu opened this issue Mar 14, 2023 · 0 comments
Closed
1 of 15 tasks

[Bug]: The ParDo#validateStateApplicableForInput the catch of this method discards the original Exception message #25841

bigduu opened this issue Mar 14, 2023 · 0 comments
Labels
awaiting triage bug java P3
Milestone
2.51.0 Release

Comments

@bigduu
Copy link
Contributor

bigduu commented Mar 14, 2023

What happened?

When I specified a KV Coder today, it failed because of the Deterministic check. I checked the error message and found it could have been more helpful. Later, when I came to Debug, I discovered that Java Serialization might be non-deterministic.
I don't think the original error message should be blocked
image
image

Issue Priority

Priority: 3 (minor)

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
@bigduu bigduu mentioned this issue Mar 14, 2023
Merged
3 tasks
Abacn pushed a commit that referenced this issue May 31, 2023
@bigduu
fixes:#25841 Improve-error-messages-for-the-validateStateApplicableFo…
fb1683d 
…rInput-method… (#25843)

* Improve-error-messages-for-the-validateStateApplicableForInput-method-in-ParDO-so-that-they-are-more-helpful

* Update sdks/java/core/src/main/java/org/apache/beam/sdk/transforms/ParDo.java
@bigduu bigduu closed this as completed Aug 14, 2023
@github-actions github-actions bot added this to the 2.51.0 Release milestone Aug 14, 2023
cushon pushed a commit to cushon/beam that referenced this issue May 24, 2024
@bigduu @cushon
fixes:apache#25841 Improve-error-messages-for-the-validateStateApplic…
7b19153 
…ableForInput-method… (apache#25843)

* Improve-error-messages-for-the-validateStateApplicableForInput-method-in-ParDO-so-that-they-are-more-helpful

* Update sdks/java/core/src/main/java/org/apache/beam/sdk/transforms/ParDo.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
awaiting triage bug java P3
Projects
None yet
Milestone
2.51.0 Release
Development

No branches or pull requests
1 participant
@bigduu