doris icon indicating copy to clipboard operation
doris copied to clipboard

[Improvement](function) optimize trivial function deserialize_and_merge_vec

Open BiteTheDDDDt opened this issue 1 month ago • 10 comments

What problem does this PR solve?

SELECT
  max(csales) tpcds_cmax
FROM
  (
    SELECT
      c_customer_sk,
      sum((ss_quantity * ss_sales_price)) csales
    FROM
      store_sales,
      customer,
      date_dim
    WHERE
      (ss_customer_sk = c_customer_sk)
      AND (ss_sold_date_sk = d_date_sk)
      AND (
        d_year IN (
          2000,
          (2000 + 1),
          (2000 + 2),
          (2000 + 3)
        )
      )
    GROUP BY
      c_customer_sk
  ) x;
QQ_1765293461333 QQ_1765456112915

Check List (For Author)

  • Test

    • [ ] Regression test
    • [ ] Unit Test
    • [ ] Manual test (add detailed scripts or steps below)
    • [ ] No need to test or manual test. Explain why:
      • [ ] This is a refactor/code format and no logic has been changed.
      • [ ] Previous test can cover this change.
      • [ ] No code files have been changed.
      • [ ] Other reason
  • Behavior changed:

    • [ ] No.
    • [ ] Yes.
  • Does this need documentation?

    • [ ] No.
    • [ ] Yes.

Check List (For Reviewer who merge this PR)

  • [ ] Confirm the release note
  • [ ] Confirm test cases
  • [ ] Confirm document
  • [ ] Add branch pick label

BiteTheDDDDt avatar Dec 09 '25 15:12 BiteTheDDDDt

run buildall

BiteTheDDDDt avatar Dec 09 '25 15:12 BiteTheDDDDt

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?

hello-stephen avatar Dec 09 '25 15:12 hello-stephen

BE UT Coverage Report

Increment line coverage 36.36% (32/88) :tada:

Increment coverage report Complete coverage report

Category Coverage
Function Coverage 53.36% (18716/35072)
Line Coverage 39.07% (173049/442959)
Region Coverage 33.76% (134301/397766)
Branch Coverage 34.66% (57691/166444)

hello-stephen avatar Dec 09 '25 18:12 hello-stephen

BE Regression && UT Coverage Report

Increment line coverage 62.50% (55/88) :tada:

Increment coverage report Complete coverage report

Category Coverage
Function Coverage 72.24% (24831/34374)
Line Coverage 58.96% (260905/442481)
Region Coverage 53.98% (217263/402489)
Branch Coverage 55.47% (92805/167319)

hello-stephen avatar Dec 09 '25 21:12 hello-stephen

run buildall

BiteTheDDDDt avatar Dec 10 '25 03:12 BiteTheDDDDt

BE UT Coverage Report

Increment line coverage 36.36% (32/88) :tada:

Increment coverage report Complete coverage report

Category Coverage
Function Coverage 53.36% (18716/35072)
Line Coverage 39.06% (173037/442959)
Region Coverage 33.74% (134215/397766)
Branch Coverage 34.65% (57680/166444)

hello-stephen avatar Dec 10 '25 04:12 hello-stephen

BE Regression && UT Coverage Report

Increment line coverage 62.50% (55/88) :tada:

Increment coverage report Complete coverage report

Category Coverage
Function Coverage 72.33% (24863/34374)
Line Coverage 59.07% (261382/442483)
Region Coverage 53.91% (216974/402489)
Branch Coverage 55.54% (92931/167319)

hello-stephen avatar Dec 10 '25 07:12 hello-stephen

run buildall

BiteTheDDDDt avatar Dec 10 '25 10:12 BiteTheDDDDt

run buildall

BiteTheDDDDt avatar Dec 11 '25 03:12 BiteTheDDDDt

run buildall

BiteTheDDDDt avatar Dec 11 '25 12:12 BiteTheDDDDt

run buildall

BiteTheDDDDt avatar Dec 12 '25 03:12 BiteTheDDDDt

run buildall

BiteTheDDDDt avatar Dec 12 '25 12:12 BiteTheDDDDt

run buildall

BiteTheDDDDt avatar Dec 12 '25 16:12 BiteTheDDDDt

run buildall

BiteTheDDDDt avatar Dec 13 '25 05:12 BiteTheDDDDt