FreeOnePlus opened a new pull request, #45267:
URL: https://github.com/apache/doris/pull/45267
### What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
In the storage_engine.cpp of the BE process, it is forcibly checked that the
`ulimit` value must be greater than 60000. When starting with Docker or
Docker-Compose, it is necessary to preemptively change the corresponding value
on the host machine. This change is very unfriendly to Docker, as it loses its
unique advantage of being able to "build quickly and anywhere," and cannot
become a fundamental capability for rapid startup. Therefore, a new parameter
has been added to control whether to skip this check value. The default value
is false, which enforces the check of this value. When set to true, it skips
the check and starts directly.
### Release note
None
### Check List (For Author)
- Test <!-- At least one of them must be included. -->
- [ ] Regression test
- [ ] Unit Test
- [x] Manual test (add detailed scripts or steps below)
- [ ] No need to test or manual test. Explain why:
- [ ] This is a refactor/code format and no logic has been changed.
- [ ] Previous test can cover this change.
- [ ] No code files have been changed.
- [ ] Other reason <!-- Add your reason? -->
1. Execute `ulimit -n 50000` to make it less than the BE check value of
60000.
2. Comment out the part that checks the ulimit size in the start_be.sh
script.
3. Add the `skip_check_ulimit = true` configuration item to the be.conf file.
4. The BE process can start normally.
5. Comment out the added `skip_check_ulimit = true` configuration item.
6. The BE cannot start, indicating a problem with the ulimit value being too
small.
- Behavior changed:
- [x] No.
- [ ] Yes. <!-- Explain the behavior change -->
- Does this need documentation?
- [x] No.
- [ ] Yes. <!-- Add document PR link here. eg:
https://github.com/apache/doris-website/pull/1214 -->
### Check List (For Reviewer who merge this PR)
- [x] Confirm the release note
- [x] Confirm test cases
- [x] Confirm document
- [ ] Add branch pick label <!-- Add branch pick label that this PR should
merge into -->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
