Skip to content

[FLINK-40597][core] Fix input type validation for Types.UUID - #29133

Open
dylanhz wants to merge 1 commit into
apache:masterfrom
dylanhz:codex/fix-uuid-type-validation
Open

[FLINK-40597][core] Fix input type validation for Types.UUID#29133
dylanhz wants to merge 1 commit into
apache:masterfrom
dylanhz:codex/fix-uuid-type-validation

Conversation

@dylanhz

@dylanhz dylanhz commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

What is the purpose of the change

Fix FLINK-40597: a DataStream created with explicit Types.UUID can fail input type validation for a non-lambda MapFunction<UUID, String> with Input mismatch: Basic type expected.

Brief change log

  • Recognize java.util.UUID when validating explicitly specified basic type information.
  • Leave UUID extraction without explicit type information unchanged (GenericTypeInfo<UUID> with Kryo by default). Automatic extraction as Types.UUID is tracked separately in FLINK-40546.

Verifying this change

  • Added a fromData(Types.UUID, ...).map(...) regression test and a check that fromData(...) still infers GenericTypeInfo<UUID>.
  • Confirmed the regression test fails without the fix and passes with it.
  • DataStreamTest and TypeExtractorTest: 102 tests passed after rebasing onto master.
  • Full ./mvnw clean verify was attempted. After resolving a local Python executable issue and resuming, verification stopped at flink-s3-fs-base: SeaweedFsTestContainerTest requires Docker, which is not installed locally. Subsequent modules have not been verified.

Does this pull request potentially affect one of the following parts:

  • Dependencies: no
  • Public API: yes, fixes validation in the existing @Public TypeExtractor; no API signatures change
  • Serializers: no
  • Runtime per-record code paths: no
  • Deployment or recovery: no
  • S3 file system connector: no

Documentation

  • Does this pull request introduce a new feature? no
  • If yes, how is the feature documented? not applicable

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

Generated-by: Codex 0.147.0 (model: gpt-6)

@dylanhz
dylanhz marked this pull request as ready for review September 8, 2026 16:05
@flinkbot

flinkbot commented Sep 8, 2026

Copy link
Copy Markdown
Collaborator

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

@dylanhz
dylanhz force-pushed the codex/fix-uuid-type-validation branch 2 times, most recently from 3936022 to e0c0526 Compare September 9, 2026 09:03
Generated-by: Codex 0.147.0 (model: gpt-6)
@dylanhz
dylanhz force-pushed the codex/fix-uuid-type-validation branch from e0c0526 to ccfe431 Compare September 9, 2026 11:38
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