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

*: mini refactor #5326

Merged
merged 11 commits into from
Jul 11, 2022
Merged

Conversation

SeaRise
Copy link
Contributor

@SeaRise SeaRise commented Jul 8, 2022

What problem does this PR solve?

Issue Number: ref #4739

Problem Summary:

What is changed and how it works?

port from #5143

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

Side effects

  • Performance regression: Consumes more CPU
  • Performance regression: Consumes more Memory
  • Breaking backward compatibility

Documentation

  • Affects user behaviors
  • Contains syntax changes
  • Contains variable changes
  • Contains experimental features
  • Changes MySQL compatibility

Release note

None

@ti-chi-bot
Copy link
Member

ti-chi-bot commented Jul 8, 2022

[REVIEW NOTIFICATION]

This pull request has been approved by:

  • windtalker
  • yibin87

To complete the pull request process, please ask the reviewers in the list to review by filling /cc @reviewer in the comment.
After your PR has acquired the required number of LGTMs, you can assign this pull request to the committer in the list by filling /assign @committer in the comment to help you merge this pull request.

The full list of commands accepted by this bot can be found here.

Reviewer can indicate their review by submitting an approval review.
Reviewer can cancel approval by submitting a request changes review.

@ti-chi-bot ti-chi-bot added do-not-merge/needs-linked-issue release-note-none Denotes a PR that doesn't merit a release note. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed do-not-merge/needs-linked-issue labels Jul 8, 2022
@SeaRise SeaRise requested review from ywqzzy, gengliqi and windtalker July 8, 2022 18:13
@SeaRise
Copy link
Contributor Author

SeaRise commented Jul 8, 2022

/run-all-tests

@sre-bot
Copy link
Collaborator

sre-bot commented Jul 8, 2022

Coverage for changed files

Filename                                           Regions    Missed Regions     Cover   Functions  Missed Functions  Executed       Lines      Missed Lines     Cover    Branches   Missed Branches     Cover
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
DataStreams/TiRemoteBlockInputStream.h                 116               116     0.00%          17                17     0.00%         157               157     0.00%          58                58     0.00%
Debug/astToExecutor.cpp                                579               211    63.56%          53                 9    83.02%        1524               553    63.71%         572               247    56.82%
Flash/Coprocessor/DAGContext.cpp                       116                80    31.03%          26                13    50.00%         182               116    36.26%          74                58    21.62%
Flash/Coprocessor/DAGContext.h                          40                14    65.00%          31                12    61.29%          88                34    61.36%          12                 6    50.00%
Flash/Coprocessor/DAGExpressionAnalyzer.cpp            384               137    64.32%          51                 7    86.27%         906               307    66.11%         298               139    53.36%
Flash/Coprocessor/DAGExpressionAnalyzer.h                2                 1    50.00%           2                 1    50.00%           2                 1    50.00%           0                 0         -
Flash/Coprocessor/DAGQueryBlockInterpreter.cpp         231                70    69.70%          31                 4    87.10%         530               100    81.13%         150                50    66.67%
Flash/Coprocessor/DAGQueryBlockInterpreter.h             2                 0   100.00%           2                 0   100.00%           2                 0   100.00%           0                 0         -
Flash/Coprocessor/GenSchemaAndColumn.cpp                12                 4    66.67%           3                 1    66.67%          41                15    63.41%          12                 5    58.33%
Flash/Coprocessor/InterpreterUtils.cpp                  27                 3    88.89%           7                 0   100.00%          86                 6    93.02%          22                 5    77.27%
Flash/tests/gtest_interpreter.cpp                      166                95    42.77%           6                 0   100.00%         492                 0   100.00%          20                20     0.00%
Interpreters/Settings.h                                  1                 0   100.00%           1                 0   100.00%           1                 0   100.00%           0                 0         -
TestUtils/mockExecutor.h                                 6                 1    83.33%           6                 1    83.33%          15                 3    80.00%           0                 0         -
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
TOTAL                                                 1682               732    56.48%         236                65    72.46%        4026              1292    67.91%        1218               588    51.72%

Coverage summary

Functions  MissedFunctions  Executed  Lines   MissedLines  Cover
18464      9607             47.97%    207968  96479        53.61%

full coverage report (for internal network access only)

@ti-chi-bot ti-chi-bot added size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Jul 11, 2022
@SeaRise
Copy link
Contributor Author

SeaRise commented Jul 11, 2022

/run-all-tests

