Skip to content

Conversation

@Mryange
Copy link
Contributor

@Mryange Mryange commented Apr 25, 2024

Proposed changes

 -  PerScannerRunningTime:  [1s194ms,  887.315ms,  1s049ms,  1s220ms,  1s209ms,  836.272ms,  1s105ms,  1s037ms,  1s063ms,  1s180ms,  1s191ms,  1s096ms,  883.761ms,  883.103ms,  1s023ms,  1s136ms,  904.661ms,  1s237ms,  1s116ms,  1s029ms,  932.601ms,  1s109ms,  1s184ms,  1s094ms,  1s256ms,  937.526ms,  1s143ms,  1s136ms,  1s047ms,  1s083ms,  940.786ms,  1s132ms,  1s055ms,  968.188ms,  1s024ms,  1s052ms,  1s045ms,  1s163ms,  1s134ms,  841.429ms,  1s064ms,  1s194ms,  1s126ms,  1s036ms,  1s012ms,  1s112ms,  1s219ms,  1s068ms,  ]
-  PerScannerProjectionTime:  [1s182ms,  850.468ms,  1s038ms,  1s164ms,  1s189ms,  825.352ms,  1s084ms,  1s017ms,  1s053ms,  1s168ms,  1s177ms,  1s058ms,  859.193ms,  833.417ms,  1s011ms,  1s122ms,  892.875ms,  1s228ms,  1s098ms,  1s006ms,  923.096ms,  1s096ms,  1s173ms,  1s079ms,  1s228ms,  927.629ms,  1s120ms,  1s099ms,  1s027ms,  1s052ms,  931.977ms,  1s099ms,  1s037ms,  949.553ms,  1s015ms,  1s042ms,  1s032ms,  1s143ms,  1s118ms,  828.954ms,  1s053ms,  1s182ms,  1s090ms,  1s022ms,  1s001ms,  1s096ms,  1s209ms,  1s057ms,  ]

Further comments

If this is a relatively large or complex change, kick off the discussion at dev@doris.apache.org by explaining why you chose the solution you did and what alternatives you considered, etc...

@doris-robot
Copy link

Thank you for your contribution to Apache Doris.
Don't know what should be done next? See How to process your PR

Since 2024-03-18, the Document has been moved to doris-website.
See Doris Document.

@Mryange
Copy link
Contributor Author

Mryange commented Apr 25, 2024

run buildall

@github-actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@doris-robot
Copy link

TeamCity be ut coverage result:
Function Coverage: 35.20% (8918/25338)
Line Coverage: 26.97% (73321/271864)
Region Coverage: 26.14% (37875/144871)
Branch Coverage: 22.96% (19283/83984)
Coverage Report: http://coverage.selectdb-in.cc/coverage/98659ef7ce4c5485b3693fd131afc126ece9704e_98659ef7ce4c5485b3693fd131afc126ece9704e/report/index.html

Copy link
Contributor

@yiguolei yiguolei left a comment

Choose a reason for hiding this comment

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

LGTM

@github-actions
Copy link
Contributor

PR approved by at least one committer and no changes requested.

@github-actions github-actions bot added approved Indicates a PR has been approved by one committer. reviewed labels Apr 25, 2024
@github-actions
Copy link
Contributor

PR approved by anyone and no changes requested.

@yiguolei yiguolei merged commit 5d5448f into apache:master Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by one committer. dev/2.0.x reviewed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants