starrocks icon indicating copy to clipboard operation
starrocks copied to clipboard

[BugFix] fix json length check

Open murphyatwork opened this issue 2 weeks ago โ€ข 6 comments

Why I'm doing:

What I'm doing:

Minimal length is 1024, which should be allowed.

Fixes #https://github.com/StarRocks/StarRocksTest/issues/10673

What type of PR is this:

  • [x] BugFix
  • [ ] Feature
  • [ ] Enhancement
  • [ ] Refactor
  • [ ] UT
  • [ ] Doc
  • [ ] Tool

Does this PR entail a change in behavior?

  • [ ] Yes, this PR will result in a change in behavior.
  • [x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

  • [ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
  • [ ] Parameter changes: default values, similar parameters but with different default values
  • [ ] Policy changes: use new policy to replace old one, functionality automatically enabled
  • [ ] Feature removed
  • [ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

  • [ ] I have added test cases for my bug fix or my new feature
  • [ ] This pr needs user documentation (for new or modified features or behaviors)
    • [ ] I have added documentation for my new feature or new function
  • [ ] This is a backport pr

Bugfix cherry-pick branch check:

  • [x] I have checked the version labels which the pr will be auto-backported to the target branch
    • [x] 4.0
    • [x] 3.5
    • [ ] 3.4
    • [ ] 3.3

[!NOTE] Permits schema change from JSON to CHAR/VARCHAR when target length equals the JSON minimum (e.g., 1024).

  • Catalog
    • In Column.checkSchemaChangeAllowed, relax JSON length check: change <= to < so converting JSON to CHAR/VARCHAR is allowed when target strLen equals PrimitiveType.JSON.getTypeSize().

Written by Cursor Bugbot for commit b81f3cd8a17459f13180465dde20dd4f2ea6c380. This will update automatically on new commits. Configure here.

murphyatwork avatar Dec 11 '25 07:12 murphyatwork

๐Ÿงช CI Insights

Here's what we observed from your CI run for b81f3cd8.

๐ŸŸข All jobs passed!

But CI Insights is watching ๐Ÿ‘€

mergify[bot] avatar Dec 11 '25 07:12 mergify[bot]

@cursor review

alvin-celerdata avatar Dec 11 '25 15:12 alvin-celerdata

[Java-Extensions Incremental Coverage Report]

:white_check_mark: pass : 0 / 0 (0%)

github-actions[bot] avatar Dec 12 '25 05:12 github-actions[bot]

[BE Incremental Coverage Report]

:white_check_mark: pass : 0 / 0 (0%)

github-actions[bot] avatar Dec 12 '25 05:12 github-actions[bot]

[FE Incremental Coverage Report]

:white_check_mark: pass : 1 / 1 (100.00%)

file detail

path covered_line new_line coverage not_covered_line_detail
:large_blue_circle: com/starrocks/catalog/Column.java 1 1 100.00% []

github-actions[bot] avatar Dec 12 '25 05:12 github-actions[bot]

@Mergifyio backport branch-3.5

github-actions[bot] avatar Dec 15 '25 02:12 github-actions[bot]

@Mergifyio backport branch-4.0

github-actions[bot] avatar Dec 15 '25 02:12 github-actions[bot]

backport branch-3.5

โœ… Backports have been created

mergify[bot] avatar Dec 15 '25 02:12 mergify[bot]

backport branch-4.0

โœ… Backports have been created

mergify[bot] avatar Dec 15 '25 02:12 mergify[bot]