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.

Sign up for GitHub

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

Jump to bottom

Improve the code of the TiFlash computing layer #5900

Closed
SeaRise opened this issue Sep 15, 2022 · 0 comments · Fixed by #7156
Closed

Improve the code of the TiFlash computing layer #5900

SeaRise opened this issue Sep 15, 2022 · 0 comments · Fixed by #7156
Labels
type/enhancement The issue or PR belongs to an enhancement.

Comments

@SeaRise
Copy link
Contributor

SeaRise commented Sep 15, 2022 •
edited
Loading

Enhancement

  1. Extract execution Summary logic from DAGResponseWriter
  2. separate DAGUtils
  3. Make codec related files into one folder, more tests of codec module.
  4. Make TiDB type related files into one folder
  5. Add one abstract class for scheduler.
  6. More comments of the core classes to express the duty of them.
  7. remove DAGQueryBlock and related codes
  8. remove sync tunnel and related codes
@SeaRise SeaRise added the type/enhancement The issue or PR belongs to an enhancement. label Sep 15, 2022
This was referenced Sep 15, 2022
Merged
Merged
Merged
ti-chi-bot pushed a commit that referenced this issue Sep 20, 2022
@SeaRise
Minor refactoring of ExchangeReceiver (#5951)
4eedf09 
ref #5900
@SeaRise SeaRise mentioned this issue Sep 21, 2022
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Sep 21, 2022
@SeaRise
.*: Minor improvement for executor ut (#5977)
97de3c5 
ref #5900
ti-chi-bot pushed a commit that referenced this issue Sep 23, 2022
@SeaRise
.*: Refine storage interpreter (#5897)
c2d8ffc 
ref #5900
@SeaRise SeaRise mentioned this issue Sep 27, 2022
Merged
12 tasks
@windtalker windtalker mentioned this issue Sep 27, 2022
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Sep 27, 2022
@SeaRise
.*: Separate FineGrainedShuffleWriter, HashParitionWriter, `Broad…
ef34f3b 
…castOrPassThroughWriter` from `StreamingDAGResponseWriter` (#5910)

ref #5900
ti-chi-bot pushed a commit that referenced this issue Sep 28, 2022
@windtalker
Some scheduler related refinements (#6044)
1844f6d 
ref #5900
ti-chi-bot pushed a commit that referenced this issue Sep 29, 2022
@SeaRise
.*: refine executeQuery (#6043)
84a71f0 
ref #5900
@ywqzzy ywqzzy mentioned this issue Oct 10, 2022
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Oct 25, 2022
@ywqzzy
*: refine executionSummary (#6090)
0082b41 
ref #5900
This was referenced Oct 25, 2022
Closed
Merged
Merged
ti-chi-bot pushed a commit that referenced this issue Oct 27, 2022
@SeaRise
.*: Remove useless settings join_concurrent_build (#6183)
a4b677e 
ref #5900
This was referenced Nov 3, 2022
Merged
Merged
@ywqzzy ywqzzy mentioned this issue Nov 7, 2022
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Nov 7, 2022
@SeaRise
.*: Refine FilterBlockInputStream (#6250)
2e71675 
ref #5900
ti-chi-bot pushed a commit that referenced this issue Nov 9, 2022
@SeaRise
.*: refine streaming writer and exchange writer (#6186)
16cf524 
ref #5900
@SeaRise SeaRise mentioned this issue Nov 16, 2022
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Nov 16, 2022
@SeaRise
Move most of WindowBlockInputStream's code to `WindowTransformActio…
06fb24a 
…n` (#6317)

ref #5900
@SeaRise SeaRise mentioned this issue Nov 20, 2022
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Nov 22, 2022
@SeaRise
Split sending execution summary from Streaming/ExchangeWriter (#6335)
f121236 
ref #5900
@SeaRise SeaRise mentioned this issue Nov 22, 2022
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Nov 23, 2022
@SeaRise
.* introduce query executor (#6245)
e7be8e7 
close #4577, ref #5900
ti-chi-bot pushed a commit that referenced this issue Dec 22, 2022
@SeaRise
refine remote execution summary (#6349)
ba882a3 
ref #5900
@SeaRise SeaRise mentioned this issue Dec 22, 2022
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Dec 23, 2022
@SeaRise
move MergingAndConvertingBlockInputStream out of Aggregate.cpp (#6531)
1e7edb8 
ref #5900
@ywqzzy ywqzzy mentioned this issue Dec 23, 2022
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Dec 23, 2022
@ywqzzy
*: remove limitByStream (#6539)
c361d3a 
ref #5900
This was referenced Dec 26, 2022
Closed
Merged
ti-chi-bot pushed a commit that referenced this issue Dec 28, 2022
@SeaRise
refine agg/topn and add more unit tests (#6549)
d989239 
ref #5900
@ywqzzy ywqzzy mentioned this issue Jan 11, 2023
Merged
12 tasks
@gengliqi gengliqi mentioned this issue Jan 11, 2023
Merged
12 tasks
@SeaRise SeaRise mentioned this issue Jan 16, 2023
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Jan 16, 2023
@SeaRise
Move the logic of encoding mpp packet from exchange writer to mpp tun…
eaf593f 
…nel set (#6644)

ref #5900
@SeaRise SeaRise mentioned this issue Jan 18, 2023
Closed
12 tasks
@SeaRise SeaRise mentioned this issue Jan 28, 2023
Merged
12 tasks
@gengliqi gengliqi mentioned this issue Jan 28, 2023
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Jan 29, 2023
@SeaRise
Remove some useless settings and related codes (#6677)
cf36c1c 
ref #5900
ti-chi-bot pushed a commit that referenced this issue Jan 30, 2023
@gengliqi
Some code refine in join (#6680)
0b1ffce 
ref #5900
@SeaRise SeaRise mentioned this issue Feb 9, 2023
Merged
12 tasks
ywqzzy pushed a commit to ywqzzy/tiflash_1 that referenced this issue Feb 13, 2023
@SeaRise @ywqzzy
Remove some useless settings and related codes (pingcap#6677)
af9a930 
ref pingcap#5900

Signed-off-by: ywqzzy <[email protected]>
ywqzzy pushed a commit to ywqzzy/tiflash_1 that referenced this issue Feb 13, 2023
@gengliqi @ywqzzy
Some code refine in join (pingcap#6680)
f62cd08 
ref pingcap#5900

Signed-off-by: ywqzzy <[email protected]>
ti-chi-bot pushed a commit that referenced this issue Feb 14, 2023
@SeaRise
Refine merge agg stream (#6793)
34fc70f 
ref #5900
@SeaRise SeaRise mentioned this issue Mar 9, 2023
Closed
12 tasks
@ywqzzy ywqzzy mentioned this issue Mar 9, 2023
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Mar 21, 2023
@ywqzzy
*: Refine execution summary with ExecutorStatisticsCollector (#6995)
91f4896 
ref #5900, ref #6518
This was referenced Mar 22, 2023
Merged
Merged
ti-chi-bot pushed a commit that referenced this issue Apr 10, 2023
@SeaRise
Refine join probe (#7156)
0f82b2e 
close #5900
@ywqzzy ywqzzy mentioned this issue Apr 21, 2023
Merged
12 tasks
ti-chi-bot pushed a commit that referenced this issue Apr 21, 2023
@ywqzzy
fix build fail (#7330)
dff6653 
ref #5900
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Refine join probe SeaRise/tiflash
1 participant
@SeaRise