forked from apache/dubbo
-
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
update #1
Open
hnlszg881219
wants to merge
1,222
commits into
hnlszg881219:master
Choose a base branch
from
apache:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
update #1
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
update |
#7902) * improve the code that RegistryDirectory.java#destroyUnusedInvokers and RegistryDirectory.java#toMergeInvokerList * 1 fix with RegistryDirectory.java: remove unused imports Co-authored-by: zhongxiong.zeng <[email protected]>
* fix #7888 add test case for ServerStatusChecker * use available port * modify getAvailablePort * fix testGetAvailablePort test case. * modify min_port=1 * make getAvailablePort Sync * use random port as init value * bugfix * Use bitset to optimize code * remote unused import * remove unused import * bugfix * bugfix * add log * bugfix * bugfix
* ssl support ciphers&protocals * fix spell bug * spell error * add test * fix import * * add ASF license * delete no need
…ig,MetadataReportConfig. (#7468)
* Cache the parsed result of the HelpCommand and HelpTelnetHandler * Modify the map type to WeakHashMap
* fix #7923 Fix DubboProtocol is not directly used externally * add cleanup when destroy * bugfix * add invoker.destroy
* Add unit test for Ready,PublishMetadata,Version command * Add clear mechanism in unit test * Fix unit test * Fix unit test * Fix unit test * Fix unit test npe * Fix unit test * Fix unit test * Debug ut * Fix ut * Debug ut * Debug ut * Debug ut * Debug ut * Debug ut * Revert * Debug ut * Debug ut * Remove ReadyTest
* Use constants instead of literals * Remove unused variables * Remove unnecessary modifiers * Use constants instead of literals
* fill conditionKeys collections when necessary * Use set container to handle multiple tildeKey situations
…DubboBootstrap#destroy method does not work (#7965)
* fix #7916 add test case for DubboProtocol * add log * add log * add log * bugfix
* Fix thrift ASF license header * add .mvn ignore * add ASF header * remove protobuf notice
``` // instanceof not throw NullPointExcetion if (result.getException() != null && result.getException() instanceof RpcException) // Examle: public static void main(String[] args) { Exception e = null; if (e instanceof Exception) { System.out.println("instanceof"); } System.out.println("success"); } //Out: success ```
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.
What is the purpose of the change
XXXXX
Brief changelog
XXXXX
Verifying this change
XXXXX
Follow this checklist to help us incorporate your contribution quickly and easily:
[Dubbo-XXX] Fix UnknownException when host config not exist #XXX
. Each commit in the pull request should have a meaningful subject line and body.mvn clean install -DskipTests=false
&mvn clean test-compile failsafe:integration-test
to make sure unit-test and integration-test pass.