8000 To merge develop to issue#8312 branch by The-Gamer-01 · Pull Request #9061 · alibaba/nacos · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

To merge develop to issue#8312 branch #9061

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

Merged
merged 40 commits into from
Aug 31, 2022
Merged

Conversation

The-Gamer-01
Copy link
Contributor

Please do not create a Pull Request without creating an issue first.

What is the purpose of the change

To merge develop to issue#8312 branch

Brief changelog

To merge develop to issue#8312 branch

Verifying this change

XXXX

Follow this checklist to help us incorporate your contribution quickly and easily:

  • Make sure there is a Github issue filed for the change (usually before you start working on it). Trivial changes like typos do not require a Github issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
  • Format the pull request title like [ISSUE #123] Fix UnknownException when host config not exist. Each commit in the pull request should have a meaningful subject line and body.
  • Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
  • Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add integration-test in test module.
  • Run mvn -B clean package apache-rat:check findbugs:findbugs -Dmaven.test.skip=true to make sure basic checks pass. Run mvn clean install -DskipITs to make sure unit-test pass. Run mvn clean test-compile failsafe:integration-test to make sure integration-test pass.

KomachiSion and others added 30 commits July 26, 2022 15:12
* fix health bug.

* fix health bug.

* fix health bug.

* test update

* format code

* 注释

* check service exist

* check service exist

* test fail

* user group()

* add unit test.

* add unit test.

* add unit test.

* add unit test.

* add unit test.

* add unit test.
…tion

Add exception detail stacks when connect to server failed.
* [ISSUE#8481]Unified TRACE capacity building

* [ISSUE#8481]Unified TRACE capacity building
1. Event type add to traceEvent.
2. The event about instance add instancePort.
3. Add health check type to healthChangeEvent.
4. Other fixup.
* import config file auth fail

* run ci

* run ci2
Feat/calculate ip count when use batchregister
* [ISSUE #8880] add a request exception handler

Close #8880

* [ISSUE #8880] retry ci
* bug fix calcu ipCount

* update logic
An IOException has been catch,so throws IOException Is invalid
…#8940)

* [fix] fix BatchInstanceData cannot Serializable question

* fix update serialVersionUID auto generate
[#ISSUE 8882]FIX nacos-client 2.1.0 erorr use endpoint configuration
* add some junit test

* fix some junit

* add license
Flip the call to equals method to avoid null pointer exception.
* fix ci

* run ci

* check cla report

* check style
* add some auth plugin test

* ru ci
* add unit test for ReflectUtils

* reformat code
…e[nacos-common] (#8997)

* [ISSUE #8993]Close input stream and output stream by try with resource[nacos-common]

* [ISSUE #8993]Close input stream and output stream by try with resource[nacos-common]

* [ISSUE #8993]Close input stream and output stream by try with resource[nacos-common]
chenhao26-nineteen and others added 10 commits August 22, 2022 15:43
add some utils UT
* performance optimization

* fix check style

* fix test unit

* fix test unit for `getTimeFormatOfSecond` method.

* fix unit test  for LimiterTest.

* fix error usage case for asserEquals method;eliminate interactions between test methods
* fix issue #9013

* reformat code style

* reformat code style

* reformat code style

* reformat code style

* reformat code style

* reformat code style

* reformat code style

* change config name

* do some enhance

* add queue size to configProperties

* reformat code style

* Change configuration load order
…ate (#9024)

* [ISSUE #9023] fix corner case config dataId 'cipher-' that cannot create

* [ISSUE #9023] remove unused import

* [ISSUE #9023] PMD fix

* [ISSUE #9023] add comments

* [ISSUE #9023] should return first token instead of any

* [ISSUE #9023] add more EncryptionHandler ut

* [ISSUE #9023] fix style

* [ISSUE #9023] fix style, keep indentation in blank line
@codecov-commenter
Copy link

Codecov Report

Merging #9061 (40651f6) into summer2022_issue#8312 (34bbd38) will increase coverage by 0.40%.
The diff coverage is 35.86%.

Impacted file tree graph

@@                     Coverage Diff                     @@
##             summer2022_issue#8312    #9061      +/-   ##
===========================================================
+ Coverage                    43.25%   43.65%   +0.40%     
- Complexity                    4713     4774      +61     
===========================================================
  Files                          904      912       +8     
  Lines                        32504    32599      +95     
  Branches                      3751     3765      +14     
===========================================================
+ Hits                         14058    14230     +172     
+ Misses                       17095    16964     -131     
- Partials                      1351     1405      +54     
Impacted Files Coverage Δ
...ba/nacos/client/naming/core/ServerListManager.java 38.15% <0.00%> (-0.51%) ⬇️
.../resource/PathMatchingResourcePatternResolver.java 24.54% <0.00%> (ø)
.../alibaba/nacos/common/remote/client/RpcClient.java 35.30% <0.00%> (ø)
...aba/nacos/common/remote/client/grpc/GrpcUtils.java 0.00% <0.00%> (ø)
...om/alibaba/nacos/common/trace/HealthCheckType.java 0.00% <0.00%> (ø)
...m/alibaba/nacos/common/trace/event/TraceEvent.java 0.00% <0.00%> (ø)
...ace/event/naming/DeregisterInstanceTraceEvent.java 0.00% <0.00%> (ø)
...race/event/naming/DeregisterServiceTraceEvent.java 0.00% <0.00%> (ø)
...race/event/naming/HealthStateChangeTraceEvent.java 0.00% <0.00%> (ø)
...os/common/trace/event/naming/NamingTraceEvent.java 0.00% <0.00%> (ø)
... and 83 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 34bbd38...40651f6. Read the comment docs.

@li-xiao-shuang li-xiao-shuang merged commit 14ebeb4 into summer2022_issue#8312 Aug 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0