@SeaRise SeaRise changed the title *: mini refactor [WIP] *: mini refactor Jul 11, 2022
@ti-chi-bot ti-chi-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 11, 2022
f
@SeaRise SeaRise force-pushed the mini_refactor_from_planner branch from 7a16730 to 1b18ac4 Compare July 11, 2022 06:13
@SeaRise SeaRise changed the title [WIP] *: mini refactor *: mini refactor Jul 11, 2022
@ti-chi-bot ti-chi-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jul 11, 2022
@SeaRise
Copy link
Contributor Author

SeaRise commented Jul 11, 2022

/run-all-tests

Copy link
Contributor

@windtalker windtalker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ti-chi-bot ti-chi-bot added the status/LGT1 Indicates that a PR has LGTM 1. label Jul 11, 2022
@sre-bot
Copy link
Collaborator

sre-bot commented Jul 11, 2022

Coverage for changed files

Filename                                           Regions    Missed Regions     Cover   Functions  Missed Functions  Executed       Lines      Missed Lines     Cover    Branches   Missed Branches     Cover
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
DataStreams/TiRemoteBlockInputStream.h                 116               116     0.00%          17                17     0.00%         157               157     0.00%          58                58     0.00%
Debug/astToExecutor.cpp                                579               211    63.56%          53                 9    83.02%        1528               553    63.81%         572               247    56.82%
Flash/Coprocessor/DAGContext.cpp                       116                80    31.03%          26                13    50.00%         182               116    36.26%          74                58    21.62%
Flash/Coprocessor/DAGContext.h                          41                14    65.85%          32                12    62.50%          91                34    62.64%          12                 6    50.00%
Flash/Coprocessor/DAGExpressionAnalyzer.cpp            384               137    64.32%          51                 7    86.27%         906               307    66.11%         298               139    53.36%
Flash/Coprocessor/DAGExpressionAnalyzer.h                2                 1    50.00%           2                 1    50.00%           2                 1    50.00%           0                 0         -
Flash/Coprocessor/DAGQueryBlockInterpreter.cpp         255                90    64.71%          33                 5    84.85%         599               155    74.12%         162                60    62.96%
Flash/Coprocessor/DAGQueryBlockInterpreter.h             2                 0   100.00%           2                 0   100.00%           2                 0   100.00%           0                 0         -
Flash/Coprocessor/GenSchemaAndColumn.cpp                12                 4    66.67%           3                 1    66.67%          41                15    63.41%          12                 5    58.33%
Flash/Coprocessor/InterpreterUtils.cpp                  33                 3    90.91%           8                 0   100.00%         119                 6    94.96%          26                 5    80.77%
Flash/tests/gtest_interpreter.cpp                      194               114    41.24%           7                 0   100.00%         557                 0   100.00%          24                24     0.00%
Interpreters/Settings.h                                  1                 0   100.00%           1                 0   100.00%           1                 0   100.00%           0                 0         -
TestUtils/mockExecutor.h                                 6                 1    83.33%           6                 1    83.33%          15                 3    80.00%           0                 0         -
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
TOTAL                                                 1741               771    55.72%         241                66    72.61%        4200              1347    67.93%        1238               602    51.37%

Coverage summary

Functions  MissedFunctions  Executed  Lines   MissedLines  Cover
18492      9570             48.25%    208440  96354        53.77%

full coverage report (for internal network access only)

Copy link
Contributor

@yibin87 yibin87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ti-chi-bot ti-chi-bot added status/LGT2 Indicates that a PR has LGTM 2. and removed status/LGT1 Indicates that a PR has LGTM 1. labels Jul 11, 2022
@SeaRise
Copy link
Contributor Author

SeaRise commented Jul 11, 2022

/merge

@ti-chi-bot
Copy link
Member

@SeaRise: It seems you want to merge this PR, I will help you trigger all the tests:

/run-all-tests

You only need to trigger /merge once, and if the CI test fails, you just re-trigger the test that failed and the bot will merge the PR for you after the CI passes.

If you have any questions about the PR merge process, please refer to pr process.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the ti-community-infra/tichi repository.

@ti-chi-bot
Copy link
Member

This pull request has been accepted and is ready to merge.

Commit hash: 4f241cb

@ti-chi-bot ti-chi-bot added the status/can-merge Indicates a PR has been approved by a committer. label Jul 11, 2022
@SeaRise
Copy link
Contributor Author

SeaRise commented Jul 11, 2022

/rebuild

@SeaRise
Copy link
Contributor Author

SeaRise commented Jul 11, 2022

/run-all-tests

@sre-bot
Copy link
Collaborator

sre-bot commented Jul 11, 2022

