forked from apache/rocketmq
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
from apache #6
Merged
Merged
from apache #6
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…in README.md (#6582) * Add a description of the Apache RocketMQ E2E repository in README.md * Update README.md Co-authored-by: Aaron Ai <[email protected]> --------- Co-authored-by: Aaron Ai <[email protected]>
…the maximum value of short (#6580) * Prevent the properties of trace message from exceeding the maximum value of short * Use reduce instead of foreach
* feat(proxy): improve proxy context Signed-off-by: lyx <[email protected]> * fix(proxy): rollback to fullname for compatibility Signed-off-by: lyx2000 <[email protected]> * fix(proxy): use simple_rpc_name add to ProxyContext action Signed-off-by: lyx <[email protected]> * fix(proxy): improve naming Signed-off-by: lyx <[email protected]> --------- Signed-off-by: lyx <[email protected]> Signed-off-by: lyx2000 <[email protected]>
Co-authored-by: guyinyou <[email protected]>
…#6531) * change map -> lambda * f * fix unit test * remove getTopicConfig function * Update MultiDispatchTest.java * Update CompactionStore.java * update * update test * update test * Update BatchConsumeMessageTest.java * Update BrokerController.java * Update BrokerController.java * check * Update BrokerController.java * Update BatchConsumeMessageTest.java
…efault java command in Darwin contained Spaces (#6592)
#6590) * (fix): logback config path seperator and brokerLogDir default value missing * (fix): replace / with file.seperator in proxy test logback config --------- Co-authored-by: Robin Han <[email protected]>
* change map -> lambda * f * fix unit test * remove getTopicConfig function * Update MultiDispatchTest.java * Update CompactionStore.java * update * update test * update test * Update BatchConsumeMessageTest.java * Update BrokerController.java * Update BrokerController.java * check * Update BrokerController.java * Update BatchConsumeMessageTest.java * prevent redundant validator * fix * ci * ci * Update AuthenticationPipeline.java * Update RemotingProtocolServer.java * Update AuthenticationPipeline.java * Update RemotingProtocolServer.java
Bumps [spring-core](https://github.com/spring-projects/spring-framework) from 5.3.26 to 5.3.27. - [Release notes](https://github.com/spring-projects/spring-framework/releases) - [Commits](spring-projects/spring-framework@v5.3.26...v5.3.27) --- updated-dependencies: - dependency-name: org.springframework:spring-core dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…o latest develop because compactionThreadNum is 0 (#6611)
…#6632) Co-authored-by: 斜阳 <[email protected]>
…rrentLogicOffset in consumeQueue build when the message is illegal (#6641) * Fix the issue that expectLogicOffset is greater than currentLogicOffset in consumeQueue build when the message is illegal * Add new UT * Fix bug that UT can not pass * Polish the variable name * Polish the comment * Add more comments
…ocessed via TieredStorage when level is FORCE (#6649) * fix(tieredstorage): fix the bug that not all message fetch request is processed via tieredStorage when enable FORCE 1. fix the bug that not all message fetch request is processed via tieredStorage when enable FORCE * style(tieredstorage): use rmq code style 1. use rmq code style * feat(tieredstorage): add more comments for `TieredStorageLevel` 1. add more comments for `TieredStorageLevel`
…iskByConsumeOffset` (#6651) * fix(tieredstorage): fix using deprecated method `MessageStore#checkInDiskByConsumeOffset` 1. fix using deprecated method `MessageStore#checkInDiskByConsumeOffset` * test(tieredstorage): replace `checkInDiskeByConsumeOffset` with `checkInStoreByConsumeOffset` 1. replace `checkInDiskeByConsumeOffset` with `checkInStoreByConsumeOffset`
…6663) * When compute confirmOffset, judge whether the slaves in syncStateSet all connect to the master. * fix the brokerId to brokerControllerId. * optimize the code: 1. Set brokerId when the replicasManager inited. 2. Rename the confusing name. * set haService's brokerControllerId after registered. * set the haService's brokerControllerId when the ReplicasManager start without brokerIdentity file.
…rmOffset when node restarts to recover (#6618) * Fix the issue that consume queue building exceeds confirmOffset when node restarts to recover
* 1. Fix the bug that confirmOffset may be larger than ReputOffset in some cases. 2. Optimize the logic when truncate dirty files. * Add more logs while expanding SyncStateSet. * reset the confirmOffset to the truncated offset. * optimize the log. * move setConfirmOffset() process into commitLog. * add the judge.
…tion occurs (#7112) * remove responseFuture when exception * Empty-Commit --------- Co-authored-by: chenyong152 <[email protected]>
…ge (#7281) * refactor handle commit exception * refactor handle commit exception * fix handle commit exception
… re delete failed (#7284)
Co-authored-by: guyinyou <[email protected]>
… broker offline cause coredump(#7307)
…and hot update switch (#7320)
* Convergence thread pool creation to facilitate subsequent iteration management * Convergence thread pool creation in ThreadPoolMonitor.java * fix unit test * Convergence ThreadPool constructor * Convergence ScheduledThreadPool constructor * remove unused import * Convergence ScheduledThreadPool constructor * remove unused import ---------
…7335) Signed-off-by: Ziy1-Tan <[email protected]>
Signed-off-by: Ziy1-Tan <[email protected]>
…erent scenarios (#7317)
Co-authored-by: guyinyou <[email protected]>
* Set experimental_ui_max_stdouterr_bytes to 10MB and Fix IT
Co-authored-by: guyinyou <[email protected]>
…xt pull offset (#7365)
… performance Co-authored-by: guyinyou <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Make sure set the target branch to
develop
What is the purpose of the change
fix #
Brief changelog
XX
Verifying this change
XXXX
Follow this checklist to help us incorporate your contribution quickly and easily. Notice,
it would be helpful if you could finish the following 5 checklist(the last one is not necessary)before request the community to review your PR
.[ISSUE #123] Fix UnknownException when host config not exist
. Each commit in the pull request should have a meaningful subject line and body.mvn -B clean apache-rat:check findbugs:findbugs checkstyle:checkstyle
to make sure basic checks pass. Runmvn clean install -DskipITs
to make sure unit-test pass. Runmvn clean test-compile failsafe:integration-test
to make sure integration-test pass.