Skip to content

[pick](Serde-2.1) fix variant serde may lost num_rows when subcolumns empty - #41438

Merged
eldenmoon merged 1 commit into
apache:branch-2.1from
eldenmoon:branch-2.1
Sep 29, 2024
Merged

[pick](Serde-2.1) fix variant serde may lost num_rows when subcolumns empty#41438
eldenmoon merged 1 commit into
apache:branch-2.1from
eldenmoon:branch-2.1

Conversation

@eldenmoon

@eldenmooneldenmoon commented Sep 28, 2024

Copy link
Copy Markdown
Member

serialization object with empty subcolumns may lost num_rows, so need to record num_rows and set back num_rows in serdes

backport #38413

… empty (apache#38413)
serialization object with empty subcolumns may lost num_rows, so need
to record num_rows and set back num_rows in serdes
backport apache#38413
@doris-robot

Copy link
Copy Markdown

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@eldenmoon

Copy link
Copy Markdown
MemberAuthor

run buildall

@github-actions

Copy link
Copy Markdown
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@doris-robot

Copy link
Copy Markdown

TeamCity be ut coverage result:
Function Coverage: 36.12% (9350/25887)
Line Coverage: 27.64% (76788/277853)
Region Coverage: 26.42% (39428/149210)
Branch Coverage: 23.20% (20064/86470)
Coverage Report: http://coverage.selectdb-in.cc/coverage/dc3b728f2d5e9718e312a38403ef01a6af8fc3ed_dc3b728f2d5e9718e312a38403ef01a6af8fc3ed/report/index.html

@eldenmoon
eldenmoon merged commit d659750 into apache:branch-2.1Sep 29, 2024
@eldenmoon
eldenmoon deleted the branch-2.1 branch September 29, 2024 01:45
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@eldenmoon@doris-robot@wm1581066