Coverage for changed files

Filename                                           Regions    Missed Regions     Cover   Functions  Missed Functions  Executed       Lines      Missed Lines     Cover    Branches   Missed Branches     Cover
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
DataStreams/TiRemoteBlockInputStream.h                 116               116     0.00%          17                17     0.00%         157               157     0.00%          58                58     0.00%
Debug/astToExecutor.cpp                                579               211    63.56%          53                 9    83.02%        1528               553    63.81%         572               247    56.82%
Flash/Coprocessor/DAGContext.cpp                       116                80    31.03%          26                13    50.00%         182               116    36.26%          74                58    21.62%
Flash/Coprocessor/DAGContext.h                          41                14    65.85%          32                12    62.50%          91                34    62.64%          12                 6    50.00%
Flash/Coprocessor/DAGExpressionAnalyzer.cpp            384               137    64.32%          51                 7    86.27%         906               307    66.11%         298               139    53.36%
Flash/Coprocessor/DAGExpressionAnalyzer.h                2                 1    50.00%           2                 1    50.00%           2                 1    50.00%           0                 0         -
Flash/Coprocessor/DAGQueryBlockInterpreter.cpp         255                90    64.71%          33                 5    84.85%         599               155    74.12%         162                60    62.96%
Flash/Coprocessor/DAGQueryBlockInterpreter.h             2                 0   100.00%           2                 0   100.00%           2                 0   100.00%           0                 0         -
Flash/Coprocessor/GenSchemaAndColumn.cpp                12                 4    66.67%           3                 1    66.67%          41                15    63.41%          12                 5    58.33%
Flash/Coprocessor/InterpreterUtils.cpp                  33                 3    90.91%           8                 0   100.00%         119                 6    94.96%          26                 5    80.77%
Flash/tests/gtest_interpreter.cpp                      194               114    41.24%           7                 0   100.00%         557                 0   100.00%          24                24     0.00%
Interpreters/Settings.h                                  1                 0   100.00%           1                 0   100.00%           1                 0   100.00%           0                 0         -
TestUtils/mockExecutor.h                                 6                 1    83.33%           6                 1    83.33%          15                 3    80.00%           0                 0         -
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
TOTAL                                                 1741               771    55.72%         241                66    72.61%        4200              1347    67.93%        1238               602    51.37%

Coverage summary

Functions  MissedFunctions  Executed  Lines   MissedLines  Cover
18507      9569             48.30%    208698  96361        53.83%

full coverage report (for internal network access only)

@SeaRise
Copy link
Contributor Author

SeaRise commented Jul 11, 2022

/run-all-tests

@sre-bot
Copy link
Collaborator

sre-bot commented Jul 11, 2022

Coverage for changed files

Filename                                           Regions    Missed Regions     Cover   Functions  Missed Functions  Executed       Lines      Missed Lines     Cover    Branches   Missed Branches     Cover
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
DataStreams/TiRemoteBlockInputStream.h                 116               116     0.00%          17                17     0.00%         157               157     0.00%          58                58     0.00%
Debug/astToExecutor.cpp                                579               211    63.56%          53                 9    83.02%        1528               553    63.81%         572               247    56.82%
Flash/Coprocessor/DAGContext.cpp                       116                80    31.03%          26                13    50.00%         182               116    36.26%          74                58    21.62%
Flash/Coprocessor/DAGContext.h                          41                14    65.85%          32                12    62.50%          91                34    62.64%          12                 6    50.00%
Flash/Coprocessor/DAGExpressionAnalyzer.cpp            384               137    64.32%          51                 7    86.27%         906               307    66.11%         298               139    53.36%
Flash/Coprocessor/DAGExpressionAnalyzer.h                2                 1    50.00%           2                 1    50.00%           2                 1    50.00%           0                 0         -
Flash/Coprocessor/DAGQueryBlockInterpreter.cpp         255                90    64.71%          33                 5    84.85%         599               155    74.12%         162                60    62.96%
Flash/Coprocessor/DAGQueryBlockInterpreter.h             2                 0   100.00%           2                 0   100.00%           2                 0   100.00%           0                 0         -
Flash/Coprocessor/GenSchemaAndColumn.cpp                12                 4    66.67%           3                 1    66.67%          41                15    63.41%          12                 5    58.33%
Flash/Coprocessor/InterpreterUtils.cpp                  33                 3    90.91%           8                 0   100.00%         119                 6    94.96%          26                 5    80.77%
Flash/tests/gtest_interpreter.cpp                      194               114    41.24%           7                 0   100.00%         557                 0   100.00%          24                24     0.00%
Interpreters/Settings.h                                  1                 0   100.00%           1                 0   100.00%           1                 0   100.00%           0                 0         -
TestUtils/mockExecutor.h                                 6                 1    83.33%           6                 1    83.33%          15                 3    80.00%           0                 0         -
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
TOTAL                                                 1741               771    55.72%         241                66    72.61%        4200              1347    67.93%        1238               602    51.37%

