Skip to content

Use min(limt, page size) as the init capacity of result block#11619

Merged
JackieTien97 merged 2 commits intomasterfrom
ty/meituan
Nov 27, 2023
Merged

Use min(limt, page size) as the init capacity of result block#11619
JackieTien97 merged 2 commits intomasterfrom
ty/meituan

Conversation

@JackieTien97
Copy link
Contributor

In some query cases, of the limit is much less than the count of each page, we should use limit instead of page size as the init capacity of the result block.

bfc77345-8ab2-4292-86dc-0c171c60c358

@sonarqubecloud
Copy link

SonarCloud Quality Gate failed.    Quality Gate failed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

0.0% 0.0% Coverage
0.0% 0.0% Duplication

idea Catch issues before they fail your Quality Gate with our IDE extension sonarlint SonarLint

@codecov-commenter
Copy link

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (b5214cb) 49.00% compared to head (4ec463b) 49.01%.

Files Patch % Lines
...ache/iotdb/tsfile/read/reader/page/PageReader.java 71.42% 2 Missing ⚠️
...tdb/tsfile/read/reader/page/AlignedPageReader.java 83.33% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             master   #11619   +/-   ##
=========================================
  Coverage     49.00%   49.01%           
- Complexity    24883    24889    +6     
=========================================
  Files          2810     2810           
  Lines        175991   176002   +11     
  Branches      21116    21116           
=========================================
+ Hits          86250    86266   +16     
+ Misses        89741    89736    -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@JackieTien97 JackieTien97 merged commit c7ab472 into master Nov 27, 2023
@JackieTien97 JackieTien97 deleted the ty/meituan branch November 27, 2023 08:57
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