Skip to content

[Profile][UT] Fix UT and remove useless profile - #4879

Merged
morningman merged 2 commits into
apache:masterfrom
morningman:fix_profile_ut
Nov 12, 2020
Merged

[Profile][UT] Fix UT and remove useless profile#4879
morningman merged 2 commits into
apache:masterfrom
morningman:fix_profile_ut

Conversation

@morningman

@morningmanmorningman commented Nov 12, 2020•
edited
Loading

Copy link
Copy Markdown
Contributor

Proposed changes

Fix UT failed by #4825 and remove useless profile

Types of changes

  • Bugfix (non-breaking change which fixes an issue)

@morningmanmorningman self-assigned this Nov 12, 2020
yangzhg
yangzhg approved these changes Nov 12, 2020
View reviewed changes

@yangzhgyangzhg left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1

@morningman

morningman commented Nov 12, 2020

Copy link
Copy Markdown
ContributorAuthor

Quick merge to fix UT.
NewDorisTest failed because another problem, will be fixed later.

@morningmanmorningman added the approved Indicates a PR has been approved by one committer. label Nov 12, 2020
@morningman
morningman merged commit e992310 into apache:masterNov 12, 2020
yiguolei pushed a commit to yiguolei/incubator-doris that referenced this pull request Dec 19, 2025
@morrySnow
branch-2.1: [opt](compatibility) let version mutableapache#55335(ap…
f59617c
…ache#4879)
picked from apache#55335
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yangzhgyangzhgyangzhg approved these changes

Assignees

@morningmanmorningman

Labels

approvedIndicates a PR has been approved by one committer.kind/test

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@morningman@yangzhg@morningman-cmy