Coverage summary

Functions  MissedFunctions  Executed  Lines   MissedLines  Cover
18507      9569             48.30%    208698  96352        53.83%

full coverage report (for internal network access only)

@ti-chi-bot ti-chi-bot merged commit 4619605 into pingcap:master Jul 11, 2022
@SeaRise SeaRise deleted the mini_refactor_from_planner branch July 11, 2022 10:09
@sre-bot
Copy link
Collaborator

sre-bot commented Jul 11, 2022

Coverage for changed files

Filename                                           Regions    Missed Regions     Cover   Functions  Missed Functions  Executed       Lines      Missed Lines     Cover    Branches   Missed Branches     Cover
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
DataStreams/TiRemoteBlockInputStream.h                 116               116     0.00%          17                17     0.00%         157               157     0.00%          58                58     0.00%
Debug/astToExecutor.cpp                                579               211    63.56%          53                 9    83.02%        1528               553    63.81%         572               247    56.82%
Flash/Coprocessor/DAGContext.cpp                       116                80    31.03%          26                13    50.00%         182               116    36.26%          74                58    21.62%
Flash/Coprocessor/DAGContext.h                          41                14    65.85%          32                12    62.50%          91                34    62.64%          12                 6    50.00%
Flash/Coprocessor/DAGExpressionAnalyzer.cpp            384               137    64.32%          51                 7    86.27%         906               307    66.11%         298               139    53.36%
Flash/Coprocessor/DAGExpressionAnalyzer.h                2                 1    50.00%           2                 1    50.00%           2                 1    50.00%           0                 0         -
Flash/Coprocessor/DAGQueryBlockInterpreter.cpp         255                90    64.71%          33                 5    84.85%         599               155    74.12%         162                60    62.96%
Flash/Coprocessor/DAGQueryBlockInterpreter.h             2                 0   100.00%           2                 0   100.00%           2                 0   100.00%           0                 0         -
Flash/Coprocessor/GenSchemaAndColumn.cpp                12                 4    66.67%           3                 1    66.67%          41                15    63.41%          12                 5    58.33%
Flash/Coprocessor/InterpreterUtils.cpp                  33                 3    90.91%           8                 0   100.00%         119                 6    94.96%          26                 5    80.77%
Flash/tests/gtest_interpreter.cpp                      194               114    41.24%           7                 0   100.00%         557                 0   100.00%          24                24     0.00%
Interpreters/Settings.h                                  1                 0   100.00%           1                 0   100.00%           1                 0   100.00%           0                 0         -
TestUtils/mockExecutor.h                                 6                 1    83.33%           6                 1    83.33%          15                 3    80.00%           0                 0         -
--------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
TOTAL                                                 1741               771    55.72%         241                66    72.61%        4200              1347    67.93%        1238               602    51.37%

Coverage summary

Functions  MissedFunctions  Executed  Lines   MissedLines  Cover
18507      9569             48.30%    208698  96290        53.86%

full coverage report (for internal network access only)

