Skip to content

[To dev/1.3] Improve state type validation in CombineRequest deserialization. (#17449)#17502

Merged
jt2594838 merged 2 commits intoapache:dev/1.3from
luoluoyuyu:cp-17449
Apr 17, 2026
Merged

[To dev/1.3] Improve state type validation in CombineRequest deserialization. (#17449)#17502
jt2594838 merged 2 commits intoapache:dev/1.3from
luoluoyuyu:cp-17449

Conversation

@luoluoyuyu
Copy link
Copy Markdown
Member

Description


This PR has:

  • been self-reviewed.
    • concurrent read
    • concurrent write
    • concurrent read and write
  • added documentation for new or modified features or behaviors.
  • added Javadocs for most classes and all non-trivial methods.
  • added or updated version, license, or notice information
  • added comments explaining the "why" and the intent of the code wherever would not be obvious
    for an unfamiliar reader.
  • added unit tests or modified existing tests to cover new code paths, ensuring the threshold
    for code coverage.
  • added integration tests.
  • been tested in a test IoTDB cluster.

Key changed/added classes (or packages if there are too many classes) in this PR

…che#17449)

* Improve state type validation in CombineRequest deserialization.

Tighten request deserialization by validating state type before instantiation to reduce unexpected type usage risk, and add targeted tests for accepted and rejected state class names.

Made-with: Cursor

* spotless

(cherry picked from commit c1d16a4)
@jt2594838 jt2594838 merged commit 8bf1640 into apache:dev/1.3 Apr 17, 2026
16 of 17 checks passed
@luoluoyuyu luoluoyuyu deleted the cp-17449 branch April 21, 2026 01:50
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