Skip to content

Pipe: fix potential ClassCastException when casting LocalDate[] in PipeTreeModelTsFileBuilderV2#15375

Merged
SteveYurongSu merged 1 commit intoapache:masterfrom
VGalaxies:fix-tree-tsfile-builder-v2
Apr 21, 2025
Merged

Pipe: fix potential ClassCastException when casting LocalDate[] in PipeTreeModelTsFileBuilderV2#15375
SteveYurongSu merged 1 commit intoapache:masterfrom
VGalaxies:fix-tree-tsfile-builder-v2

Conversation

@VGalaxies
Copy link
Contributor

fixup #15276

@VGalaxies VGalaxies changed the title Pipe: fix potential ClassCastException when casting LocalDate[] in PipeTsFileBuilder Pipe: fix potential ClassCastException when casting LocalDate[] in PipeTreeModelTsFileBuilderV2 Apr 20, 2025
@SteveYurongSu SteveYurongSu self-assigned this Apr 21, 2025
@SteveYurongSu SteveYurongSu requested a review from Copilot April 21, 2025 02:15
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes a potential ClassCastException when casting to LocalDate[] in the PipeTreeModelTsFileBuilderV2 file.

  • Added an additional type check (instanceof LocalDate[]) in the conditional to ensure safe casting.
  • Adjusted the conditional logic to prevent unintended ClassCastExceptions.

@SteveYurongSu SteveYurongSu merged commit 8fff9f2 into apache:master Apr 21, 2025
50 checks passed
VGalaxies added a commit to VGalaxies/iotdb that referenced this pull request Apr 21, 2025
SteveYurongSu pushed a commit that referenced this pull request Apr 22, 2025
@VGalaxies VGalaxies deleted the fix-tree-tsfile-builder-v2 branch April 23, 2025 10:25
JackieTien97 pushed a commit that referenced this pull request May 14, 2025
…peTreeModelTsFileBuilderV2 (#15375)

(cherry picked from commit 8fff9f2)
Caideyipi pushed a commit to Caideyipi/iotdb that referenced this pull request Mar 25, 2026
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.

4 participants