Lloyd-Pottiger pushed a commit to Lloyd-Pottiger/tiflash that referenced this pull request Jul 12, 2022
…s in README (pingcap#5182)

close pingcap#5172, ref pingcap#5178

Enhancement: add a integrated test on DDL module (pingcap#5130)

ref pingcap#5129

Revert "Revise default background threads size" (pingcap#5176)

close pingcap#5177

chore: remove extra dyn cast (pingcap#5186)

close pingcap#5185

Add MPPReceiverSet, which includes ExchangeReceiver and CoprocessorReader (pingcap#5175)

ref pingcap#5095

DDL: Use Column Name Instead of Offset to Find the common handle cluster index (pingcap#5166)

close pingcap#5154

Add random failpoint in critical paths (pingcap#4876)

close pingcap#4807

Segment test framework (pingcap#5150)

close pingcap#5151

optimize ps v3 restore (pingcap#5163)

ref pingcap#4914

Fix build failed (pingcap#5196)

close pingcap#5195

feat: delta tree dispatching (pingcap#5199)

close pingcap#5200

feat: introduce specialized API to write fixed length data rapidly (pingcap#5181)

close pingcap#5183

Add gtest for Limit, TopN, Projection (pingcap#5187) (pingcap#5188)

close pingcap#5187

add `MPPTask::handleError()` (pingcap#5202)

ref pingcap#5095

Check result of starting grpc server (pingcap#5257)

close pingcap#5255

feat: add optimized routines for aarch64 (pingcap#5231)

close pingcap#5240

fix: aarch64-quick-fix (pingcap#5259)

close pingcap#5260

Update client-c to support ipv6 (pingcap#5270)

close pingcap#5247

upgrade prometheus-cpp to v1.0.1 (pingcap#5279)

ref pingcap#2103, close pingcap#5278

Fix README type error (pingcap#5273)

ref pingcap#5178

fix(cmake): make sure libc++ is utilized by tiflash-proxy (pingcap#5281)

close pingcap#5282

fix the wrong order of execution summary for list based executors (pingcap#5242)

close pingcap#5241

Schema: allow loading empty schema diff when the version grows up. (pingcap#5245)

close pingcap#5244

Optimize apply speed under heavy write pressure (pingcap#4883)

ref pingcap#4728

update proxy to raftstore-proxy-6.2 (pingcap#5287)

ref pingcap#4982

Flush segment cache when doing the compaction (pingcap#5284)

close pingcap#5179

metrics: Fix incorrect metrics for delta_merge tasks (pingcap#5061)

close pingcap#5055

dep: upgrade jemalloc (pingcap#5197)

close pingcap#5258

*: TiFlash pagectl/dttool use only-decryption mode (pingcap#5271)

close pingcap#5122

suppresion false positive report from tsan (pingcap#5303)

close pingcap#5088

Refine test framework code and tests (pingcap#5261)

close pingcap#5262

feat: add logical cpu cores and memory into grafana (pingcap#5124)

close pingcap#3821

Implement TimeToSec function push down (pingcap#5235)

close pingcap#5116

feat: implement shiftRight function push down (pingcap#5156)

close pingcap#5100

schema : make update to partition tables when 'set tiflash replica' (pingcap#5267)

close pingcap#5266

Replace initializer_list with vector for planner test framework (pingcap#5307)

close pingcap#5295

KVStore: decouple flush region and CompactLog with a new FFI fn_try_flush_data (pingcap#5283)

ref pingcap#5170

refine error message in mpptask (pingcap#5304)

ref pingcap#5095

Implement ReverseUTF8/Reverse function push down (pingcap#5233)

close pingcap#5111

Optimize comparision for collation `UTF8_BIN` and `UTF8MB4_BIN` (pingcap#5299)

ref pingcap#5294

feat : support set tiflash mode ddl action (pingcap#5256)

ref pingcap#5252

Add non-blocking functions for MPMCQueue (pingcap#5311)

close pingcap#5310

add random segment test for CI weekly (pingcap#5300)

close pingcap#5301

*: tidy FunctionString.cpp (pingcap#5312)

close pingcap#5313

ci: fix check-license github action (pingcap#5318)

close pingcap#5317

update proxy to raftstore-proxy-6.2 (pingcap#5316)

ref pingcap#4982

Change one `additional_input_at_end` to many streams in `ParallelInputsProcessor`  (pingcap#5274)

close pingcap#4856, close pingcap#5263

support fine grained shuffle for window function (pingcap#5048)

close pingcap#5142

feat: pushdown get_format into TiFlash (pingcap#5269)

close pingcap#5115

fix: format throw data truncated error (pingcap#5272)

close pingcap#4891

Print content of columns for gtest (pingcap#5243)

close pingcap#5203

*: also enable O3 for aarch64 (pingcap#5338)

close pingcap#5342

Add debug image build target for CentOS7 (pingcap#5344)

close pingcap#5343

*: mini refactor (pingcap#5326)

close pingcap#4739

Refactor initialize of background pool (pingcap#5190)

close pingcap#5189

delete copy/move ctor of MPMCQueue explicitly (pingcap#5328)

close pingcap#5329

Introduce proxy_server and new-mock-engine-store (pingcap#5319)

ref pingcap#5170

fix: incorrect uptime in grafana panel

Signed-off-by: Lloyd-Pottiger <[email protected]>
Lloyd-Pottiger pushed a commit to Lloyd-Pottiger/tiflash that referenced this pull request Jul 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note-none Denotes a PR that doesn't merit a release note. size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. status/can-merge Indicates a PR has been approved by a committer. status/LGT2 Indicates that a PR has LGTM 2.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants