apache / doris

Apache Doris is an easy-to-use, high performance and unified analytics database.
https://doris.apache.org
Apache License 2.0
12.78k stars 3.29k forks source link

[fix](cloud-mow)Fix some implicit conversion loses when complie mow code #44410

Open hust-hhb opened 3 days ago

hust-hhb commented 3 days ago

What problem does this PR solve?

Related PR: #40204

Problem Summary: There is some implicit conversion loses like uint64_t to uint32_t on mow code, this pr fix it.

Release note

None

Check List (For Author)

Check List (For Reviewer who merge this PR)

doris-robot commented 3 days ago

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

Please clearly describe your PR:

  1. What problem was fixed (it's best to include specific error reporting information). How it was fixed.
  2. Which behaviors were modified. What was the previous behavior, what is it now, why was it modified, and what possible impacts might there be.
  3. What features were added. Why was this function added?
  4. Which code was refactored and why was this part of the code refactored?
  5. Which functions were optimized and what is the difference before and after the optimization?
hust-hhb commented 3 days ago

run buildall

github-actions[bot] commented 3 days ago

clang-tidy review says "All clean, LGTM! :+1:"

github-actions[bot] commented 3 days ago

PR approved by at least one committer and no changes requested.

github-actions[bot] commented 3 days ago

PR approved by anyone and no changes requested.

github-actions[bot] commented 3 days ago

clang-tidy review says "All clean, LGTM! :+1:"

doris-robot commented 3 days ago

TeamCity be ut coverage result: Function Coverage: 38.04% (9902/26033) Line Coverage: 29.21% (82817/283514) Region Coverage: 28.34% (42541/150083) Branch Coverage: 24.90% (21564/86590) Coverage Report: http://coverage.selectdb-in.cc/coverage/61069bea820b88c05415c8c494771be4746f501a_61069bea820b88c05415c8c494771be4746f501a/report/index.html

dataroaring commented 2 days ago

run buildall

github-actions[bot] commented 2 days ago

clang-tidy review says "All clean, LGTM! :+1:"

doris-robot commented 2 days ago

TeamCity be ut coverage result: Function Coverage: 38.31% (9979/26049) Line Coverage: 29.42% (83511/283827) Region Coverage: 28.59% (42981/150360) Branch Coverage: 25.16% (21829/86746) Coverage Report: http://coverage.selectdb-in.cc/coverage/30f579ddf3df36bb6dd40fb1b8131f3b15112195_30f579ddf3df36bb6dd40fb1b8131f3b15112195/report/index.html