starrocks icon indicating copy to clipboard operation
starrocks copied to clipboard

[Enhancement] Use the max_data_size of last recent partitions instead of average size in the calAvgBucketNumOfRecentPartitions()

Open chaoyli opened this issue 3 years ago • 2 comments
trafficstars

  1. To alleviate the small tablet, speculate the tablet num according to the max_data_size.
  2. The max table num of the partition must be guarantee not to be larger than calBucketNumAccordingToBackends()

What type of PR is this:

  • [ ] BugFix
  • [ ] Feature
  • [x] Enhancement
  • [ ] Refactor
  • [ ] UT
  • [ ] Doc
  • [ ] Tool

Which issues of this PR fixes :

Fixes #

Problem Summary(Required) :

Checklist:

  • [ ] I have added test cases for my bug fix or my new feature
  • [ ] I have added user document for my new feature or new function

chaoyli avatar Sep 21 '22 23:09 chaoyli

[FE PR Coverage Check]

:disappointed: fail : 0 / 4 (00.00%)

file detail

path covered_line new_line coverage not_covered_line_detail
:large_blue_circle: com/starrocks/server/LocalMetastore.java 0 4 00.00% [975, 981, 983, 987]

wanpengfei-git avatar Sep 26 '22 01:09 wanpengfei-git

run starrocks_admit_test

wanpengfei-git avatar Sep 27 '22 01:09 wanpengfei-git

run starrocks_admit_test

Astralidea avatar Sep 27 '22 01:09 Astralidea