eldenmoon opened a new pull request, #64270:
URL: https://github.com/apache/doris/pull/64270
### What problem does this PR solve?
Issue Number: None
Related PR: None
Problem Summary: ColumnVariant::insert_from rebuilt non-root-only rows
through Field and try_insert, which re-inferred variant paths and could change
sparse/dense layout while sorted run merge copied rows. Subcolumn range copy
also dropped pending default suffix rows. Together these could surface type
mismatches for mixed-shape variant rows under sparse subcolumn limits.
### Release note
None
### Check List (For Author)
- Test:
- Regression test
- Red on upstream/master: `./run-regression-test.sh --run -d
variant_p0 -s test_variant_sparse_sort_insert_from -c
'jdbc:mysql://127.0.0.1:22030/?useLocalSessionState=true&allowLoadLocalInfile=true&zeroDateTimeBehavior=round'
-ha '127.0.0.1:21030' -parallel 1 -suiteParallel 1` failed with `Field::get
type mismatch: requested STRING, actual BIGINT`
- Green after fix: `./run-regression-test.sh --run -d variant_p0 -s
test_variant_sparse_sort_insert_from -c
'jdbc:mysql://127.0.0.1:22030/?useLocalSessionState=true&allowLoadLocalInfile=true&zeroDateTimeBehavior=round'
-ha '127.0.0.1:21030' -parallel 1 -suiteParallel 1`
- Unit Test
- Red on upstream/master: `./run-be-ut.sh --run
--filter='ColumnVariantTest.insert_range_from_materializes_pending_default_suffix:ColumnVariantTest.insert_from_preserves_sparse_limit_for_object_rows'`
failed both added tests
- Green after fix: `./run-be-ut.sh --run
--filter='ColumnVariantTest.insert_range_from_materializes_pending_default_suffix:ColumnVariantTest.insert_from_preserves_sparse_limit_for_object_rows'`
- Behavior changed: No
- Does this need documentation: No
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
