Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[fix](ccr) Enable feature but no sync table property #46219

Closed
wants to merge 1 commit into from

Conversation

wyxxxcat
Copy link
Contributor

@wyxxxcat wyxxxcat commented Dec 31, 2024

What problem does this PR solve?

now:

Related PR: #xxx

Problem Summary:

Release note

None

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

@hello-stephen
Copy link
Contributor

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?

@wyxxxcat
Copy link
Contributor Author

run buildall

@doris-robot
Copy link

TPC-H: Total hot run time: 32430 ms
machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
Tpch sf100 test result on commit b3444cb3b025be12a645fcd8fd83e7395ef95d7a, data reload: false

------ Round 1 ----------------------------------
q1	17570	6109	6029	6029
q2	2042	304	162	162
q3	10427	1235	744	744
q4	10214	875	438	438
q5	7529	2143	1960	1960
q6	209	181	147	147
q7	889	742	598	598
q8	9230	1329	1110	1110
q9	5357	5029	4825	4825
q10	6770	2336	1866	1866
q11	496	286	263	263
q12	345	361	216	216
q13	17776	3547	2978	2978
q14	233	233	247	233
q15	560	495	504	495
q16	633	602	603	602
q17	568	850	327	327
q18	6871	6520	6408	6408
q19	1553	967	564	564
q20	301	330	189	189
q21	2802	2106	1967	1967
q22	364	333	309	309
Total cold run time: 102739 ms
Total hot run time: 32430 ms

----- Round 2, with runtime_filter_mode=off -----
q1	6298	6175	6215	6175
q2	233	327	238	238
q3	2248	2654	2326	2326
q4	1447	1843	1415	1415
q5	4372	4757	4732	4732
q6	185	180	149	149
q7	2086	1949	1819	1819
q8	2645	2806	2690	2690
q9	7412	7255	7308	7255
q10	3082	3377	2863	2863
q11	615	512	494	494
q12	659	722	553	553
q13	3455	3786	3165	3165
q14	290	303	272	272
q15	561	523	515	515
q16	654	685	634	634
q17	1208	1722	1253	1253
q18	7749	7497	7298	7298
q19	837	1089	1082	1082
q20	1962	2086	1869	1869
q21	5660	5288	4847	4847
q22	615	586	595	586
Total cold run time: 54273 ms
Total hot run time: 52230 ms

@wyxxxcat wyxxxcat closed this Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants