Skip to content

Conversation

@adonis0147
Copy link
Contributor

Proposed changes

Issue Number: close #xxx

Problem summary

Backport:

  1. [chore](macOS) Use LLVM Clang by default #17292
  2. [chore](workflow) Fix the workflow BE UT (macOS) #17403
  3. [chore](macOS) Disable detect_container_overflow at BE startup #17514
  4. [chore](macOS) Specify the version of LLVM for Homebrew to install it #17945

Checklist(Required)

  • Does it affect the original behavior
  • Has unit tests been added
  • Has document been added or modified
  • Does it need to update dependencies
  • Is this PR support rollback (If NO, please explain WHY)

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...

…e#17514)

BE failed to start up due to container-overflow errors reported by address sanitizer.
…apache#17945)

Clang 16 was released last week and we haven't ported the codebase to it. If Homebrew bumped the version of LLVM, our workflows would fail.
apache#17292 enabled detect_container_overflow which made the workflow BE UT (macOS) fail.
@github-actions github-actions bot added the kind/docs Categorizes issue or PR as related to documentation. label Apr 24, 2023
@adonis0147 adonis0147 merged commit cf16cdf into apache:branch-1.2-lts Apr 24, 2023
@adonis0147 adonis0147 deleted the branch-1.2-lts branch April 24, 2023 13:46
mongo360 pushed a commit to mongo360/doris that referenced this pull request Jul 12, 2023
Backport:
1. [chore](macOS) Use LLVM Clang by default apache#17292
2. [chore](workflow) Fix the workflow BE UT (macOS) apache#17403
3. [chore](macOS) Disable detect_container_overflow at BE startup apache#17514
4. [chore](macOS) Specify the version of LLVM for Homebrew to install it apache#17945
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/docs Categorizes issue or PR as related to documentation.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant