Bump org.apache.maven.plugins:maven-dependency-plugin (#14364)

Bumps [org.apache.maven.plugins:maven-dependency-plugin](https://github.com/apache/maven-dependency-plugin) from 3.7.0 to 3.7.1.
- [Release notes](https://github.com/apache/maven-dependency-plugin/releases)
- [Commits](https://github.com/apache/maven-dependency-plugin/compare/maven-dependency-plugin-3.7.0...maven-dependency-plugin-3.7.1)

---
updated-dependencies:
- dependency-name: org.apache.maven.plugins:maven-dependency-plugin
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

.asf.yaml

@@ -55,3 +55,5 @@ github:
     - http
     - grpc
     - web
+  collaborators:
+    - oxsean

.editorconfig

@@ -19,13 +19,55 @@
 # top-most EditorConfig file
 root = true
 
-# Unix-style newlines with a newline ending every file
 [*]
 charset = utf-8
 end_of_line = lf
-insert_final_newline = true
-
-# 4 space indentation
-[*.{java,xml}]
-indent_style = space
 indent_size = 4
+indent_style = space
+tab_width = 4
+max_line_length = 120
+insert_final_newline = true
+trim_trailing_whitespace = true
+
+[*.java]
+ij_java_continuation_indent_size = 8
+ij_java_keep_control_statement_in_one_line = false
+ij_java_for_brace_force = always
+ij_java_if_brace_force = always
+ij_java_keep_first_column_comment = false
+ij_java_keep_line_breaks = false
+ij_java_keep_simple_blocks_in_one_line = true
+ij_java_keep_simple_classes_in_one_line = true
+ij_java_keep_simple_lambdas_in_one_line = true
+ij_java_keep_simple_methods_in_one_line = true
+ij_java_keep_blank_lines_in_code = 1
+ij_java_keep_blank_lines_in_declarations = 1
+ij_java_class_count_to_use_import_on_demand = 999
+ij_java_names_count_to_use_import_on_demand = 999
+ij_java_imports_layout = org.apache.dubbo.**, |, javax.**, |, java.**, |, *, |, $*
+ij_java_insert_inner_class_imports = true
+ij_java_space_before_array_initializer_left_brace = true
+ij_java_method_parameters_new_line_after_left_paren = true
+ij_java_wrap_comments = true
+ij_java_wrap_long_lines = true
+ij_java_enum_constants_wrap = split_into_lines
+ij_java_method_call_chain_wrap = split_into_lines
+ij_java_method_parameters_wrap = on_every_item
+ij_java_extends_list_wrap = on_every_item
+ij_java_extends_keyword_wrap = normal
+ij_java_binary_operation_wrap = normal
+ij_java_binary_operation_sign_on_next_line = true
+
+[*.json]
+tab_width = 2
+
+[*.{yml,yaml}]
+indent_size = 2
+
+[*.xml]
+ij_xml_attribute_wrap = off
+ij_xml_text_wrap = off
+ij_xml_keep_blank_lines = 1
+
+[pom.xml]
+indent_size = 2

.github/DISCUSSION_TEMPLATE/general.yml

@@ -0,0 +1,88 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+title: "[General][Java SDK (Component)] xxx"
+body:
+- type: markdown
+  attributes:
+    value: |
+      <img src="https://dubbo.apache.org/imgs/nav_logo2.png" alt="Dubbo logo" height="90px" align="right" />
+
+      Thank you for finding the time to share your idea! We really appreciate the community efforts to improve Dubbo.
+
+      If you need to report a security issue, please visit [our security policy](https://github.com/apache/dubbo/security/policy).
+
+      **Dubbo is open for world wide collaboration, please make sure that all the content you provide is in English.**
+      Remember that non-English idea is quite not friendly for everyone, and might unable to get the response!
+
+- type: checkboxes
+  attributes:
+    label: Pre-check
+    options:
+      - label: >
+          I am sure that all the content I provide is in English.
+        required: true
+
+- type: dropdown
+  attributes:
+    label: Apache Dubbo Component
+    description: |
+      What Apache Dubbo component are you using? Apache Dubbo has many subprojects, please make sure
+      to choose the component that you want to ask questions about.
+    multiple: false
+    options:
+      - "Java SDK (apache/dubbo)"
+      - "Java Samples (apache/dubbo-samples)"
+      - "Java Integration Cases (apache/dubbo-integration-cases)"
+      - "Java SPI Extensions (apache/dubbo-spi-extensions)"
+      - "Java Benchmark (apache/dubbo-benchmark)"
+      - "Go SDK (apache/dubbo-go)"
+      - "Go Samples (apache/dubbo-go-samples)"
+      - "Rust SDK (apache/dubbo-rust)"
+      - "Node.js SDK (apache/dubbo-js)"
+      - "Python SDK (apache/dubbo-python)"
+      - "Kubernetes Integration (apache/dubbo-kubernetes)"
+      - "Pixiu Gateway (apache/dubbo-go-pixiu)"
+      - "Pixiu Gateway Samples (apache/dubbo-go-pixiu-samples)"
+      - "Admin (apache/dubbo-admin)"
+      - "Website (apache/dubbo-website)"
+      - "Awesome (apache/dubbo-awesome)"
+      - "Initializer (apache/dubbo-intializer)"
+      - "Others (apache/dubbo-xxx)"
+  validations:
+    required: true
+
+- type: textarea
+  attributes:
+    label: Details
+    description: Anything you want to ask?
+  validations:
+    required: true
+
+- type: checkboxes
+  attributes:
+    label: Code of Conduct
+    description: The Code of Conduct helps create a safe space for everyone. We require that everyone agrees to it.
+    options:
+      - label: >
+          I agree to follow this project's
+          [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
+        required: true
+
+- type: markdown
+  attributes:
+    value: "Thanks for completing our form!"

.github/DISCUSSION_TEMPLATE/question.yml

@@ -0,0 +1,88 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+title: "[Q/A][Java SDK (Component)] xxx"
+body:
+- type: markdown
+  attributes:
+    value: |
+      <img src="https://dubbo.apache.org/imgs/nav_logo2.png" alt="Dubbo logo" height="90px" align="right" />
+
+      Thank you for finding the time to report the question! We really appreciate the community efforts to improve Dubbo.
+
+      If you need to report a security issue, please visit [our security policy](https://github.com/apache/dubbo/security/policy).
+
+      **Dubbo is open for world wide collaboration, please make sure that all the content you provide is in English.**
+      Remember that non-English question is quite not friendly for everyone, and might unable to get the response!
+
+- type: checkboxes
+  attributes:
+    label: Pre-check
+    options:
+      - label: >
+          I am sure that all the content I provide is in English.
+        required: true
+
+- type: dropdown
+  attributes:
+    label: Apache Dubbo Component
+    description: |
+      What Apache Dubbo component are you using? Apache Dubbo has many subprojects, please make sure
+      to choose the component that you want to ask questions about.
+    multiple: false
+    options:
+      - "Java SDK (apache/dubbo)"
+      - "Java Samples (apache/dubbo-samples)"
+      - "Java Integration Cases (apache/dubbo-integration-cases)"
+      - "Java SPI Extensions (apache/dubbo-spi-extensions)"
+      - "Java Benchmark (apache/dubbo-benchmark)"
+      - "Go SDK (apache/dubbo-go)"
+      - "Go Samples (apache/dubbo-go-samples)"
+      - "Rust SDK (apache/dubbo-rust)"
+      - "Node.js SDK (apache/dubbo-js)"
+      - "Python SDK (apache/dubbo-python)"
+      - "Kubernetes Integration (apache/dubbo-kubernetes)"
+      - "Pixiu Gateway (apache/dubbo-go-pixiu)"
+      - "Pixiu Gateway Samples (apache/dubbo-go-pixiu-samples)"
+      - "Admin (apache/dubbo-admin)"
+      - "Website (apache/dubbo-website)"
+      - "Awesome (apache/dubbo-awesome)"
+      - "Initializer (apache/dubbo-intializer)"
+      - "Others (apache/dubbo-xxx)"
+  validations:
+    required: true
+
+- type: textarea
+  attributes:
+    label: Details
+    description: Anything you want to know?
+  validations:
+    required: true
+
+- type: checkboxes
+  attributes:
+    label: Code of Conduct
+    description: The Code of Conduct helps create a safe space for everyone. We require that everyone agrees to it.
+    options:
+      - label: >
+          I agree to follow this project's
+          [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
+        required: true
+
+- type: markdown
+  attributes:
+    value: "Thanks for completing our form!"

.github/ISSUE_TEMPLATE/1-bug.yml

@@ -0,0 +1,142 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+name: Bug Report
+description: File a bug report
+title: "[Bug] "
+labels: ["type/need-triage", "component/need-triage"]
+projects: ["apache/337"]
+body:
+- type: markdown
+  attributes:
+    value: |
+      <img src="https://dubbo.apache.org/imgs/nav_logo2.png" alt="Dubbo logo" height="90px" align="right" />
+
+      Thank you for finding the time to report the problem! We really appreciate the community efforts to improve Dubbo.
+
+      Please make sure what you are reporting is indeed a bug with reproducible steps, if you want to ask questions
+      or share ideas, please [subscribe to our mailing list](mailto:dev-subscribe@dubbo.apache.org) and sent
+      emails to [our mailing list](mailto:dev@dubbo.apache.org), you can also head to our
+      [Discussion](https://github.com/apache/dubbo/discussions) tab.
+
+      If you need to report a security issue, please visit [our security policy](https://github.com/apache/dubbo/security/policy).
+
+      **Dubbo is open for world wide collaboration, please make sure that all the content you provide is in English.**
+      Remember that non-English issues is quite not friendly for everyone, and might unable to get the response!
+
+- type: checkboxes
+  attributes:
+    label: Pre-check
+    options:
+      - label: >
+          I am sure that all the content I provide is in English.
+        required: true
+
+- type: checkboxes
+  attributes:
+    label: Search before asking
+    description: >
+      Please make sure to search in the [issues](https://github.com/apache/dubbo/issues?q=is%3Aissue) first to see
+      whether the same issue was reported already.
+    options:
+      - label: >
+          I had searched in the [issues](https://github.com/apache/dubbo/issues?q=is%3Aissue) and found no similar
+          issues.
+        required: true
+
+- type: dropdown
+  attributes:
+    label: Apache Dubbo Component
+    description: |
+      What Apache Dubbo component are you using? Apache Dubbo has many subprojects, please make sure
+      to choose the component that you want to ask questions about.
+    multiple: false
+    options:
+      - "Java SDK (apache/dubbo)"
+      - "Java Samples (apache/dubbo-samples)"
+      - "Java Integration Cases (apache/dubbo-integration-cases)"
+      - "Java SPI Extensions (apache/dubbo-spi-extensions)"
+      - "Java Benchmark (apache/dubbo-benchmark)"
+  validations:
+    required: true
+
+- type: textarea
+  attributes:
+    label: Dubbo Version
+    description: "Which Dubbo version, JDK version and operating system did you use?"
+    placeholder: "Example: Dubbo Java 3.2.12, OpenJDK 1.8, Ubuntu 20.04"
+  validations:
+    required: true
+
+- type: textarea
+  attributes:
+    label: Steps to reproduce this issue
+    description: >
+      Describe how to reproduce this issue.If you are not able to provide a reproducible case,
+      please open a [Discussion](https://github.com/apache/dubbo/discussions) instead.
+    placeholder: >
+      Please provide the context in which the problem occurred and explain what happened.
+      A [GitHub address] would be helpful for maintainers to reproduce the problem.
+  validations:
+    required: true
+
+- type: textarea
+  attributes:
+    label: What you expected to happen
+    description: What do you think went wrong?
+    placeholder: >
+      Please explain why you think the behaviour is erroneous. It is extremely helpful if you copy and paste
+      the fragment of logs showing the exact error messages or wrong behaviour and screenshots for
+      UI problems. You can include files by dragging and dropping them here.
+
+      **NOTE**: please copy and paste texts instead of taking screenshots of them for easy future search.
+  validations:
+    required: true
+
+- type: textarea
+  attributes:
+    label: Anything else
+    description: Anything else we need to know?
+    placeholder: >
+      How often does this problem occur? (Once? Every time? Only when certain conditions are met?)
+      Any relevant logs to include? Put them here inside fenced
+      ``` ``` blocks or inside a collapsable details tag if it's too long:
+      <details><summary>x.log</summary> lots of stuff </details>
+
+- type: checkboxes
+  attributes:
+    label: Are you willing to submit a pull request to fix on your own?
+    description: >
+      This is absolutely not required, but we are happy to guide you in the contribution process
+      especially if you already have a good understanding of how to implement the fix.
+      Dubbo is a totally community-driven project and we love to bring new contributors in.
+    options:
+      - label: Yes I am willing to submit a pull request on my own!
+
+- type: checkboxes
+  attributes:
+    label: Code of Conduct
+    description: The Code of Conduct helps create a safe space for everyone. We require that everyone agrees to it.
+    options:
+      - label: >
+          I agree to follow this project's
+          [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
+        required: true
+
+- type: markdown
+  attributes:
+    value: "Thanks for completing our form!"

.github/ISSUE_TEMPLATE/2-feature.yml

@@ -0,0 +1,106 @@
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+name: Feature Request
+description: Create a Feature Request for Dubbo
+title: "[Feature] "
+labels: ["type/need-triage", "component/need-triage"]
+projects: ["apache/337"]
+body:
+- type: markdown
+  attributes:
+    value: |
+      <img src="https://dubbo.apache.org/imgs/nav_logo2.png" alt="Dubbo logo" height="90px" align="right" />
+
+      Thank you for finding the time to propose new feature!
+
+      We really appreciate the community efforts to improve Dubbo.
+
+      **Dubbo is open for world wide collaboration, please make sure that all the content you provide is in English.**
+      Remember that non-English issues is quite not friendly for everyone, and might unable to get the response!
+
+- type: checkboxes
+  attributes:
+    label: Pre-check
+    options:
+      - label: >
+          I am sure that all the content I provide is in English.
+        required: true
+
+- type: checkboxes
+  attributes:
+    label: Search before asking
+    description: >
+      Please make sure to search in the [issues](https://github.com/apache/dubbo/issues?q=is%3Aissue) first to see
+      whether the same feature was requested already.
+    options:
+      - label: >
+          I had searched in the [issues](https://github.com/apache/dubbo/issues?q=is%3Aissue) and found no similar
+          feature requirement.
+        required: true
+
+- type: dropdown
+  attributes:
+    label: Apache Dubbo Component
+    description: |
+      What Apache Dubbo component are you using? Apache Dubbo has many subprojects, please make sure
+      to choose the component that you want to ask questions about.
+    multiple: false
+    options:
+      - "Java SDK (apache/dubbo)"
+      - "Java Samples (apache/dubbo-samples)"
+      - "Java Integration Cases (apache/dubbo-integration-cases)"
+      - "Java SPI Extensions (apache/dubbo-spi-extensions)"
+      - "Java Benchmark (apache/dubbo-benchmark)"
+  validations:
+    required: true
+
+- type: textarea
+  attributes:
+    label: Descriptions
+    description: A short description of your feature
+  validations:
+    required: true
+
+- type: textarea
+  attributes:
+    label: Related issues
+    description: Is there currently another issue associated with this?
+
+- type: checkboxes
+  attributes:
+    label: Are you willing to submit a pull request to fix on your own?
+    description: >
+      This is absolutely not required, but we are happy to guide you in the contribution process
+      especially if you already have a good understanding of how to implement the feature.
+      Dubbo is a totally community-driven project and we love to bring new contributors in.
+    options:
+      - label: Yes I am willing to submit a pull request on my own!
+
+- type: checkboxes
+  attributes:
+    label: Code of Conduct
+    description: The Code of Conduct helps create a safe space for everyone. We require that everyone agrees to it.
+    options:
+      - label: >
+          I agree to follow this project's
+          [Code of Conduct](https://www.apache.org/foundation/policies/conduct)
+        required: true
+
+- type: markdown
+  attributes:
+    value: "Thanks for completing our form!"

.github/ISSUE_TEMPLATE/bug_report.md

@@ -1,39 +0,0 @@
----
-name: Bug Report
-about: Report a bug in Dubbo
-title: ''
-labels: type/bug
-assignees: ''
-
----
-<!-- If you need to report a security issue please visit https://github.com/apache/dubbo/security/policy -->
-
-- [ ] I have searched the [issues](https://github.com/apache/dubbo/issues) of this repository and believe that this is not a duplicate.
-
-### Environment
-
-* Dubbo version: xxx
-* Operating System version: xxx
-* Java version: xxx
-
-### Steps to reproduce this issue
-
-1. xxx
-2. xxx
-3. xxx
-
-Pls. provide [GitHub address] to reproduce this issue.
-
-### Expected Behavior
-
-<!-- What do you expect from the above steps？-->
-
-### Actual Behavior
-
-<!-- What actually happens? -->
-
-If there is an exception, please attach the exception trace:
-
-```
-Just put your stack trace here!
-```

.github/ISSUE_TEMPLATE/config.yml

@@ -1,8 +1,22 @@
-blank_issues_enabled: true
+#
+# Licensed to the Apache Software Foundation (ASF) under one or more
+# contributor license agreements.  See the NOTICE file distributed with
+# this work for additional information regarding copyright ownership.
+# The ASF licenses this file to You under the Apache License, Version 2.0
+# (the "License"); you may not use this file except in compliance with
+# the License.  You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+#
+
+blank_issues_enabled: false
 contact_links:
-  - name: Dubbo Community Repo
+  - name: Question & FAQ & Proposal
-    url: https://github.com/apache/dubbo-website
+    url: https://github.com/apache/dubbo/discussions/
-    about: Please see our community docs here.
+    about: Ask a question, request support or submit a proposal for Apache Dubbo.
-  - name: Report a security issue
-    url: https://github.com/apache/dubbo/security/policy
-    about: Please report security vulnerabilities using these instructions.

.github/ISSUE_TEMPLATE/discussion.md

@@ -1,11 +0,0 @@
----
-name: Discussion
-about: Start a discussion for Dubbo
-title: ''
-labels: type/discussion
-assignees: ''
----
-
-<!-- If you need to report a security issue please visit https://github.com/apache/dubbo/security/policy -->
-
-<!-- For all design discussions please continue. -->

.github/ISSUE_TEMPLATE/feature_request.md

@@ -1,16 +0,0 @@
----
-name: Feature Request
-about: Create a Feature Request for Dubbo
-title: ''
-labels: type/feature
-assignees: ''
-
----
-<!-- If you need to report a security issue please visit https://github.com/apache/dubbo/security/policy -->
-
-- [ ] I have searched the [issues](https://github.com/apache/dubbo/issues) of this repository and believe that this is not a duplicate.
-- [ ] I have searched the [release notes](https://github.com/apache/dubbo/releases) of this repository and believe that this is not a duplicate.
-
-## Describe the feature
-<!-- Please also discuss possible business value -->
-

.github/ISSUE_TEMPLATE/proposal.md

@@ -1,15 +0,0 @@
----
-name: Proposal
-about: Create a technical proposal for Dubbo
-title: ''
-labels: type/proposal
-assignees: ''
-
----
-<!-- If you need to report a security issue please visit https://github.com/apache/dubbo/security/policy -->
-
-- [ ] I have searched the [issues](https://github.com/apache/dubbo/issues) of this repository and believe that this is not a duplicate.
-
-## Describe the proposal
-<!-- Please use this for a concrete design proposal for functionality. -->
-<!-- If you just want to request a new feature and discuss the possible business value, create a Feature Request. -->

.github/ISSUE_TEMPLATE/question.md

@@ -1,13 +0,0 @@
----
-name: Question
-about: Ask a question about Dubbo
-title: ''
-labels: type/question
-assignees: ''
-
----
-<!-- If you need to report a security issue please visit https://github.com/apache/dubbo/security/policy -->
-
-- [ ] I have searched the [issues](https://github.com/apache/dubbo/issues) of this repository and believe that this is not a duplicate.
-
-## Ask your question here

.github/workflows/build-and-test-pr.yml

@@ -13,13 +13,54 @@ env:
   VERSIONS_LIMIT: 4
   JACOCO_ENABLE: true
   CANDIDATE_VERSIONS: '
-    spring.version:4.3.30.RELEASE;
+    spring.version:5.3.24;
-    spring-boot.version:1.5.22.RELEASE;
+    spring-boot.version:2.7.6;
-    spring-boot.version:2.4.1;
     '
 
 jobs:
+  check-format:
+    name: Check if code needs formatting
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+      - name: Setup Maven
+        uses: actions/setup-java@v4
+        with:
+          java-version: 21
+          distribution: zulu
+      - name: Check if code aligns with code style
+        id: check
+        run: mvn --log-file mvn.log spotless:check
+        continue-on-error: true
+      - name: Upload checkstyle result
+        uses: actions/upload-artifact@v4
+        with:
+          name: checkstyle-result
+          path: mvn.log
+      - name: Generate Summary for successful run
+        if: ${{ steps.check.outcome == 'success' }}
+        run: |
+          echo ":ballot_box_with_check: Kudos! No formatting issues found!" >> $GITHUB_STEP_SUMMARY
+      - name: Generate Summary for failed run
+        if: ${{ steps.check.outcome == 'failure' }}
+        run: |
+          echo "## :negative_squared_cross_mark: Formatting issues found!" >> $GITHUB_STEP_SUMMARY
+          echo "\`\`\`" >> $GITHUB_STEP_SUMMARY
+          cat mvn.log | grep "ERROR" | sed 's/Check if code needs formatting    Check if code aligns with code style   [0-9A-Z:.-]\+//' | sed 's/\[ERROR] //' | head -n -11 >> $GITHUB_STEP_SUMMARY
+          echo "\`\`\`" >> $GITHUB_STEP_SUMMARY
+          echo "Please run \`mvn spotless:apply\` to fix the formatting issues." >> $GITHUB_STEP_SUMMARY
+      - name: Fail if code needs formatting
+        if: ${{ steps.check.outcome == 'failure' }}
+        uses: actions/github-script@v7.0.1
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          script: |
+            core.setFailed("Formatting issues found!")
+
   license:
+    name: "Check License"
+    needs: check-format
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
@@ -27,11 +68,11 @@ jobs:
         uses: apache/skywalking-eyes@e1a02359b239bd28de3f6d35fdc870250fa513d5
         env:
           GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-      - name: "Set up JDK 17"
+      - name: "Set up JDK 21"
         uses: actions/setup-java@v3
         with:
           distribution: 'zulu'
-          java-version: 17
+          java-version: 21
       - name: "Compile Dubbo (Linux)"
         run: |
           ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast -T 2C clean install -DskipTests=true -DskipIntegrationTests=true -Dcheckstyle.skip=true -Dcheckstyle_unix.skip=true -Drat.skip=true -Dmaven.javadoc.skip=true
@@ -44,6 +85,8 @@ jobs:
           mode: check
 
   build-source:
+    name: "Build Dubbo"
+    needs: check-format
     runs-on: ubuntu-latest
     outputs:
       version: ${{ steps.dubbo-version.outputs.version }}
@@ -99,6 +142,7 @@ jobs:
 
   unit-test-prepare:
     name: "Preparation for Unit Test"
+    needs: check-format
     runs-on: ubuntu-latest
     strategy:
       fail-fast: false
@@ -147,7 +191,7 @@ jobs:
           echo "--- End Print Rsa Public Key"
 
   unit-test:
-    needs: [build-source, unit-test-prepare]
+    needs: [check-format, build-source, unit-test-prepare]
     name: "Unit Test On ubuntu-latest"
     runs-on: ubuntu-latest
     strategy:
@@ -164,7 +208,7 @@ jobs:
         uses: actions/setup-java@v3
         with:
           distribution: 'zulu'
-          java-version: 17
+          java-version: 21
       - uses: actions/cache@v3
         name: "Cache local Maven repository"
         with:
@@ -212,7 +256,8 @@ jobs:
           name: coverage-result
           path: "**/target/site/**/jacoco.xml"
 
-  integration-test-prepare:
+  samples-test-prepare:
+    needs: check-format
     runs-on: ubuntu-latest
     env:
       JOB_COUNT: 3
@@ -227,12 +272,11 @@ jobs:
       - name: "Upload test list"
         uses: actions/upload-artifact@v3
         with:
-          name: test-list
+          name: samples-test-list
           path: test/jobs
-
+  samples-test-job:
-  integration-test-job:
+    needs: [check-format, build-source, samples-test-prepare]
-    needs: [build-source, integration-test-prepare]
+    name: "Samples Test on ubuntu-latest (JobId: ${{matrix.job_id}})"
-    name: "Integration Test on ubuntu-latest (JobId: ${{matrix.job_id}})"
     runs-on: ubuntu-latest
     timeout-minutes: 90
     env:
@@ -263,6 +307,110 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}
             ${{ runner.os }}-dubbo-snapshot-
+      - name: "Download test list"
+        uses: actions/download-artifact@v3
+        with:
+          name: samples-test-list
+          path: test/jobs/
+      - name: "Set up JDK 8"
+        uses: actions/setup-java@v3
+        with:
+          distribution: 'zulu'
+          java-version: 8
+      - name: "Init Candidate Versions"
+        run: |
+          DUBBO_VERSION="${{needs.build-source.outputs.version}}"
+          CANDIDATE_VERSIONS="dubbo.version:$DUBBO_VERSION;compiler.version:$DUBBO_VERSION;$CANDIDATE_VERSIONS;dubbo.compiler.version:$DUBBO_VERSION"
+          echo "CANDIDATE_VERSIONS=$CANDIDATE_VERSIONS" >> $GITHUB_ENV
+      - name: "Build test image"
+        run: |
+          cd test && bash ./build-test-image.sh
+      - name: "Run tests"
+        run: cd test && bash ./run-tests.sh
+      - name: "merge jacoco resule"
+        run: |
+          cd test/dubbo-test-jacoco-merger && mvn clean compile exec:java -Dexec.mainClass="org.apache.dubbo.test.JacocoMerge" -Dexec.args="${{github.workspace}}"
+      - name: "Upload jacoco"
+        uses: actions/upload-artifact@v3
+        with:
+          name: samples-jacoco-result
+          path: target/jacoco*.exec
+      - name: "Upload test result"
+        if: always()
+        uses: actions/upload-artifact@v3
+        with:
+          name: samples-test-result
+          path: test/jobs/*-result*
+  samples-test-result:
+    needs: [check-format, samples-test-job]
+    if: always()
+    runs-on: ubuntu-latest
+    env:
+      JAVA_VER: 8
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-samples'
+          ref: master
+      - name: "Download test result"
+        uses: actions/download-artifact@v3
+        with:
+          name: samples-test-result
+          path: test/jobs/
+      - name: "Merge test result"
+        run: ./test/scripts/merge-test-results.sh
+
+  integration-test-prepare:
+    needs: check-format
+    runs-on: ubuntu-latest
+    env:
+      JOB_COUNT: 3
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          ref: main
+      - name: "Prepare test list"
+        run: |
+          bash ./test/scripts/prepare-test.sh
+      - name: "Upload test list"
+        uses: actions/upload-artifact@v3
+        with:
+          name: test-list
+          path: test/jobs
+  integration-test-job:
+    needs: [check-format, build-source, integration-test-prepare]
+    name: "Integration Test on ubuntu-latest (JobId: ${{matrix.job_id}})"
+    runs-on: ubuntu-latest
+    timeout-minutes: 90
+    env:
+      JAVA_VER: 8
+      TEST_CASE_FILE: jobs/testjob_${{matrix.job_id}}.txt
+    strategy:
+      fail-fast: false
+      matrix:
+        job_id: [1, 2, 3]
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          ref: main
+      - name: "Cache local Maven repository"
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2/repository
+          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
+            ${{ runner.os }}-maven-
+      - name: "Restore Dubbo cache"
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2/repository/org/apache/dubbo
+          key: ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}
+            ${{ runner.os }}-dubbo-snapshot-
       - name: "Download test list"
         uses: actions/download-artifact@v3
         with:
@@ -297,9 +445,8 @@ jobs:
         with:
           name: test-result
           path: test/jobs/*-result*
-
   integration-test-result:
-    needs: [integration-test-job]
+    needs: [check-format, integration-test-job]
     if: always()
     runs-on: ubuntu-latest
     env:
@@ -307,8 +454,8 @@ jobs:
     steps:
       - uses: actions/checkout@v3
         with:
-          repository: 'apache/dubbo-samples'
+          repository: 'apache/dubbo-integration-cases'
-          ref: master
+          ref: main
       - name: "Download test result"
         uses: actions/download-artifact@v3
         with:
@@ -319,7 +466,7 @@ jobs:
 
   jacoco-result-merge:
     runs-on: ubuntu-latest
-    needs: [integration-test-result, unit-test]
+    needs: [check-format, integration-test-result, samples-test-result, unit-test]
     steps:
       - uses: actions/checkout@v3
         with:
@@ -328,11 +475,15 @@ jobs:
         with:
           repository: 'apache/dubbo-samples'
           path: "./dubbo-samples"
-      - name: "Set up JDK 17"
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          path: "./dubbo-integration-cases"
+      - name: "Set up JDK 21"
         uses: actions/setup-java@v3
         with:
           distribution: 'zulu'
-          java-version: 17
+          java-version: 21
       - name: "Restore class result"
         uses: actions/download-artifact@v3
         with:
@@ -345,12 +496,21 @@ jobs:
       - name: "Restore jacoco exec"
         uses: actions/download-artifact@v3
         with:
-          name: jacoco-result
+          name: samples-jacoco-result
           path: dubbo-samples/target/
+      - name: "Restore integration jacoco exec"
+        uses: actions/download-artifact@v3
+        with:
+          name: jacoco-result
+          path: dubbo-integration-cases/target/
       - name: "Merge jacoco result"
         run: |
           cd ${{ github.workspace }}/dubbo-samples/test/dubbo-test-jacoco-merger
           mvn clean compile exec:java -Dexec.mainClass="org.apache.dubbo.test.JacocoReport" -Dexec.args="${{github.workspace}}/dubbo-samples ${{github.workspace}}/dubbo"
+      - name: "Merge samples jacoco result"
+        run: |
+          cd ${{ github.workspace }}/dubbo-integration-cases/test/dubbo-test-jacoco-merger
+          mvn clean compile exec:java -Dexec.mainClass="org.apache.dubbo.test.JacocoReport" -Dexec.args="${{github.workspace}}/dubbo-integration-cases ${{github.workspace}}/dubbo"
       - name: "Restore coverage result"
         uses: actions/download-artifact@v3
         with:
@@ -358,8 +518,11 @@ jobs:
           path: dubbo/
       - name: "Upload coverage to Codecov"
         uses: codecov/codecov-action@v3
+        with:
+          verbose: true
 
   error-code-inspecting:
+    needs: check-format
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
@@ -372,11 +535,11 @@ jobs:
           ref: main
           path: "./dubbo-test-tools"
 
-      - name: "Set up JDK 17"
+      - name: "Set up JDK 21"
         uses: actions/setup-java@v3
         with:
           distribution: 'zulu'
-          java-version: 17
+          java-version: 21
 
       - name: "Compile Dubbo (Linux)"
         run: |
@@ -398,6 +561,7 @@ jobs:
           path: ${{ github.workspace }}/dubbo-test-tools/dubbo-error-code-inspector/error-inspection-result.txt
 
   native-image-inspecting:
+    needs: check-format
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v3
@@ -415,10 +579,10 @@ jobs:
 
       - name: "Setup Zookeeper environment"
         run: |
-          wget https://dlcdn.apache.org/zookeeper/zookeeper-3.7.1/apache-zookeeper-3.7.1-bin.tar.gz
+          wget https://dlcdn.apache.org/zookeeper/zookeeper-3.8.4/apache-zookeeper-3.8.4-bin.tar.gz
-          tar -zxvf apache-zookeeper-3.7.1-bin.tar.gz
+          tar -zxvf apache-zookeeper-3.8.4-bin.tar.gz
-          mv apache-zookeeper-3.7.1-bin/conf/zoo_sample.cfg apache-zookeeper-3.7.1-bin/conf/zoo.cfg
+          mv apache-zookeeper-3.8.4-bin/conf/zoo_sample.cfg apache-zookeeper-3.8.4-bin/conf/zoo.cfg
-          apache-zookeeper-3.7.1-bin/bin/zkServer.sh start
+          apache-zookeeper-3.8.4-bin/bin/zkServer.sh start
 
       - name: "Check environment"
         run: |

.github/workflows/build-and-test-scheduled-3.0.yml

@@ -1,251 +0,0 @@
-name: Build and Test Scheduled On 3.0
-
-on:
-  schedule:
-    - cron: '0 0/6 * * *'
-  workflow_dispatch:
-
-permissions:
-  contents: read
-
-env:
-  FORK_COUNT: 2
-  FAIL_FAST: 0
-  SHOW_ERROR_DETAIL: 1
-  #multi-version size limit
-  VERSIONS_LIMIT: 4
-  CANDIDATE_VERSIONS: '
-    spring.version:4.3.30.RELEASE;
-    spring-boot.version:1.5.22.RELEASE;
-    spring-boot.version:2.4.1;
-    '
-
-jobs:
-  build-source:
-    runs-on: ubuntu-latest
-    outputs:
-      version: ${{ steps.dubbo-version.outputs.version }}
-    steps:
-      - uses: actions/checkout@v3
-        with:
-          ref: "3.0"
-          path: dubbo
-      - uses: actions/setup-java@v3
-        with:
-          distribution: 'zulu'
-          java-version: 8
-      - uses: actions/cache@v3
-        name: "Cache local Maven repository"
-        with:
-          path: ~/.m2/repository
-          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}-${{ github.run_id }}
-          restore-keys: |
-            ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
-            ${{ runner.os }}-maven-
-      - name: "Dubbo cache"
-        uses: actions/cache@v3
-        with:
-          path: ~/.m2/repository/org/apache/dubbo
-          key: ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}-${{ github.run_id }}
-      - name: "Build Dubbo with Maven"
-        run: |
-          cd ./dubbo
-          ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean source:jar install -Pjacoco,checkstyle -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.count=5 -Dmaven.test.skip=true -Dmaven.test.skip.exec=true -DembeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper
-      - name: "Pack checkstyle file if failure"
-        if: failure()
-        run: 7z a ${{ github.workspace }}/checkstyle.zip *checkstyle* -r
-      - name: "Upload checkstyle file if failure"
-        if: failure()
-        uses: actions/upload-artifact@v3
-        with:
-          name: "checkstyle-file"
-          path: ${{ github.workspace }}/checkstyle.zip
-      - name: "Calculate Dubbo Version"
-        id: dubbo-version
-        run: |
-          REVISION=`awk '/<revision>[^<]+<\/revision>/{gsub(/<revision>|<\/revision>/,"",$1);print $1;exit;}' ./dubbo/pom.xml`
-          echo "version=$REVISION" >> $GITHUB_OUTPUT
-          echo "dubbo version: $REVISION"
-
-  unit-test-prepare:
-    name: " Preparation for Unit Test On ${{ matrix.os }}"
-    runs-on: ${{ matrix.os }}
-    strategy:
-      fail-fast: false
-      matrix:
-        os: [ ubuntu-latest, windows-latest ]
-    env:
-      ZOOKEEPER_VERSION: 3.6.3
-    steps:
-      - uses: actions/cache@v3
-        name: "Cache zookeeper binary archive"
-        id: "cache-zookeeper"
-        with:
-          path: ${{ github.workspace }}/.tmp/zookeeper
-          key: zookeeper-${{ runner.os }}-${{ env.ZOOKEEPER_VERSION }}
-          restore-keys: |
-            zookeeper-${{ runner.os }}-${{ env.ZOOKEEPER_VERSION }}
-      - name: "Set up msys2 if necessary"
-        if: ${{ startsWith( matrix.os, 'windows') && steps.cache-zookeeper.outputs.cache-hit != 'true' }}
-        uses: msys2/setup-msys2@v2
-        with:
-          release: false  # support cache, see https://github.com/msys2/setup-msys2#context
-      - name: "Download zookeeper binary archive in Linux OS"
-        if: ${{ startsWith( matrix.os, 'ubuntu') && steps.cache-zookeeper.outputs.cache-hit != 'true' }}
-        run: |
-          mkdir -p ${{ github.workspace }}/.tmp/zookeeper
-          wget -c https://archive.apache.org/dist/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
-          wget -c https://apache.website-solution.net/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
-          wget -c http://apache.stu.edu.tw/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
-          wget -c http://ftp.jaist.ac.jp/pub/apache/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
-          wget -c http://apache.mirror.cdnetworks.com/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
-          wget -c http://mirror.apache-kr.org/apache/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz
-          echo "list the downloaded zookeeper binary archive"
-          ls -al ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz
-      - name: "Download zookeeper binary archive in Windows OS"
-        if: ${{ startsWith( matrix.os, 'windows') && steps.cache-zookeeper.outputs.cache-hit != 'true' }}
-        shell: msys2 {0}
-        run: |
-          mkdir -p ${{ github.workspace }}/.tmp/zookeeper
-          wget -c https://archive.apache.org/dist/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
-          wget -c https://apache.website-solution.net/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
-          wget -c http://apache.stu.edu.tw/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
-          wget -c http://ftp.jaist.ac.jp/pub/apache/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
-          wget -c http://apache.mirror.cdnetworks.com/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
-          wget -c http://mirror.apache-kr.org/apache/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz
-          echo "list the downloaded zookeeper binary archive"
-          ls -al ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz
-
-  unit-test:
-    needs: [build-source, unit-test-prepare]
-    name: "Unit Test On ${{ matrix.os }} (JDK: ${{ matrix.jdk }})"
-    runs-on: ${{ matrix.os }}
-    strategy:
-      fail-fast: false
-      matrix:
-        os: [ ubuntu-latest, windows-latest ]
-        jdk: [ 8, 11, 17 ]
-    env:
-      DISABLE_FILE_SYSTEM_TEST: true
-    steps:
-      - uses: actions/checkout@v3
-        with:
-          ref: "3.0"
-      - name: "Set up JDK ${{ matrix.jdk }}"
-        uses: actions/setup-java@v1
-        with:
-          java-version: ${{ matrix.jdk }}
-      - uses: actions/cache@v3
-        name: "Cache local Maven repository"
-        with:
-          path: ~/.m2/repository
-          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}-${{ github.run_id }}
-          restore-keys: |
-            ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
-            ${{ runner.os }}-maven-
-      - name: "Test with Maven with Integration Tests"
-        timeout-minutes: 70
-        if: ${{ startsWith( matrix.os, 'ubuntu') }}
-        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.count=5 -DskipTests=false -DskipIntegrationTests=false -Dcheckstyle.skip=false -Dcheckstyle_unix.skip=false -Drat.skip=false -Dmaven.javadoc.skip=true -DembeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper
-      - name: "Test with Maven without Integration Tests"
-        timeout-minutes: 90
-        if: ${{ startsWith( matrix.os, 'windows') }}
-        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
-      - name: "Upload coverage to Codecov"
-        uses: codecov/codecov-action@v3
-
-  integration-test-prepare:
-    runs-on: ubuntu-latest
-    env:
-      JOB_COUNT: 3
-    steps:
-      - uses: actions/checkout@v3
-        with:
-          repository: 'apache/dubbo-samples'
-          ref: master
-      - name: "Prepare test list"
-        run: |
-          bash ./test/scripts/prepare-test.sh
-      - name: "Upload test list"
-        uses: actions/upload-artifact@v3
-        with:
-          name: test-list
-          path: test/jobs
-
-  integration-test-job:
-    needs: [build-source, integration-test-prepare]
-    name: "Integration Test on ubuntu-latest (JobId: ${{matrix.job_id}})"
-    runs-on: ubuntu-latest
-    timeout-minutes: 30
-    env:
-      JAVA_VER: 8
-      TEST_CASE_FILE: jobs/testjob_${{matrix.job_id}}.txt
-    strategy:
-      fail-fast: false
-      matrix:
-        job_id: [1, 2, 3]
-    steps:
-      - uses: actions/checkout@v3
-        with:
-          repository: 'apache/dubbo-samples'
-          ref: master
-      - name: "Cache local Maven repository"
-        uses: actions/cache@v3
-        with:
-          path: ~/.m2/repository
-          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}-${{ github.run_id }}
-          restore-keys: |
-            ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
-            ${{ runner.os }}-maven-
-      - name: "Restore Dubbo cache"
-        uses: actions/cache@v3
-        with:
-          path: ~/.m2/repository/org/apache/dubbo
-          key: ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}-${{ github.run_id }}
-          restore-keys: |
-            ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}
-            ${{ runner.os }}-dubbo-snapshot-
-      - name: "Download test list"
-        uses: actions/download-artifact@v3
-        with:
-          name: test-list
-          path: test/jobs/
-      - name: "Set up JDK 8"
-        uses: actions/setup-java@v1
-        with:
-          java-version: 8
-      - name: "Init Candidate Versions"
-        run: |
-          DUBBO_VERSION="${{needs.build-source.outputs.version}}"
-          CANDIDATE_VERSIONS="dubbo.version:$DUBBO_VERSION;compiler.version:$DUBBO_VERSION;$CANDIDATE_VERSIONS;dubbo.compiler.version:$DUBBO_VERSION"
-          echo "CANDIDATE_VERSIONS=$CANDIDATE_VERSIONS" >> $GITHUB_ENV
-      - name: "Build test image"
-        run: |
-          cd test && bash ./build-test-image.sh
-      - name: "Run tests"
-        run: cd test && bash ./run-tests.sh
-      - name: "Upload test result"
-        if: always()
-        uses: actions/upload-artifact@v3
-        with:
-          name: test-result
-          path: test/jobs/*-result*
-
-  integration-test-result:
-    needs: [integration-test-job]
-    if: always()
-    runs-on: ubuntu-latest
-    env:
-      JAVA_VER: 8
-    steps:
-      - uses: actions/checkout@v3
-        with:
-          repository: 'apache/dubbo-samples'
-          ref: master
-      - name: "Download test result"
-        uses: actions/download-artifact@v3
-        with:
-          name: test-result
-          path: test/jobs/
-      - name: "Merge test result"
-        run: ./test/scripts/merge-test-results.sh

.github/workflows/build-and-test-scheduled-3.1.yml

@@ -134,7 +134,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [ ubuntu-latest, windows-latest ]
-        jdk: [ 8, 11, 17, 19 ]
+        jdk: [ 8, 11, 17, 21 ]
     env:
       DISABLE_FILE_SYSTEM_TEST: true
     steps:
@@ -169,8 +169,6 @@ jobs:
         timeout-minutes: 90
         if: ${{ startsWith( matrix.os, 'windows') }}
         run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
-      - name: "Upload coverage to Codecov"
-        uses: codecov/codecov-action@v3
 
   unit-test-fastjson2:
     needs: [build-source, unit-test-prepare]
@@ -180,18 +178,17 @@ jobs:
       fail-fast: false
       matrix:
         os: [ ubuntu-latest, windows-latest ]
-        jdk: [ 8, 11, 17, 19 ]
+        jdk: [ 8, 11, 17, 21 ]
     env:
       DISABLE_FILE_SYSTEM_TEST: true
       DUBBO_DEFAULT_SERIALIZATION: fastjson2
       MAVEN_SUREFIRE_ADD_OPENS: true
     steps:
       - uses: actions/checkout@v3
-        with:
-          ref: "3.1"
       - name: "Set up JDK ${{ matrix.jdk }}"
-        uses: actions/setup-java@v1
+        uses: actions/setup-java@v3
         with:
+          distribution: 'zulu'
           java-version: ${{ matrix.jdk }}
       - uses: actions/cache@v3
         name: "Cache local Maven repository"
@@ -209,18 +206,24 @@ jobs:
           key: zookeeper-${{ runner.os }}-${{ env.ZOOKEEPER_VERSION }}
           restore-keys: |
             zookeeper-${{ runner.os }}-
+      - name: "Test with Maven with Integration Tests on JDK 8"
+        timeout-minutes: 70
+        if: ${{ startsWith( matrix.os, 'ubuntu') && matrix.jdk == '8' }}
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco,'!jdk15ge' -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.count=5 -DskipTests=false -DskipIntegrationTests=false -Dcheckstyle.skip=false -Dcheckstyle_unix.skip=false -Drat.skip=false -Dmaven.javadoc.skip=true -DembeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper
+      - name: "Test with Maven without Integration Tests on JDK 8"
+        timeout-minutes: 90
+        if: ${{ startsWith( matrix.os, 'windows') && matrix.jdk == '8' }}
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -P"jacoco,'!jdk15ge'" -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
       - name: "Test with Maven with Integration Tests"
         timeout-minutes: 70
-        if: ${{ startsWith( matrix.os, 'ubuntu') }}
+        if: ${{ startsWith( matrix.os, 'ubuntu') && matrix.jdk != '8' }}
-        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.count=5 -DskipTests=false -DskipIntegrationTests=false -Dcheckstyle.skip=false -Dcheckstyle_unix.skip=false -Drat.skip=false -Dmaven.javadoc.skip=true -DembeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco,jdk15ge-simple,'!jdk15ge' -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.count=5 -DskipTests=false -DskipIntegrationTests=false -Dcheckstyle.skip=false -Dcheckstyle_unix.skip=false -Drat.skip=false -Dmaven.javadoc.skip=true -DembeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper
       - name: "Test with Maven without Integration Tests"
         timeout-minutes: 90
-        if: ${{ startsWith( matrix.os, 'windows') }}
+        if: ${{ startsWith( matrix.os, 'windows') && matrix.jdk != '8' }}
-        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -P"jacoco,jdk15ge-simple,'!jdk15ge'" -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
-      - name: "Upload coverage to Codecov"
-        uses: codecov/codecov-action@v3
 
-  integration-test-prepare:
+  samples-test-prepare:
     runs-on: ubuntu-latest
     env:
       JOB_COUNT: 5
@@ -235,12 +238,11 @@ jobs:
       - name: "Upload test list"
         uses: actions/upload-artifact@v3
         with:
-          name: test-list
+          name: samples-test-list
           path: test/jobs
-
+  samples-test-job:
-  integration-test-job:
+    needs: [build-source, samples-test-prepare]
-    needs: [build-source, integration-test-prepare]
+    name: "Samples Test on ubuntu-latest (JobId: ${{matrix.job_id}} JavaVer: ${{matrix.jdk}})"
-    name: "Integration Test on ubuntu-latest (JobId: ${{matrix.job_id}})"
     runs-on: ubuntu-latest
     timeout-minutes: 90
     env:
@@ -275,7 +277,7 @@ jobs:
       - name: "Download test list"
         uses: actions/download-artifact@v3
         with:
-          name: test-list
+          name: samples-test-list
           path: test/jobs/
       - name: "Set up JDK ${{matrix.jdk}}"
         uses: actions/setup-java@v1
@@ -295,11 +297,10 @@ jobs:
         if: always()
         uses: actions/upload-artifact@v3
         with:
-          name: test-result-${{matrix.jdk}}
+          name: samples-test-result-${{matrix.jdk}}
           path: test/jobs/*-result*
-
+  samples-test-result:
-  integration-test-result:
+    needs: [samples-test-job]
-    needs: [integration-test-job]
     if: always()
     runs-on: ubuntu-latest
     env:
@@ -315,7 +316,105 @@ jobs:
       - name: "Download test result"
         uses: actions/download-artifact@v3
         with:
-          name: test-result-${{matrix.jdk}}
+          name: samples-test-result-${{matrix.jdk}}
+          path: test/jobs/
+      - name: "Merge test result"
+        run: ./test/scripts/merge-test-results.sh
+
+  integration-test-prepare:
+    runs-on: ubuntu-latest
+    env:
+      JOB_COUNT: 5
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          ref: main
+      - name: "Prepare test list"
+        run: |
+          bash ./test/scripts/prepare-test.sh
+      - name: "Upload test list"
+        uses: actions/upload-artifact@v3
+        with:
+          name: integration-test-list
+          path: test/jobs
+  integration-test-job:
+    needs: [build-source, integration-test-prepare]
+    name: "Integration Test on ubuntu-latest (JobId: ${{matrix.job_id}} JavaVer: ${{matrix.jdk}})"
+    runs-on: ubuntu-latest
+    timeout-minutes: 90
+    env:
+      JAVA_VER: ${{matrix.jdk}}
+      TEST_CASE_FILE: jobs/testjob_${{matrix.job_id}}.txt
+    strategy:
+      fail-fast: false
+      matrix:
+        jdk: [ 8, 11 ]
+        job_id: [1, 2, 3, 4, 5]
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          ref: main
+      - name: "Cache local Maven repository"
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2/repository
+          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
+            ${{ runner.os }}-maven-
+      - name: "Restore Dubbo cache"
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2/repository/org/apache/dubbo
+          key: ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}
+            ${{ runner.os }}-dubbo-snapshot-
+      - name: "Download test list"
+        uses: actions/download-artifact@v3
+        with:
+          name: integration-test-list
+          path: test/jobs/
+      - name: "Set up JDK ${{matrix.jdk}}"
+        uses: actions/setup-java@v1
+        with:
+          java-version: ${{matrix.jdk}}
+      - name: "Init Candidate Versions"
+        run: |
+          DUBBO_VERSION="${{needs.build-source.outputs.version}}"
+          CANDIDATE_VERSIONS="dubbo.version:$DUBBO_VERSION;compiler.version:$DUBBO_VERSION;$CANDIDATE_VERSIONS;dubbo.compiler.version:$DUBBO_VERSION"
+          echo "CANDIDATE_VERSIONS=$CANDIDATE_VERSIONS" >> $GITHUB_ENV
+      - name: "Build test image"
+        run: |
+          cd test && bash ./build-test-image.sh
+      - name: "Run tests"
+        run: cd test && bash ./run-tests.sh
+      - name: "Upload test result"
+        if: always()
+        uses: actions/upload-artifact@v3
+        with:
+          name: integration-test-result-${{matrix.jdk}}
+          path: test/jobs/*-result*
+  integration-test-result:
+    needs: [integration-test-job]
+    if: always()
+    runs-on: ubuntu-latest
+    env:
+      JAVA_VER: ${{matrix.jdk}}
+    strategy:
+      matrix:
+        jdk: [ 8, 11 ]
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          ref: main
+      - name: "Download test result"
+        uses: actions/download-artifact@v3
+        with:
+          name: integration-test-result-${{matrix.jdk}}
           path: test/jobs/
       - name: "Merge test result"
         run: ./test/scripts/merge-test-results.sh
@@ -334,10 +433,11 @@ jobs:
           ref: main
           path: "./dubbo-test-tools"
 
-      - name: "Set up JDK 17"
+      - name: "Set up JDK 21"
-        uses: actions/setup-java@v1
+        uses: actions/setup-java@v3
         with:
-          java-version: 17
+          distribution: 'zulu'
+          java-version: 21
 
       - name: "Compile Dubbo (Linux)"
         run: |
@@ -377,10 +477,10 @@ jobs:
 
       - name: "Setup Zookeeper environment"
         run: |
-          wget https://dlcdn.apache.org/zookeeper/zookeeper-3.7.1/apache-zookeeper-3.7.1-bin.tar.gz
+          wget https://dlcdn.apache.org/zookeeper/zookeeper-3.8.4/apache-zookeeper-3.8.4-bin.tar.gz
-          tar -zxvf apache-zookeeper-3.7.1-bin.tar.gz
+          tar -zxvf apache-zookeeper-3.8.4-bin.tar.gz
-          mv apache-zookeeper-3.7.1-bin/conf/zoo_sample.cfg apache-zookeeper-3.7.1-bin/conf/zoo.cfg
+          mv apache-zookeeper-3.8.4-bin/conf/zoo_sample.cfg apache-zookeeper-3.8.4-bin/conf/zoo.cfg
-          apache-zookeeper-3.7.1-bin/bin/zkServer.sh start
+          apache-zookeeper-3.8.4-bin/bin/zkServer.sh start
 
       - name: "Check environment"
         run: |
@@ -396,7 +496,7 @@ jobs:
         run: |
           cd ${{ github.workspace }}/dubbo/dubbo-demo/dubbo-demo-native/dubbo-demo-native-provider
           ${{ github.workspace }}/dubbo/mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast -T 2C clean package -P native -Dmaven.test.skip=true native:compile
-          nohup ./target/demo-native-provider &
+          nohup ./target/dubbo-demo-native-provider &
           cd ${{ github.workspace }}/dubbo/dubbo-demo/dubbo-demo-native/dubbo-demo-native-consumer
           ${{ github.workspace }}/dubbo/mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast -T 2C clean package -P native -Dmaven.test.skip=true native:compile
-          ./target/demo-native-consumer
+          ./target/dubbo-demo-native-consumer

.github/workflows/build-and-test-scheduled-3.2.yml

@@ -134,7 +134,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [ ubuntu-latest, windows-latest ]
-        jdk: [ 8, 11, 17, 19 ]
+        jdk: [ 8, 11, 17, 21 ]
     env:
       DISABLE_FILE_SYSTEM_TEST: true
     steps:
@@ -169,8 +169,6 @@ jobs:
         timeout-minutes: 90
         if: ${{ startsWith( matrix.os, 'windows') }}
         run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
-      - name: "Upload coverage to Codecov"
-        uses: codecov/codecov-action@v3
 
   unit-test-fastjson2:
     needs: [build-source, unit-test-prepare]
@@ -180,18 +178,17 @@ jobs:
       fail-fast: false
       matrix:
         os: [ ubuntu-latest, windows-latest ]
-        jdk: [ 8, 11, 17, 19 ]
+        jdk: [ 8, 11, 17, 21 ]
     env:
       DISABLE_FILE_SYSTEM_TEST: true
       DUBBO_DEFAULT_SERIALIZATION: fastjson2
       MAVEN_SUREFIRE_ADD_OPENS: true
     steps:
       - uses: actions/checkout@v3
-        with:
-          ref: "3.2"
       - name: "Set up JDK ${{ matrix.jdk }}"
-        uses: actions/setup-java@v1
+        uses: actions/setup-java@v3
         with:
+          distribution: 'zulu'
           java-version: ${{ matrix.jdk }}
       - uses: actions/cache@v3
         name: "Cache local Maven repository"
@@ -209,18 +206,24 @@ jobs:
           key: zookeeper-${{ runner.os }}-${{ env.ZOOKEEPER_VERSION }}
           restore-keys: |
             zookeeper-${{ runner.os }}-
+      - name: "Test with Maven with Integration Tests on JDK 8"
+        timeout-minutes: 70
+        if: ${{ startsWith( matrix.os, 'ubuntu') && matrix.jdk == '8' }}
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco,'!jdk15ge' -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.count=5 -DskipTests=false -DskipIntegrationTests=false -Dcheckstyle.skip=false -Dcheckstyle_unix.skip=false -Drat.skip=false -Dmaven.javadoc.skip=true -DembeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper
+      - name: "Test with Maven without Integration Tests on JDK 8"
+        timeout-minutes: 90
+        if: ${{ startsWith( matrix.os, 'windows') && matrix.jdk == '8' }}
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -P"jacoco,'!jdk15ge'" -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
       - name: "Test with Maven with Integration Tests"
         timeout-minutes: 70
-        if: ${{ startsWith( matrix.os, 'ubuntu') }}
+        if: ${{ startsWith( matrix.os, 'ubuntu') && matrix.jdk != '8' }}
         run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco,jdk15ge-simple,'!jdk15ge' -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.count=5 -DskipTests=false -DskipIntegrationTests=false -Dcheckstyle.skip=false -Dcheckstyle_unix.skip=false -Drat.skip=false -Dmaven.javadoc.skip=true -DembeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper
       - name: "Test with Maven without Integration Tests"
         timeout-minutes: 90
-        if: ${{ startsWith( matrix.os, 'windows') }}
+        if: ${{ startsWith( matrix.os, 'windows') && matrix.jdk != '8' }}
         run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -P"jacoco,jdk15ge-simple,'!jdk15ge'" -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
-      - name: "Upload coverage to Codecov"
-        uses: codecov/codecov-action@v3
 
-  integration-test-prepare:
+  samples-test-prepare:
     runs-on: ubuntu-latest
     env:
       JOB_COUNT: 5
@@ -235,12 +238,11 @@ jobs:
       - name: "Upload test list"
         uses: actions/upload-artifact@v3
         with:
-          name: test-list
+          name: samples-test-list
           path: test/jobs
-
+  samples-test-job:
-  integration-test-job:
+    needs: [build-source, samples-test-prepare]
-    needs: [build-source, integration-test-prepare]
+    name: "Samples Test on ubuntu-latest (JobId: ${{matrix.job_id}} JavaVer: ${{matrix.jdk}})"
-    name: "Integration Test on ubuntu-latest (JobId: ${{matrix.job_id}})"
     runs-on: ubuntu-latest
     timeout-minutes: 90
     env:
@@ -249,7 +251,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        jdk: [ 8, 11, 17, 19 ]
+        jdk: [ 8, 11, 17, 21 ]
         job_id: [1, 2, 3, 4, 5]
     steps:
       - uses: actions/checkout@v3
@@ -275,7 +277,7 @@ jobs:
       - name: "Download test list"
         uses: actions/download-artifact@v3
         with:
-          name: test-list
+          name: samples-test-list
           path: test/jobs/
       - name: "Set up JDK ${{matrix.jdk}}"
         uses: actions/setup-java@v1
@@ -295,18 +297,17 @@ jobs:
         if: always()
         uses: actions/upload-artifact@v3
         with:
-          name: test-result-${{matrix.jdk}}
+          name: samples-test-result-${{matrix.jdk}}
           path: test/jobs/*-result*
-
+  samples-test-result:
-  integration-test-result:
+    needs: [samples-test-job]
-    needs: [integration-test-job]
     if: always()
     runs-on: ubuntu-latest
     env:
       JAVA_VER: ${{matrix.jdk}}
     strategy:
       matrix:
-        jdk: [ 8, 11, 17, 19 ]
+        jdk: [ 8, 11, 17, 21 ]
     steps:
       - uses: actions/checkout@v3
         with:
@@ -315,7 +316,105 @@ jobs:
       - name: "Download test result"
         uses: actions/download-artifact@v3
         with:
-          name: test-result-${{matrix.jdk}}
+          name: samples-test-result-${{matrix.jdk}}
+          path: test/jobs/
+      - name: "Merge test result"
+        run: ./test/scripts/merge-test-results.sh
+
+  integration-test-prepare:
+    runs-on: ubuntu-latest
+    env:
+      JOB_COUNT: 5
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          ref: main
+      - name: "Prepare test list"
+        run: |
+          bash ./test/scripts/prepare-test.sh
+      - name: "Upload test list"
+        uses: actions/upload-artifact@v3
+        with:
+          name: integration-test-list
+          path: test/jobs
+  integration-test-job:
+    needs: [build-source, integration-test-prepare]
+    name: "Integration Test on ubuntu-latest (JobId: ${{matrix.job_id}} JavaVer: ${{matrix.jdk}})"
+    runs-on: ubuntu-latest
+    timeout-minutes: 90
+    env:
+      JAVA_VER: ${{matrix.jdk}}
+      TEST_CASE_FILE: jobs/testjob_${{matrix.job_id}}.txt
+    strategy:
+      fail-fast: false
+      matrix:
+        jdk: [ 8, 11, 17, 21 ]
+        job_id: [1, 2, 3, 4, 5]
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          ref: main
+      - name: "Cache local Maven repository"
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2/repository
+          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
+            ${{ runner.os }}-maven-
+      - name: "Restore Dubbo cache"
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2/repository/org/apache/dubbo
+          key: ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}
+            ${{ runner.os }}-dubbo-snapshot-
+      - name: "Download test list"
+        uses: actions/download-artifact@v3
+        with:
+          name: integration-test-list
+          path: test/jobs/
+      - name: "Set up JDK ${{matrix.jdk}}"
+        uses: actions/setup-java@v1
+        with:
+          java-version: ${{matrix.jdk}}
+      - name: "Init Candidate Versions"
+        run: |
+          DUBBO_VERSION="${{needs.build-source.outputs.version}}"
+          CANDIDATE_VERSIONS="dubbo.version:$DUBBO_VERSION;compiler.version:$DUBBO_VERSION;$CANDIDATE_VERSIONS;dubbo.compiler.version:$DUBBO_VERSION"
+          echo "CANDIDATE_VERSIONS=$CANDIDATE_VERSIONS" >> $GITHUB_ENV
+      - name: "Build test image"
+        run: |
+          cd test && bash ./build-test-image.sh
+      - name: "Run tests"
+        run: cd test && bash ./run-tests.sh
+      - name: "Upload test result"
+        if: always()
+        uses: actions/upload-artifact@v3
+        with:
+          name: integration-test-result-${{matrix.jdk}}
+          path: test/jobs/*-result*
+  integration-test-result:
+    needs: [integration-test-job]
+    if: always()
+    runs-on: ubuntu-latest
+    env:
+      JAVA_VER: ${{matrix.jdk}}
+    strategy:
+      matrix:
+        jdk: [ 8, 11, 17, 21 ]
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          ref: main
+      - name: "Download test result"
+        uses: actions/download-artifact@v3
+        with:
+          name: integration-test-result-${{matrix.jdk}}
           path: test/jobs/
       - name: "Merge test result"
         run: ./test/scripts/merge-test-results.sh
@@ -334,10 +433,11 @@ jobs:
           ref: main
           path: "./dubbo-test-tools"
 
-      - name: "Set up JDK 17"
+      - name: "Set up JDK 21"
-        uses: actions/setup-java@v1
+        uses: actions/setup-java@v3
         with:
-          java-version: 17
+          distribution: 'zulu'
+          java-version: 21
 
       - name: "Compile Dubbo (Linux)"
         run: |
@@ -377,10 +477,10 @@ jobs:
 
       - name: "Setup Zookeeper environment"
         run: |
-          wget https://dlcdn.apache.org/zookeeper/zookeeper-3.7.1/apache-zookeeper-3.7.1-bin.tar.gz
+          wget https://dlcdn.apache.org/zookeeper/zookeeper-3.8.4/apache-zookeeper-3.8.4-bin.tar.gz
-          tar -zxvf apache-zookeeper-3.7.1-bin.tar.gz
+          tar -zxvf apache-zookeeper-3.8.4-bin.tar.gz
-          mv apache-zookeeper-3.7.1-bin/conf/zoo_sample.cfg apache-zookeeper-3.7.1-bin/conf/zoo.cfg
+          mv apache-zookeeper-3.8.4-bin/conf/zoo_sample.cfg apache-zookeeper-3.8.4-bin/conf/zoo.cfg
-          apache-zookeeper-3.7.1-bin/bin/zkServer.sh start
+          apache-zookeeper-3.8.4-bin/bin/zkServer.sh start
 
       - name: "Check environment"
         run: |

.github/workflows/build-and-test-scheduled-3.3.yml

@@ -0,0 +1,502 @@
+name: Build and Test Scheduled On 3.3
+
+on:
+  schedule:
+    - cron: '0 0/6 * * *'
+  workflow_dispatch:
+
+permissions:
+  contents: read
+
+env:
+  FORK_COUNT: 2
+  FAIL_FAST: 0
+  SHOW_ERROR_DETAIL: 1
+  #multi-version size limit
+  VERSIONS_LIMIT: 4
+  ALL_REMOTE_VERSION: true
+  CANDIDATE_VERSIONS: '
+    spring.version:5.3.24;
+    spring-boot.version:2.7.6;
+    '
+
+jobs:
+  license:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          ref: "3.3"
+      - name: Check License
+        uses: apache/skywalking-eyes@main
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+  build-source:
+    runs-on: ubuntu-latest
+    outputs:
+      version: ${{ steps.dubbo-version.outputs.version }}
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          ref: "3.3"
+          path: dubbo
+      - uses: actions/setup-java@v3
+        with:
+          distribution: 'zulu'
+          java-version: 21
+      - uses: actions/cache@v3
+        name: "Cache local Maven repository"
+        with:
+          path: ~/.m2/repository
+          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
+            ${{ runner.os }}-maven-
+      - name: "Dubbo cache"
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2/repository/org/apache/dubbo
+          key: ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}-${{ github.run_id }}
+      - name: "Build Dubbo with Maven"
+        run: |
+          cd ./dubbo
+          ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean source:jar install -Pjacoco,checkstyle -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.count=5 -Dmaven.test.skip=true -Dmaven.test.skip.exec=true -DembeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper
+      - name: "Pack checkstyle file if failure"
+        if: failure()
+        run: 7z a ${{ github.workspace }}/checkstyle.zip *checkstyle* -r
+      - name: "Upload checkstyle file if failure"
+        if: failure()
+        uses: actions/upload-artifact@v3
+        with:
+          name: "checkstyle-file"
+          path: ${{ github.workspace }}/checkstyle.zip
+      - name: "Calculate Dubbo Version"
+        id: dubbo-version
+        run: |
+          REVISION=`awk '/<revision>[^<]+<\/revision>/{gsub(/<revision>|<\/revision>/,"",$1);print $1;exit;}' ./dubbo/pom.xml`
+          echo "version=$REVISION" >> $GITHUB_OUTPUT
+          echo "dubbo version: $REVISION"
+
+  unit-test-prepare:
+    name: " Preparation for Unit Test On ${{ matrix.os }}"
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ ubuntu-latest, windows-latest ]
+    env:
+      ZOOKEEPER_VERSION: 3.6.3
+    steps:
+      - uses: actions/cache@v3
+        name: "Cache zookeeper binary archive"
+        id: "cache-zookeeper"
+        with:
+          path: ${{ github.workspace }}/.tmp/zookeeper
+          key: zookeeper-${{ runner.os }}-${{ env.ZOOKEEPER_VERSION }}
+          restore-keys: |
+            zookeeper-${{ runner.os }}-${{ env.ZOOKEEPER_VERSION }}
+      - name: "Set up msys2 if necessary"
+        if: ${{ startsWith( matrix.os, 'windows') && steps.cache-zookeeper.outputs.cache-hit != 'true' }}
+        uses: msys2/setup-msys2@v2
+        with:
+          release: false  # support cache, see https://github.com/msys2/setup-msys2#context
+      - name: "Download zookeeper binary archive in Linux OS"
+        if: ${{ startsWith( matrix.os, 'ubuntu') && steps.cache-zookeeper.outputs.cache-hit != 'true' }}
+        run: |
+          mkdir -p ${{ github.workspace }}/.tmp/zookeeper
+          wget -c https://archive.apache.org/dist/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
+          wget -c https://apache.website-solution.net/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
+          wget -c http://apache.stu.edu.tw/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
+          wget -c http://ftp.jaist.ac.jp/pub/apache/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
+          wget -c http://apache.mirror.cdnetworks.com/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
+          wget -c http://mirror.apache-kr.org/apache/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz
+          echo "list the downloaded zookeeper binary archive"
+          ls -al ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz
+      - name: "Download zookeeper binary archive in Windows OS"
+        if: ${{ startsWith( matrix.os, 'windows') && steps.cache-zookeeper.outputs.cache-hit != 'true' }}
+        shell: msys2 {0}
+        run: |
+          mkdir -p ${{ github.workspace }}/.tmp/zookeeper
+          wget -c https://archive.apache.org/dist/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
+          wget -c https://apache.website-solution.net/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
+          wget -c http://apache.stu.edu.tw/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
+          wget -c http://ftp.jaist.ac.jp/pub/apache/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
+          wget -c http://apache.mirror.cdnetworks.com/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz ||
+          wget -c http://mirror.apache-kr.org/apache/zookeeper/zookeeper-${{ env.ZOOKEEPER_VERSION }}/apache-zookeeper-${{ env.ZOOKEEPER_VERSION }}-bin.tar.gz -O ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz
+          echo "list the downloaded zookeeper binary archive"
+          ls -al ${{ github.workspace }}/.tmp/zookeeper/apache-zookeeper-bin.tar.gz
+
+  unit-test:
+    needs: [build-source, unit-test-prepare]
+    name: "Unit Test On ${{ matrix.os }} (JDK: ${{ matrix.jdk }})"
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ ubuntu-latest, windows-latest ]
+        jdk: [ 8, 11, 17, 21 ]
+    env:
+      DISABLE_FILE_SYSTEM_TEST: true
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          ref: "3.3"
+      - name: "Set up JDK ${{ matrix.jdk }}"
+        uses: actions/setup-java@v1
+        with:
+          java-version: ${{ matrix.jdk }}
+      - uses: actions/cache@v3
+        name: "Cache local Maven repository"
+        with:
+          path: ~/.m2/repository
+          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
+            ${{ runner.os }}-maven-
+      - uses: actions/cache@v3
+        name: "Cache zookeeper binary archive"
+        id: "cache-zookeeper"
+        with:
+          path: ${{ github.workspace }}/.tmp/zookeeper
+          key: zookeeper-${{ runner.os }}-${{ env.ZOOKEEPER_VERSION }}
+          restore-keys: |
+            zookeeper-${{ runner.os }}-
+      - name: "Test with Maven with Integration Tests"
+        timeout-minutes: 70
+        if: ${{ startsWith( matrix.os, 'ubuntu') }}
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.count=5 -DskipTests=false -DskipIntegrationTests=false -Dcheckstyle.skip=false -Dcheckstyle_unix.skip=false -Drat.skip=false -Dmaven.javadoc.skip=true -DembeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper
+      - name: "Test with Maven without Integration Tests"
+        timeout-minutes: 90
+        if: ${{ startsWith( matrix.os, 'windows') }}
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
+
+  unit-test-fastjson2:
+    needs: [build-source, unit-test-prepare]
+    name: "Unit Test On ${{ matrix.os }} (JDK: ${{ matrix.jdk }}, Serialization: fastjson2)"
+    runs-on: ${{ matrix.os }}
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ ubuntu-latest, windows-latest ]
+        jdk: [ 8, 11, 17, 21 ]
+    env:
+      DISABLE_FILE_SYSTEM_TEST: true
+      DUBBO_DEFAULT_SERIALIZATION: fastjson2
+      MAVEN_SUREFIRE_ADD_OPENS: true
+    steps:
+      - uses: actions/checkout@v3
+      - name: "Set up JDK ${{ matrix.jdk }}"
+        uses: actions/setup-java@v3
+        with:
+          distribution: 'zulu'
+          java-version: ${{ matrix.jdk }}
+      - uses: actions/cache@v3
+        name: "Cache local Maven repository"
+        with:
+          path: ~/.m2/repository
+          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
+            ${{ runner.os }}-maven-
+      - uses: actions/cache@v3
+        name: "Cache zookeeper binary archive"
+        id: "cache-zookeeper"
+        with:
+          path: ${{ github.workspace }}/.tmp/zookeeper
+          key: zookeeper-${{ runner.os }}-${{ env.ZOOKEEPER_VERSION }}
+          restore-keys: |
+            zookeeper-${{ runner.os }}-
+      - name: "Test with Maven with Integration Tests on JDK 8"
+        timeout-minutes: 70
+        if: ${{ startsWith( matrix.os, 'ubuntu') && matrix.jdk == '8' }}
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco,'!jdk15ge' -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.count=5 -DskipTests=false -DskipIntegrationTests=false -Dcheckstyle.skip=false -Dcheckstyle_unix.skip=false -Drat.skip=false -Dmaven.javadoc.skip=true -DembeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper
+      - name: "Test with Maven without Integration Tests on JDK 8"
+        timeout-minutes: 90
+        if: ${{ startsWith( matrix.os, 'windows') && matrix.jdk == '8' }}
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -P"jacoco,'!jdk15ge'" -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
+      - name: "Test with Maven with Integration Tests"
+        timeout-minutes: 70
+        if: ${{ startsWith( matrix.os, 'ubuntu') && matrix.jdk != '8' }}
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco,jdk15ge-simple,'!jdk15ge' -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.count=5 -DskipTests=false -DskipIntegrationTests=false -Dcheckstyle.skip=false -Dcheckstyle_unix.skip=false -Drat.skip=false -Dmaven.javadoc.skip=true -DembeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper
+      - name: "Test with Maven without Integration Tests"
+        timeout-minutes: 90
+        if: ${{ startsWith( matrix.os, 'windows') && matrix.jdk != '8' }}
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -P"jacoco,jdk15ge-simple,'!jdk15ge'" -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
+
+  samples-test-prepare:
+    runs-on: ubuntu-latest
+    env:
+      JOB_COUNT: 5
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-samples'
+          ref: master
+      - name: "Prepare test list"
+        run: |
+          bash ./test/scripts/prepare-test.sh
+      - name: "Upload test list"
+        uses: actions/upload-artifact@v3
+        with:
+          name: samples-test-list
+          path: test/jobs
+  samples-test-job:
+    needs: [build-source, samples-test-prepare]
+    name: "Samples Test on ubuntu-latest (JobId: ${{matrix.job_id}} JavaVer: ${{matrix.jdk}})"
+    runs-on: ubuntu-latest
+    timeout-minutes: 90
+    env:
+      JAVA_VER: ${{matrix.jdk}}
+      TEST_CASE_FILE: jobs/testjob_${{matrix.job_id}}.txt
+    strategy:
+      fail-fast: false
+      matrix:
+        jdk: [ 8, 11, 17, 21 ]
+        job_id: [1, 2, 3, 4, 5]
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-samples'
+          ref: master
+      - name: "Cache local Maven repository"
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2/repository
+          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
+            ${{ runner.os }}-maven-
+      - name: "Restore Dubbo cache"
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2/repository/org/apache/dubbo
+          key: ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}
+            ${{ runner.os }}-dubbo-snapshot-
+      - name: "Download test list"
+        uses: actions/download-artifact@v3
+        with:
+          name: samples-test-list
+          path: test/jobs/
+      - name: "Set up JDK ${{matrix.jdk}}"
+        uses: actions/setup-java@v1
+        with:
+          java-version: ${{matrix.jdk}}
+      - name: "Init Candidate Versions"
+        run: |
+          DUBBO_VERSION="${{needs.build-source.outputs.version}}"
+          CANDIDATE_VERSIONS="dubbo.version:$DUBBO_VERSION;compiler.version:$DUBBO_VERSION;$CANDIDATE_VERSIONS;dubbo.compiler.version:$DUBBO_VERSION"
+          echo "CANDIDATE_VERSIONS=$CANDIDATE_VERSIONS" >> $GITHUB_ENV
+      - name: "Build test image"
+        run: |
+          cd test && bash ./build-test-image.sh
+      - name: "Run tests"
+        run: cd test && bash ./run-tests.sh
+      - name: "Upload test result"
+        if: always()
+        uses: actions/upload-artifact@v3
+        with:
+          name: samples-test-result-${{matrix.jdk}}
+          path: test/jobs/*-result*
+  samples-test-result:
+    needs: [samples-test-job]
+    if: always()
+    runs-on: ubuntu-latest
+    env:
+      JAVA_VER: ${{matrix.jdk}}
+    strategy:
+      matrix:
+        jdk: [ 8, 11, 17, 21 ]
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-samples'
+          ref: master
+      - name: "Download test result"
+        uses: actions/download-artifact@v3
+        with:
+          name: samples-test-result-${{matrix.jdk}}
+          path: test/jobs/
+      - name: "Merge test result"
+        run: ./test/scripts/merge-test-results.sh
+
+  integration-test-prepare:
+    runs-on: ubuntu-latest
+    env:
+      JOB_COUNT: 5
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          ref: main
+      - name: "Prepare test list"
+        run: |
+          bash ./test/scripts/prepare-test.sh
+      - name: "Upload test list"
+        uses: actions/upload-artifact@v3
+        with:
+          name: integration-test-list
+          path: test/jobs
+  integration-test-job:
+    needs: [build-source, integration-test-prepare]
+    name: "Integration Test on ubuntu-latest (JobId: ${{matrix.job_id}} JavaVer: ${{matrix.jdk}})"
+    runs-on: ubuntu-latest
+    timeout-minutes: 90
+    env:
+      JAVA_VER: ${{matrix.jdk}}
+      TEST_CASE_FILE: jobs/testjob_${{matrix.job_id}}.txt
+    strategy:
+      fail-fast: false
+      matrix:
+        jdk: [ 8, 11, 17, 21 ]
+        job_id: [1, 2, 3, 4, 5]
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          ref: main
+      - name: "Cache local Maven repository"
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2/repository
+          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
+            ${{ runner.os }}-maven-
+      - name: "Restore Dubbo cache"
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2/repository/org/apache/dubbo
+          key: ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}
+            ${{ runner.os }}-dubbo-snapshot-
+      - name: "Download test list"
+        uses: actions/download-artifact@v3
+        with:
+          name: integration-test-list
+          path: test/jobs/
+      - name: "Set up JDK ${{matrix.jdk}}"
+        uses: actions/setup-java@v1
+        with:
+          java-version: ${{matrix.jdk}}
+      - name: "Init Candidate Versions"
+        run: |
+          DUBBO_VERSION="${{needs.build-source.outputs.version}}"
+          CANDIDATE_VERSIONS="dubbo.version:$DUBBO_VERSION;compiler.version:$DUBBO_VERSION;$CANDIDATE_VERSIONS;dubbo.compiler.version:$DUBBO_VERSION"
+          echo "CANDIDATE_VERSIONS=$CANDIDATE_VERSIONS" >> $GITHUB_ENV
+      - name: "Build test image"
+        run: |
+          cd test && bash ./build-test-image.sh
+      - name: "Run tests"
+        run: cd test && bash ./run-tests.sh
+      - name: "Upload test result"
+        if: always()
+        uses: actions/upload-artifact@v3
+        with:
+          name: integration-test-result-${{matrix.jdk}}
+          path: test/jobs/*-result*
+  integration-test-result:
+    needs: [integration-test-job]
+    if: always()
+    runs-on: ubuntu-latest
+    env:
+      JAVA_VER: ${{matrix.jdk}}
+    strategy:
+      matrix:
+        jdk: [ 8, 11, 17, 21 ]
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          ref: main
+      - name: "Download test result"
+        uses: actions/download-artifact@v3
+        with:
+          name: integration-test-result-${{matrix.jdk}}
+          path: test/jobs/
+      - name: "Merge test result"
+        run: ./test/scripts/merge-test-results.sh
+
+  error-code-inspecting:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          ref: "3.3"
+          path: "./dubbo"
+
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-test-tools'
+          ref: main
+          path: "./dubbo-test-tools"
+
+      - name: "Set up JDK 21"
+        uses: actions/setup-java@v3
+        with:
+          distribution: 'zulu'
+          java-version: 21
+
+      - name: "Compile Dubbo (Linux)"
+        run: |
+          cd ${{ github.workspace }}/dubbo
+          ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast -T 2C clean install -DskipTests=true -DskipIntegrationTests=true -Dcheckstyle.skip=true -Dcheckstyle_unix.skip=true -Drat.skip=true -Dmaven.javadoc.skip=true
+      - name: "Run Error Code Inspecting"
+        env:
+          DUBBO_ECI_REPORT_AS_ERROR: true
+        run: |
+          cd ${{ github.workspace }}/dubbo-test-tools/dubbo-error-code-inspector
+          ../mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast -T 2C package exec:java -Ddubbo.eci.report-as-error=${DUBBO_ECI_REPORT_AS_ERROR} -Dmaven.test.skip=true -Dmaven.test.skip.exec=true -Ddubbo.eci.path=${{ github.workspace }}/dubbo
+
+      - name: "Upload error code inspection result"
+        # always() should not be used here, since we don't need to handle the 'canceled' situation.
+        if: ${{ success() || failure() }}
+        uses: actions/upload-artifact@v3
+        with:
+          name: "error-inspection-result"
+          path: ${{ github.workspace }}/dubbo-test-tools/dubbo-error-code-inspector/error-inspection-result.txt
+
+  native-image-inspecting:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          ref: "3.3"
+          path: "./dubbo"
+
+      - name: "Setup GraalVM environment"
+        uses: graalvm/setup-graalvm@v1
+        with:
+          version: '22.3.0'
+          java-version: '17'
+          components: 'native-image'
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          native-image-job-reports: 'true'
+
+      - name: "Setup Zookeeper environment"
+        run: |
+          wget https://dlcdn.apache.org/zookeeper/zookeeper-3.8.4/apache-zookeeper-3.8.4-bin.tar.gz
+          tar -zxvf apache-zookeeper-3.8.4-bin.tar.gz
+          mv apache-zookeeper-3.8.4-bin/conf/zoo_sample.cfg apache-zookeeper-3.8.4-bin/conf/zoo.cfg
+          apache-zookeeper-3.8.4-bin/bin/zkServer.sh start
+
+      - name: "Check environment"
+        run: |
+          java --version
+          native-image --version
+
+      - name: "Compile Dubbo (Linux)"
+        run: |
+          cd ${{ github.workspace }}/dubbo
+          ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast -T 2C clean install -DskipTests=true -DskipIntegrationTests=true -Dcheckstyle.skip=true -Dcheckstyle_unix.skip=true -Drat.skip=true -Dmaven.javadoc.skip=true
+
+      - name: "Compile and run Dubbo demo for native (Linux)"
+        run: |
+          cd ${{ github.workspace }}/dubbo/dubbo-demo/dubbo-demo-native/dubbo-demo-native-provider
+          ${{ github.workspace }}/dubbo/mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast -T 2C clean package -P native -Dmaven.test.skip=true native:compile
+          nohup ./target/dubbo-demo-native-provider &
+          cd ${{ github.workspace }}/dubbo/dubbo-demo/dubbo-demo-native/dubbo-demo-native-consumer
+          ${{ github.workspace }}/dubbo/mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast -T 2C clean package -P native -Dmaven.test.skip=true native:compile
+          ./target/dubbo-demo-native-consumer

.github/workflows/release-test.yml

@@ -64,7 +64,7 @@ jobs:
         run: 7z a ${{ github.workspace }}/checkstyle.zip *checkstyle* -r
       - name: "Upload checkstyle file if failure"
         if: failure()
-        uses: actions/upload-artifact@v2
+        uses: actions/upload-artifact@v3
         with:
           name: "checkstyle-file"
           path: ${{ github.workspace }}/checkstyle.zip
@@ -132,7 +132,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [ ubuntu-latest, windows-latest ]
-        jdk: [ 8, 11, 17, 19 ]
+        jdk: [ 8, 11, 17, 21 ]
     env:
       DISABLE_FILE_SYSTEM_TEST: true
     steps:
@@ -150,6 +150,14 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
             ${{ runner.os }}-maven-
+      - uses: actions/cache@v3
+        name: "Cache zookeeper binary archive"
+        id: "cache-zookeeper"
+        with:
+          path: ${{ github.workspace }}/.tmp/zookeeper
+          key: zookeeper-${{ runner.os }}-${{ env.ZOOKEEPER_VERSION }}
+          restore-keys: |
+            zookeeper-${{ runner.os }}-
       - name: "Test with Maven with Integration Tests"
         timeout-minutes: 70
         if: ${{ startsWith( matrix.os, 'ubuntu') }}
@@ -158,8 +166,6 @@ jobs:
         timeout-minutes: 90
         if: ${{ startsWith( matrix.os, 'windows') }}
         run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
-      - name: "Upload coverage to Codecov"
-        uses: codecov/codecov-action@v3
 
   unit-test-fastjson2:
     needs: [build-source, unit-test-prepare]
@@ -169,7 +175,7 @@ jobs:
       fail-fast: false
       matrix:
         os: [ ubuntu-latest, windows-latest ]
-        jdk: [ 8, 11, 17, 19 ]
+        jdk: [ 8, 11, 17, 21 ]
     env:
       DISABLE_FILE_SYSTEM_TEST: true
       DUBBO_DEFAULT_SERIALIZATION: fastjson2
@@ -189,18 +195,32 @@ jobs:
           restore-keys: |
             ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
             ${{ runner.os }}-maven-
+      - uses: actions/cache@v3
+        name: "Cache zookeeper binary archive"
+        id: "cache-zookeeper"
+        with:
+          path: ${{ github.workspace }}/.tmp/zookeeper
+          key: zookeeper-${{ runner.os }}-${{ env.ZOOKEEPER_VERSION }}
+          restore-keys: |
+            zookeeper-${{ runner.os }}-
+      - name: "Test with Maven with Integration Tests on JDK 8"
+        timeout-minutes: 70
+        if: ${{ startsWith( matrix.os, 'ubuntu') && matrix.jdk == '8' }}
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco,'!jdk15ge' -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.count=5 -DskipTests=false -DskipIntegrationTests=false -Dcheckstyle.skip=false -Dcheckstyle_unix.skip=false -Drat.skip=false -Dmaven.javadoc.skip=true -DembeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper
+      - name: "Test with Maven without Integration Tests on JDK 8"
+        timeout-minutes: 90
+        if: ${{ startsWith( matrix.os, 'windows') && matrix.jdk == '8' }}
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -P"jacoco,'!jdk15ge'" -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
       - name: "Test with Maven with Integration Tests"
         timeout-minutes: 70
-        if: ${{ startsWith( matrix.os, 'ubuntu') }}
+        if: ${{ startsWith( matrix.os, 'ubuntu') && matrix.jdk != '8' }}
-        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.count=5 -DskipTests=false -DskipIntegrationTests=false -Dcheckstyle.skip=false -Dcheckstyle_unix.skip=false -Drat.skip=false -Dmaven.javadoc.skip=true -DembeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco,jdk15ge-simple,'!jdk15ge' -Dmaven.wagon.httpconnectionManager.ttlSeconds=120 -Dmaven.wagon.http.retryHandler.count=5 -DskipTests=false -DskipIntegrationTests=false -Dcheckstyle.skip=false -Dcheckstyle_unix.skip=false -Drat.skip=false -Dmaven.javadoc.skip=true -DembeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper
       - name: "Test with Maven without Integration Tests"
         timeout-minutes: 90
-        if: ${{ startsWith( matrix.os, 'windows') }}
+        if: ${{ startsWith( matrix.os, 'windows') && matrix.jdk != '8' }}
-        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -Pjacoco -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
+        run: ./mvnw --batch-mode --no-snapshot-updates -e --no-transfer-progress --fail-fast clean test verify -P"jacoco,jdk15ge-simple,'!jdk15ge'" -D"http.keepAlive=false" -D"maven.wagon.http.pool=false" -D"maven.wagon.httpconnectionManager.ttlSeconds=120" -D"maven.wagon.http.retryHandler.count=5" -DskipTests=false -DskipIntegrationTests=true -D"checkstyle.skip=false" -D"checkstyle_unix.skip=true" -D"rat.skip=false" -D"maven.javadoc.skip=true" -D"embeddedZookeeperPath=${{ github.workspace }}/.tmp/zookeeper"
-      - name: "Upload coverage to Codecov"
-        uses: codecov/codecov-action@v3
 
-  integration-test-prepare:
+  samples-test-prepare:
     runs-on: ubuntu-latest
     env:
       JOB_COUNT: 5
@@ -215,12 +235,11 @@ jobs:
       - name: "Upload test list"
         uses: actions/upload-artifact@v3
         with:
-          name: test-list
+          name: samples-test-list
           path: test/jobs
-
+  samples-test-job:
-  integration-test-job:
+    needs: [build-source, samples-test-prepare]
-    needs: [build-source, integration-test-prepare]
+    name: "Samples Test on ubuntu-latest (JobId: ${{matrix.job_id}} JavaVer: ${{matrix.jdk}})"
-    name: "Integration Test on ubuntu-latest (JobId: ${{matrix.job_id}})"
     runs-on: ubuntu-latest
     timeout-minutes: 90
     env:
@@ -229,7 +248,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        jdk: [ 8, 11, 17, 19 ]
+        jdk: [ 8, 11, 17, 21 ]
         job_id: [1, 2, 3, 4, 5]
     steps:
       - uses: actions/checkout@v3
@@ -255,7 +274,7 @@ jobs:
       - name: "Download test list"
         uses: actions/download-artifact@v3
         with:
-          name: test-list
+          name: samples-test-list
           path: test/jobs/
       - name: "Set up JDK ${{matrix.jdk}}"
         uses: actions/setup-java@v1
@@ -275,18 +294,17 @@ jobs:
         if: always()
         uses: actions/upload-artifact@v3
         with:
-          name: test-result-${{matrix.jdk}}
+          name: samples-test-result-${{matrix.jdk}}
           path: test/jobs/*-result*
-
+  samples-test-result:
-  integration-test-result:
+    needs: [samples-test-job]
-    needs: [integration-test-job]
     if: always()
     runs-on: ubuntu-latest
     env:
       JAVA_VER: ${{matrix.jdk}}
     strategy:
       matrix:
-        jdk: [ 8, 11, 17, 19 ]
+        jdk: [ 8, 11, 17, 21 ]
     steps:
       - uses: actions/checkout@v3
         with:
@@ -295,7 +313,105 @@ jobs:
       - name: "Download test result"
         uses: actions/download-artifact@v3
         with:
-          name: test-result-${{matrix.jdk}}
+          name: samples-test-result-${{matrix.jdk}}
+          path: test/jobs/
+      - name: "Merge test result"
+        run: ./test/scripts/merge-test-results.sh
+
+  integration-test-prepare:
+    runs-on: ubuntu-latest
+    env:
+      JOB_COUNT: 5
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          ref: main
+      - name: "Prepare test list"
+        run: |
+          bash ./test/scripts/prepare-test.sh
+      - name: "Upload test list"
+        uses: actions/upload-artifact@v3
+        with:
+          name: integration-test-list
+          path: test/jobs
+  integration-test-job:
+    needs: [build-source, integration-test-prepare]
+    name: "Integration Test on ubuntu-latest (JobId: ${{matrix.job_id}} JavaVer: ${{matrix.jdk}})"
+    runs-on: ubuntu-latest
+    timeout-minutes: 90
+    env:
+      JAVA_VER: ${{matrix.jdk}}
+      TEST_CASE_FILE: jobs/testjob_${{matrix.job_id}}.txt
+    strategy:
+      fail-fast: false
+      matrix:
+        jdk: [ 8, 11, 17, 21 ]
+        job_id: [1, 2, 3, 4, 5]
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          ref: main
+      - name: "Cache local Maven repository"
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2/repository
+          key: ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-maven-${{ hashFiles('**/pom.xml') }}
+            ${{ runner.os }}-maven-
+      - name: "Restore Dubbo cache"
+        uses: actions/cache@v3
+        with:
+          path: ~/.m2/repository/org/apache/dubbo
+          key: ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}-${{ github.run_id }}
+          restore-keys: |
+            ${{ runner.os }}-dubbo-snapshot-${{ github.sha }}
+            ${{ runner.os }}-dubbo-snapshot-
+      - name: "Download test list"
+        uses: actions/download-artifact@v3
+        with:
+          name: integration-test-list
+          path: test/jobs/
+      - name: "Set up JDK ${{matrix.jdk}}"
+        uses: actions/setup-java@v1
+        with:
+          java-version: ${{matrix.jdk}}
+      - name: "Init Candidate Versions"
+        run: |
+          DUBBO_VERSION="${{needs.build-source.outputs.version}}"
+          CANDIDATE_VERSIONS="dubbo.version:$DUBBO_VERSION;compiler.version:$DUBBO_VERSION;$CANDIDATE_VERSIONS;dubbo.compiler.version:$DUBBO_VERSION"
+          echo "CANDIDATE_VERSIONS=$CANDIDATE_VERSIONS" >> $GITHUB_ENV
+      - name: "Build test image"
+        run: |
+          cd test && bash ./build-test-image.sh
+      - name: "Run tests"
+        run: cd test && bash ./run-tests.sh
+      - name: "Upload test result"
+        if: always()
+        uses: actions/upload-artifact@v3
+        with:
+          name: integration-test-result-${{matrix.jdk}}
+          path: test/jobs/*-result*
+  integration-test-result:
+    needs: [integration-test-job]
+    if: always()
+    runs-on: ubuntu-latest
+    env:
+      JAVA_VER: ${{matrix.jdk}}
+    strategy:
+      matrix:
+        jdk: [ 8, 11, 17, 21 ]
+    steps:
+      - uses: actions/checkout@v3
+        with:
+          repository: 'apache/dubbo-integration-cases'
+          ref: main
+      - name: "Download test result"
+        uses: actions/download-artifact@v3
+        with:
+          name: integration-test-result-${{matrix.jdk}}
           path: test/jobs/
       - name: "Merge test result"
         run: ./test/scripts/merge-test-results.sh
@@ -313,10 +429,10 @@ jobs:
           ref: main
           path: "./dubbo-test-tools"
 
-      - name: "Set up JDK 17"
+      - name: "Set up JDK 21"
         uses: actions/setup-java@v3
         with:
-          java-version: 17
+          java-version: 21
           distribution: 'zulu'
 
       - name: "Compile Dubbo (Linux)"
@@ -333,7 +449,7 @@ jobs:
       - name: "Upload error code inspection result"
         # always() should not be used here, since we don't need to handle the 'canceled' situation.
         if: ${{ success() || failure() }}
-        uses: actions/upload-artifact@v2
+        uses: actions/upload-artifact@v3
         with:
           name: "error-inspection-result"
           path: ${{ github.workspace }}/dubbo-test-tools/dubbo-error-code-inspector/error-inspection-result.txt
@@ -356,10 +472,10 @@ jobs:
 
       - name: "Setup Zookeeper environment"
         run: |
-          wget https://dlcdn.apache.org/zookeeper/zookeeper-3.7.1/apache-zookeeper-3.7.1-bin.tar.gz
+          wget https://dlcdn.apache.org/zookeeper/zookeeper-3.8.4/apache-zookeeper-3.8.4-bin.tar.gz
-          tar -zxvf apache-zookeeper-3.7.1-bin.tar.gz
+          tar -zxvf apache-zookeeper-3.8.4-bin.tar.gz
-          mv apache-zookeeper-3.7.1-bin/conf/zoo_sample.cfg apache-zookeeper-3.7.1-bin/conf/zoo.cfg
+          mv apache-zookeeper-3.8.4-bin/conf/zoo_sample.cfg apache-zookeeper-3.8.4-bin/conf/zoo.cfg
-          apache-zookeeper-3.7.1-bin/bin/zkServer.sh start
+          apache-zookeeper-3.8.4-bin/bin/zkServer.sh start
 
       - name: "Check environment"
         run: |

.gitignore

@@ -11,6 +11,8 @@ target/
 .settings/
 .project
 .classpath
+.externalToolBuilders
+maven-eclipse.xml
 
 # idea ignore
 .idea/
@@ -50,3 +52,6 @@ dubbo-demo/dubbo-demo-triple/build/*
 
 .git.exec.error
 
+# log files generated by testcase.
+dubbo-rpc/dubbo-rpc-api/custom-access.log*
+

.licenserc.yaml

@@ -38,7 +38,6 @@ header:
     - '**/*.fc'
     - '**/*.javascript'
     - '**/*.properties'
-    - '**/*.thrift'
     - '**/*.sh'
     - '**/*.bat'
     - '**/*.md'

NOTICE

@@ -1,5 +1,5 @@
 Apache Dubbo
-Copyright 2018-2023 The Apache Software Foundation
+Copyright 2018-2024 The Apache Software Foundation
 
 This product includes software developed at
 The Apache Software Foundation (http://www.apache.org/).

README.md

@@ -6,13 +6,10 @@
 ![License](https://img.shields.io/github/license/alibaba/dubbo.svg)
 [![Average time to resolve an issue](http://isitmaintained.com/badge/resolution/apache/dubbo.svg)](http://isitmaintained.com/project/apache/dubbo "Average time to resolve an issue")
 [![Percentage of issues still open](http://isitmaintained.com/badge/open/apache/dubbo.svg)](http://isitmaintained.com/project/apache/dubbo "Percentage of issues still open")
-[![Tweet](https://img.shields.io/twitter/url/http/shields.io.svg?style=social)](https://twitter.com/intent/tweet?text=Apache%20Dubbo%20is%20a%20high-performance%2C%20java%20based%2C%20open%20source%20RPC%20framework.&url=http://dubbo.apache.org/&via=ApacheDubbo&hashtags=rpc,java,dubbo,micro-service)
-[![Twitter Follow](https://img.shields.io/twitter/follow/ApacheDubbo.svg?label=Follow&style=social&logoWidth=0)](https://twitter.com/intent/follow?screen_name=ApacheDubbo)
-[![Gitter](https://badges.gitter.im/alibaba/dubbo.svg)](https://gitter.im/alibaba/dubbo?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge)
 
 Apache Dubbo is a high-performance, Java-based open-source RPC framework. Please visit the [official site](http://dubbo.apache.org) for the quick start guide and documentation, as well as the [wiki](https://github.com/apache/dubbo/wiki) for news, FAQ, and release notes.
 
-We are now collecting Dubbo user info to help us to improve Dubbo further. Kindly support us by providing your usage information on [issue#9436: Wanted: who's using dubbo](https://github.com/apache/dubbo/issues/9436), thanks :)
+We are now collecting Dubbo user info to help us to improve Dubbo further. Kindly support us by providing your usage information on [Wanted: who's using dubbo](https://github.com/apache/dubbo/discussions/13842), thanks :)
 
 ## Architecture
 
@@ -42,7 +39,7 @@ There's a [README](https://github.com/apache/dubbo-samples/blob/389cd612f1ea57ee
 
 ```xml
 <properties>
-    <dubbo.version>3.2.5</dubbo.version>
+    <dubbo.version>3.2.13-SNAPSHOT</dubbo.version>
 </properties>
 
 <dependencies>
@@ -186,55 +183,85 @@ To avoid intermittent test failures (i.e., flaky tests), it is recommended to ha
 * Minimum of 2CPUs.
 * Minimum of 2Gb of RAM.
 
-## Contact
+### How does the Dubbo Community collaborate?
 
-* Mailing list:
+The Dubbo Community primarily communicates on GitHub through issues, discussions, and pull requests.
-  * dev list: for dev/user discussion. [subscribe](mailto:dev-subscribe@dubbo.apache.org), [unsubscribe](mailto:dev-unsubscribe@dubbo.apache.org), [archive](https://lists.apache.org/list.html?dev@dubbo.apache.org),  [guide](https://github.com/apache/dubbo/wiki/Mailing-list-subscription-guide)
 
-* Bugs: [Issues](https://github.com/apache/dubbo/issues/new?template=dubbo-issue-report-template.md)
+- Issues: We use issues to track bugs and tasks. Any **work-related** item is associated with an issue.
-* Gitter: [Gitter channel](https://gitter.im/alibaba/dubbo)
+- Discussions: We use discussions for questions, early proposals, and announcements. Any **idea-related** item is associated with a discussion.
-* Twitter: [@ApacheDubbo](https://twitter.com/ApacheDubbo)
+- Pull Requests: We use pull requests to merge a set of changes from contributors into Dubbo.
 
-## Contributing
+We have also implemented [a project board](https://github.com/orgs/apache/projects/337) to monitor all the items.
 
-See [CONTRIBUTING](https://github.com/apache/dubbo/blob/master/CONTRIBUTING.md) for details on submitting patches and the contribution workflow.
+Any essential changes should be discussed on the mailing list before they happen.
 
-### How can I contribute?
+### Seeking for help
 
-* Take a look at issues with tags marked [`Good first issue`](https://github.com/apache/dubbo/issues?q=is%3Aopen+is%3Aissue+label%3A%22good+first+issue%22) or [`Help wanted`](https://github.com/apache/dubbo/issues?q=is%3Aopen+is%3Aissue+label%3A%22help+wanted%22).
+If you have questions such as:
-* Join the discussion on the mailing list, subscription [guide](https://github.com/apache/dubbo/wiki/Mailing-list-subscription-guide).
+
-* Answer questions on [issues](https://github.com/apache/dubbo/issues).
+- What is Dubbo?
-* Fix bugs reported on [issues](https://github.com/apache/dubbo/issues), and send us a pull request.
+- How do I use Dubbo?
-* Review the existing [pull request](https://github.com/apache/dubbo/pulls).
+- Why did an unexpected result occur?
-* Improve the [website](https://github.com/apache/dubbo-website), typically we need
+
-  * blog post
+Please start a discussion at https://github.com/apache/dubbo/discussions.
-  * translation on documentation
+
-  * use cases around the integration of Dubbo in enterprise systems.
+However, if you encounter the following situations:
-* Improve the [dubbo-admin/dubbo-monitor](https://github.com/apache/dubbo-admin).
+
-* Contribute to the projects listed in [ecosystem](https://github.com/dubbo).
+- You're certain there's a bug that Dubbo needs to fix,
-* Other forms of contribution not explicitly enumerated above.
+- You believe a feature could be enhanced,
-* If you would like to contribute, please send an email to dev@dubbo.apache.org to let us know!
+- You have a detailed proposal for improving Dubbo,
+
+Please open an issue at https://github.com/apache/dubbo/issues.
+
+To ask effective questions, we recommend reading **[How To Ask Questions The Smart Way](https://github.com/selfteaching/How-To-Ask-Questions-The-Smart-Way/blob/master/How-To-Ask-Questions-The-Smart-Way.md)** first.
+
+### Contribution
+
+- Browse the "help wanted" tasks in the [Dubbo project board](https://github.com/orgs/apache/projects/337).
+- Participate in discussions on the mailing list. See the subscription [guide](https://github.com/apache/dubbo/wiki/Mailing-list-subscription-guide).
+- Respond to queries in the [discussions](https://github.com/apache/dubbo/issues).
+- Resolve bugs reported in [issues](https://github.com/apache/dubbo/issues) and send us a pull request.
+- Review existing [pull requests](https://github.com/apache/dubbo/pulls).
+- Enhance the [website](https://github.com/apache/dubbo-website). We typically need:
+    - Blog posts
+    - Translations for documentation
+    - Use cases showcasing Dubbo integration in enterprise systems.
+- Improve the [dubbo-admin](https://github.com/apache/dubbo-admin).
+- Contribute to the projects listed in the [ecosystem](https://github.com/apache/?q=dubbo&type=all&language=&sort=).
+- Any other forms of contribution not listed above are also welcome.
+- If you're interested in contributing, please send an email to [dev@dubbo.apache.org](mailto:dev@dubbo.apache.org) to let us know!
+
+For further details, please refer our [guide](https://github.com/apache/dubbo/blob/master/CONTRIBUTING.md) about how to contribute Dubbo.
 
 ## Reporting bugs
 
-Please follow the [template](https://github.com/apache/dubbo/issues/new?template=dubbo-issue-report-template.md) for reporting any issues.
+Please follow the [template](https://github.com/apache/dubbo/issues/new?template=dubbo-issue-report-template.md) for reporting any issues.
 
 ## Reporting a security vulnerability
 
 Please report security vulnerabilities to [us](mailto:security@dubbo.apache.org) privately.
 
+## Contact
+
+* Mailing list:
+    * dev list: for dev/user discussion. [subscribe](mailto:dev-subscribe@dubbo.apache.org), [unsubscribe](mailto:dev-unsubscribe@dubbo.apache.org), [archive](https://lists.apache.org/list.html?dev@dubbo.apache.org),  [guide](https://github.com/apache/dubbo/wiki/Mailing-list-subscription-guide)
+
+* Bugs: [Issues](https://github.com/apache/dubbo/issues/new?template=dubbo-issue-report-template.md)
+* Gitter: [Gitter channel](https://gitter.im/alibaba/dubbo)
+* Twitter: [@ApacheDubbo](https://twitter.com/ApacheDubbo)
+
 ## Dubbo ecosystem
 
 * [Dubbo Ecosystem Entry](https://github.com/apache?utf8=%E2%9C%93&q=dubbo&type=&language=) - A GitHub group `dubbo` to gather all Dubbo relevant projects not appropriate in [apache](https://github.com/apache) group yet
 * [Dubbo Website](https://github.com/apache/dubbo-website) - Apache Dubbo official website
 * [Dubbo Samples](https://github.com/apache/dubbo-samples) - samples for Apache Dubbo
-* [Dubbo Spring Boot](https://github.com/apache/dubbo-spring-boot-project) - Spring Boot Project for Dubbo
 * [Dubbo Admin](https://github.com/apache/dubbo-admin) - The reference implementation for Dubbo admin
 * [Dubbo Awesome](https://github.com/apache/dubbo-awesome) - Dubbo's slides and video links in Meetup
 
 #### Language
 
-* [Go](https://github.com/dubbo/dubbo-go) (recommended)
+* [Go](https://github.com/apache/dubbo-go) (recommended)
+* [Rust](https://github.com/apache/dubbo-rust)
 * [Node.js](https://github.com/apache/dubbo-js)
 * [Python](https://github.com/dubbo/py-client-for-apache-dubbo)
 * [PHP](https://github.com/apache/dubbo-php-framework)
@@ -242,4 +269,4 @@ Please report security vulnerabilities to [us](mailto:security@dubbo.apache.org)
 
 ## License
 
-Apache Dubbo software is licensed under the Apache License Version 2.0. See the [LICENSE](https://github.com/apache/dubbo/blob/master/LICENSE) file for details.
+Apache Dubbo software is licensed under the Apache License Version 2.0. See the [LICENSE](https://github.com/apache/dubbo/blob/master/LICENSE) file for details.

SECURITY.md

@@ -5,13 +5,15 @@
 Below is a table that shows versions that we accept security fixes.
 
 | Version | Supported          |
-| ------- | ------------------ |
+|---------| ------------------ |
-| 3.0.x   | :white_check_mark: |
+| 3.3.x   | :white_check_mark: |
-| 2.7.x   | :white_check_mark: |
+| 3.2.x   | :white_check_mark: |
-| 2.6.x   | :white_check_mark: |
+| 3.1.x   | :white_check_mark: |
+| 3.0.x   | :x: |
+| 2.7.x   | :x: |
+| 2.6.x   | :x: |
 | 2.5.x   | :x: |
 
-
 ## Reporting a Vulnerability
 
 The Apache Software Foundation takes a rigorous standpoint in annihilating the security issues in its software projects. Apache Dubbo is highly sensitive and forthcoming to issues pertaining to its features and functionality.

codecov.yml

@@ -32,4 +32,3 @@ ignore:
   - "**/dubbo-native-plugin/**"
   - "**/dubbo-common/src/main/java/org/apache/dubbo/common/json/*.java" #  internal JSON impl is deprecate, ignore test coverage for them
   - "**/dubbo-config/dubbo-config-spring/src/main/java/org/apache/dubbo/config/spring/AnnotationBean.java" # Deprecated
-  - "**/dubbo-rpc/dubbo-rpc-thrift/**"

dubbo-build-tools/pom.xml

@@ -13,18 +13,77 @@
   See the License for the specific language governing permissions and
   limitations under the License.
   -->
-<project xmlns="http://maven.apache.org/POM/4.0.0"
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
-         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+  <modelVersion>4.0.0</modelVersion>
-         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
-    <modelVersion>4.0.0</modelVersion>
 
-    <groupId>org.apache.dubbo</groupId>
+  <groupId>org.apache.dubbo</groupId>
-    <artifactId>dubbo-build-tools</artifactId>
+  <artifactId>dubbo-build-tools</artifactId>
-    <version>1.0.0</version>
+  <version>1.0.0</version>
-    <packaging>jar</packaging>
+  <packaging>jar</packaging>
 
-    <properties>
+  <properties>
-      <maven.deploy.skip>true</maven.deploy.skip>
+    <maven.deploy.skip>true</maven.deploy.skip>
-    </properties>
+    <spotless-maven-plugin.version>2.43.0</spotless-maven-plugin.version>
+    <spotless.action>check</spotless.action>
+    <dubbo-shared-resources.version>1.0.0</dubbo-shared-resources.version>
+    <palantirJavaFormat.version>2.38.0</palantirJavaFormat.version>
+  </properties>
 
-</project>
+  <profiles>
+    <profile>
+      <id>java11+</id>
+      <activation>
+        <jdk>[11,)</jdk>
+      </activation>
+      <build>
+        <!--- newer versions of plugins requires JDK 11 -->
+        <plugins>
+          <plugin>
+            <groupId>com.diffplug.spotless</groupId>
+            <artifactId>spotless-maven-plugin</artifactId>
+            <version>${spotless-maven-plugin.version}</version>
+            <configuration>
+              <java>
+                <palantirJavaFormat>
+                  <version>${palantirJavaFormat.version}</version>
+                </palantirJavaFormat>
+                <removeUnusedImports />
+                <importOrder>
+                  <file>dubbo-importorder.txt</file>
+                </importOrder>
+                <licenseHeader>
+                  <file>checkstyle-header.txt</file>
+                </licenseHeader>
+              </java>
+              <pom>
+                <sortPom>
+                  <expandEmptyElements>false</expandEmptyElements>
+                  <spaceBeforeCloseEmptyElement>true</spaceBeforeCloseEmptyElement>
+                </sortPom>
+              </pom>
+              <upToDateChecking>
+                <enabled>true</enabled>
+              </upToDateChecking>
+            </configuration>
+            <dependencies>
+              <dependency>
+                <groupId>com.alibaba</groupId>
+                <artifactId>dubbo-shared-resources</artifactId>
+                <version>${dubbo-shared-resources.version}</version>
+              </dependency>
+            </dependencies>
+            <executions>
+              <execution>
+                <goals>
+                  <goal>${spotless.action}</goal>
+                </goals>
+                <phase>process-sources</phase>
+              </execution>
+            </executions>
+          </plugin>
+        </plugins>
+      </build>
+    </profile>
+  </profiles>
+
+</project>

dubbo-cluster/pom.xml

@@ -1,3 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
 <!--
   Licensed to the Apache Software Foundation (ASF) under one or more
   contributor license agreements.  See the NOTICE file distributed with
@@ -14,82 +15,104 @@
   See the License for the specific language governing permissions and
   limitations under the License.
   -->
-<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+<project xmlns="http://maven.apache.org/POM/4.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
-         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/maven-v4_0_0.xsd">
+  <modelVersion>4.0.0</modelVersion>
-    <modelVersion>4.0.0</modelVersion>
+  <parent>
-    <parent>
+    <groupId>org.apache.dubbo</groupId>
-        <groupId>org.apache.dubbo</groupId>
+    <artifactId>dubbo-parent</artifactId>
-        <artifactId>dubbo-parent</artifactId>
+    <version>${revision}</version>
-        <version>${revision}</version>
+    <relativePath>../pom.xml</relativePath>
-        <relativePath>../pom.xml</relativePath>
+  </parent>
-    </parent>
+  <artifactId>dubbo-cluster</artifactId>
-    <artifactId>dubbo-cluster</artifactId>
+  <packaging>jar</packaging>
-    <packaging>jar</packaging>
+  <name>${project.artifactId}</name>
-    <name>${project.artifactId}</name>
+  <description>The cluster module of dubbo project</description>
-    <description>The cluster module of dubbo project</description>
+  <properties>
-    <properties>
+    <skip_maven_deploy>false</skip_maven_deploy>
-        <skip_maven_deploy>false</skip_maven_deploy>
+    <nashorn-core.version>15.4</nashorn-core.version>
-    </properties>
+  </properties>
-    <dependencies>
+  <dependencies>
+    <dependency>
+      <groupId>org.apache.dubbo</groupId>
+      <artifactId>dubbo-rpc-api</artifactId>
+      <version>${project.parent.version}</version>
+    </dependency>
+    <dependency>
+      <groupId>org.yaml</groupId>
+      <artifactId>snakeyaml</artifactId>
+    </dependency>
+    <dependency>
+      <groupId>org.apache.dubbo</groupId>
+      <artifactId>dubbo-rpc-injvm</artifactId>
+      <version>${project.parent.version}</version>
+      <scope>test</scope>
+    </dependency>
+    <dependency>
+      <groupId>org.apache.curator</groupId>
+      <artifactId>curator-framework</artifactId>
+      <scope>test</scope>
+    </dependency>
+    <dependency>
+      <groupId>org.apache.zookeeper</groupId>
+      <artifactId>zookeeper</artifactId>
+      <scope>test</scope>
+    </dependency>
+    <dependency>
+      <groupId>org.apache.dubbo</groupId>
+      <artifactId>dubbo-serialization-hessian2</artifactId>
+      <version>${project.parent.version}</version>
+      <scope>test</scope>
+    </dependency>
+    <dependency>
+      <groupId>org.apache.dubbo</groupId>
+      <artifactId>dubbo-serialization-fastjson2</artifactId>
+      <version>${project.parent.version}</version>
+      <scope>test</scope>
+    </dependency>
+    <dependency>
+      <groupId>org.apache.dubbo</groupId>
+      <artifactId>dubbo-test-check</artifactId>
+      <version>${project.parent.version}</version>
+      <scope>test</scope>
+    </dependency>
+    <dependency>
+      <groupId>org.apache.dubbo</groupId>
+      <artifactId>dubbo-metrics-registry</artifactId>
+      <version>${project.parent.version}</version>
+      <scope>compile</scope>
+    </dependency>
+    <dependency>
+      <groupId>org.apache.dubbo</groupId>
+      <artifactId>dubbo-metrics-default</artifactId>
+      <version>${project.parent.version}</version>
+      <optional>true</optional>
+    </dependency>
+    <dependency>
+      <groupId>io.micrometer</groupId>
+      <artifactId>micrometer-tracing-integration-test</artifactId>
+      <scope>test</scope>
+    </dependency>
+  </dependencies>
+
+  <profiles>
+    <!--
+            https://github.com/openjdk/nashorn
+            Nashorn used to be part of the JDK until Java 14.
+            This project provides a standalone version of Nashorn suitable for use with Java 11 and later.
+        -->
+    <profile>
+      <id>nashorn-jdk11</id>
+      <activation>
+        <jdk>[11,)</jdk>
+      </activation>
+      <dependencies>
         <dependency>
-            <groupId>org.apache.dubbo</groupId>
+          <groupId>org.openjdk.nashorn</groupId>
-            <artifactId>dubbo-rpc-api</artifactId>
+          <artifactId>nashorn-core</artifactId>
-            <version>${project.parent.version}</version>
+          <version>${nashorn-core.version}</version>
+          <scope>test</scope>
         </dependency>
-        <dependency>
+      </dependencies>
-            <groupId>org.yaml</groupId>
+    </profile>
-            <artifactId>snakeyaml</artifactId>
+  </profiles>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.dubbo</groupId>
-            <artifactId>dubbo-rpc-injvm</artifactId>
-            <version>${project.parent.version}</version>
-            <scope>test</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.curator</groupId>
-            <artifactId>curator-framework</artifactId>
-            <scope>test</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.zookeeper</groupId>
-            <artifactId>zookeeper</artifactId>
-            <scope>test</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.dubbo</groupId>
-            <artifactId>dubbo-serialization-hessian2</artifactId>
-            <version>${project.parent.version}</version>
-            <scope>test</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.dubbo</groupId>
-            <artifactId>dubbo-serialization-fastjson2</artifactId>
-            <version>${project.parent.version}</version>
-            <scope>test</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.dubbo</groupId>
-            <artifactId>dubbo-test-check</artifactId>
-            <version>${project.parent.version}</version>
-            <scope>test</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.dubbo</groupId>
-            <artifactId>dubbo-metrics-registry</artifactId>
-            <version>${project.parent.version}</version>
-            <scope>compile</scope>
-        </dependency>
-        <dependency>
-            <groupId>org.apache.dubbo</groupId>
-            <artifactId>dubbo-metrics-default</artifactId>
-            <version>${project.parent.version}</version>
-            <optional>true</optional>
-        </dependency>
-        <dependency>
-            <groupId>io.micrometer</groupId>
-            <artifactId>micrometer-tracing-integration-test</artifactId>
-            <scope>test</scope>
-        </dependency>
-    </dependencies>
 </project>

dubbo-cluster/src/main/java/org/apache/dubbo/registry/AddressListener.java

@@ -35,8 +35,5 @@ public interface AddressListener {
      */
     List<URL> notify(List<URL> addresses, URL consumerUrl, Directory registryDirectory);
 
-    default void destroy(URL consumerUrl, Directory registryDirectory) {
+    default void destroy(URL consumerUrl, Directory registryDirectory) {}
-
-    }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/Cluster.java

@@ -55,6 +55,8 @@ public interface Cluster {
         if (StringUtils.isEmpty(name)) {
             name = Cluster.DEFAULT;
         }
-        return ScopeModelUtil.getApplicationModel(scopeModel).getExtensionLoader(Cluster.class).getExtension(name, wrap);
+        return ScopeModelUtil.getApplicationModel(scopeModel)
+                .getExtensionLoader(Cluster.class)
+                .getExtension(name, wrap);
     }
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/Configurator.java

@@ -51,7 +51,6 @@ public interface Configurator extends Comparable<Configurator> {
      */
     URL configure(URL url);
 
-
     /**
      * Convert override urls to map for use when re-refer. Send all rules every time, the urls will be reassembled and
      * calculated
@@ -73,7 +72,9 @@ public interface Configurator extends Comparable<Configurator> {
             return Optional.empty();
         }
 
-        ConfiguratorFactory configuratorFactory = urls.get(0).getOrDefaultApplicationModel().getExtensionLoader(ConfiguratorFactory.class)
+        ConfiguratorFactory configuratorFactory = urls.get(0)
+                .getOrDefaultApplicationModel()
+                .getExtensionLoader(ConfiguratorFactory.class)
                 .getAdaptiveExtension();
 
         List<Configurator> configurators = new ArrayList<>(urls.size());
@@ -83,7 +84,8 @@ public interface Configurator extends Comparable<Configurator> {
                 break;
             }
             Map<String, String> override = new HashMap<>(url.getParameters());
-            //The anyhost parameter of override may be added automatically, it can't change the judgement of changing url
+            // The anyhost parameter of override may be added automatically, it can't change the judgement of changing
+            // url
             override.remove(ANYHOST_KEY);
             if (CollectionUtils.isEmptyMap(override)) {
                 continue;

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/ConfiguratorFactory.java

@@ -36,5 +36,4 @@ public interface ConfiguratorFactory {
      */
     @Adaptive(CommonConstants.PROTOCOL_KEY)
     Configurator getConfigurator(URL url);
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/Constants.java

@@ -100,7 +100,6 @@ public interface Constants {
 
     String OVERRIDE_PROVIDERS_KEY = "providerAddresses";
 
-
     /**
      * key for router type, for e.g., "script"/"file",  corresponding to ScriptRouterFactory.NAME, FileRouterFactory.NAME
      */

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/LoadBalance.java

@@ -46,5 +46,4 @@ public interface LoadBalance {
      */
     @Adaptive("loadbalance")
     <T> Invoker<T> select(List<Invoker<T>> invokers, URL url, Invocation invocation) throws RpcException;
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/Merger.java

@@ -22,5 +22,4 @@ import org.apache.dubbo.common.extension.SPI;
 public interface Merger<T> {
 
     T merge(T... items);
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/ProviderURLMergeProcessor.java

@@ -33,7 +33,9 @@ public interface ProviderURLMergeProcessor {
      */
     URL mergeUrl(URL remoteUrl, Map<String, String> localParametersMap);
 
-    default Map<String, String> mergeLocalParams(Map<String, String> localMap) { return localMap; }
+    default Map<String, String> mergeLocalParams(Map<String, String> localMap) {
+        return localMap;
+    }
 
     default boolean accept(URL providerUrl, Map<String, String> localParametersMap) {
         return true;

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/Router.java

@@ -68,8 +68,8 @@ public interface Router extends Comparable<Router> {
      * @return state with route result
      * @throws RpcException
      */
-    default <T> RouterResult<Invoker<T>> route(List<Invoker<T>> invokers, URL url, Invocation invocation,
+    default <T> RouterResult<Invoker<T>> route(
-                                                     boolean needToPrintMessage) throws RpcException {
+            List<Invoker<T>> invokers, URL url, Invocation invocation, boolean needToPrintMessage) throws RpcException {
         return new RouterResult<>(route(invokers, url, invocation));
     }
 
@@ -80,9 +80,7 @@ public interface Router extends Comparable<Router> {
      * @param invokers invoker list
      * @param <T>      invoker's type
      */
-    default <T> void notify(List<Invoker<T>> invokers) {
+    default <T> void notify(List<Invoker<T>> invokers) {}
-
-    }
 
     /**
      * To decide whether this router need to execute every time an RPC comes or should only execute when addresses or
@@ -109,7 +107,7 @@ public interface Router extends Comparable<Router> {
     int getPriority();
 
     default void stop() {
-        //do nothing by default
+        // do nothing by default
     }
 
     @Override

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/RouterChain.java

@@ -16,12 +16,6 @@
  */
 package org.apache.dubbo.rpc.cluster;
 
-import java.util.List;
-import java.util.concurrent.atomic.AtomicReference;
-import java.util.concurrent.locks.ReadWriteLock;
-import java.util.concurrent.locks.ReentrantReadWriteLock;
-import java.util.stream.Collectors;
-
 import org.apache.dubbo.common.URL;
 import org.apache.dubbo.common.config.ConfigurationUtils;
 import org.apache.dubbo.common.constants.LoggerCodeConstants;
@@ -36,6 +30,12 @@ import org.apache.dubbo.rpc.cluster.router.state.StateRouterFactory;
 import org.apache.dubbo.rpc.model.ModuleModel;
 import org.apache.dubbo.rpc.model.ScopeModelUtil;
 
+import java.util.List;
+import java.util.concurrent.atomic.AtomicReference;
+import java.util.concurrent.locks.ReadWriteLock;
+import java.util.concurrent.locks.ReentrantReadWriteLock;
+import java.util.stream.Collectors;
+
 import static org.apache.dubbo.rpc.cluster.Constants.ROUTER_KEY;
 
 /**
@@ -52,31 +52,30 @@ public class RouterChain<T> {
     public static <T> RouterChain<T> buildChain(Class<T> interfaceClass, URL url) {
         SingleRouterChain<T> chain1 = buildSingleChain(interfaceClass, url);
         SingleRouterChain<T> chain2 = buildSingleChain(interfaceClass, url);
-        return new RouterChain<>(new SingleRouterChain[]{chain1, chain2});
+        return new RouterChain<>(new SingleRouterChain[] {chain1, chain2});
     }
 
     public static <T> SingleRouterChain<T> buildSingleChain(Class<T> interfaceClass, URL url) {
         ModuleModel moduleModel = url.getOrDefaultModuleModel();
 
-        List<RouterFactory> extensionFactories = moduleModel.getExtensionLoader(RouterFactory.class)
+        List<RouterFactory> extensionFactories =
-            .getActivateExtension(url, ROUTER_KEY);
+                moduleModel.getExtensionLoader(RouterFactory.class).getActivateExtension(url, ROUTER_KEY);
 
         List<Router> routers = extensionFactories.stream()
-            .map(factory -> factory.getRouter(url))
+                .map(factory -> factory.getRouter(url))
-            .sorted(Router::compareTo)
+                .sorted(Router::compareTo)
-            .collect(Collectors.toList());
+                .collect(Collectors.toList());
 
-        List<StateRouter<T>> stateRouters = moduleModel
+        List<StateRouter<T>> stateRouters =
-            .getExtensionLoader(StateRouterFactory.class)
+                moduleModel.getExtensionLoader(StateRouterFactory.class).getActivateExtension(url, ROUTER_KEY).stream()
-            .getActivateExtension(url, ROUTER_KEY)
+                        .map(factory -> factory.getRouter(interfaceClass, url))
-            .stream()
+                        .collect(Collectors.toList());
-            .map(factory -> factory.getRouter(interfaceClass, url))
-            .collect(Collectors.toList());
 
+        boolean shouldFailFast = Boolean.parseBoolean(
+                ConfigurationUtils.getProperty(moduleModel, Constants.SHOULD_FAIL_FAST_KEY, "true"));
 
-        boolean shouldFailFast = Boolean.parseBoolean(ConfigurationUtils.getProperty(moduleModel, Constants.SHOULD_FAIL_FAST_KEY, "true"));
+        RouterSnapshotSwitcher routerSnapshotSwitcher =
-
+                ScopeModelUtil.getFrameworkModel(moduleModel).getBeanFactory().getBean(RouterSnapshotSwitcher.class);
-        RouterSnapshotSwitcher routerSnapshotSwitcher = ScopeModelUtil.getFrameworkModel(moduleModel).getBeanFactory().getBean(RouterSnapshotSwitcher.class);
 
         return new SingleRouterChain<>(routers, stateRouters, shouldFailFast, routerSnapshotSwitcher);
     }
@@ -106,9 +105,9 @@ public class RouterChain<T> {
         //     If `availableInvokers` is created from origin invokers => use backup chain
         //     If `availableInvokers` is created from newly invokers  => use main chain
         BitList<Invoker<T>> notifying = notifyingInvokers.get();
-        if (notifying != null &&
+        if (notifying != null
-            currentChain == backupChain &&
+                && currentChain == backupChain
-            availableInvokers.getOriginList() == notifying.getOriginList()) {
+                && availableInvokers.getOriginList() == notifying.getOriginList()) {
             return mainChain;
         }
         return currentChain;
@@ -140,7 +139,8 @@ public class RouterChain<T> {
         }
 
         // Refresh main chain.
-        // No one can request to use main chain. `currentChain` is backup chain. `route` method cannot access main chain.
+        // No one can request to use main chain. `currentChain` is backup chain. `route` method cannot access main
+        // chain.
         try {
             // Lock main chain to wait all invocation end
             // To wait until no one is using main chain.
@@ -157,26 +157,32 @@ public class RouterChain<T> {
         }
 
         // Set the reference of newly invokers to temp variable.
-        // Reason: The next step will switch the invokers reference in directory, so we should check the `availableInvokers`
+        // Reason: The next step will switch the invokers reference in directory, so we should check the
-        //         argument when `route`. If the current invocation use newly invokers, we should use main chain to route, and
+        // `availableInvokers`
+        //         argument when `route`. If the current invocation use newly invokers, we should use main chain to
+        // route, and
         //         this can prevent use newly invokers to route backup chain, which can only route origin invokers now.
         notifyingInvokers.set(invokers);
 
         // Switch the invokers reference in directory.
-        // Cannot switch before update main chain or after backup chain update success. Or that will cause state inconsistent.
+        // Cannot switch before update main chain or after backup chain update success. Or that will cause state
+        // inconsistent.
         switchAction.run();
 
         try {
             // Lock to prevent directory continue list
-            // The invokers reference in directory now should be the newly one and should always use the newly one once lock released.
+            // The invokers reference in directory now should be the newly one and should always use the newly one once
+            // lock released.
             lock.writeLock().lock();
 
             // Switch to main chain. Will update backup chain later.
             currentChain = mainChain;
 
             // Clean up temp variable.
-            // `availableInvokers` check is useless now, because `route` method will no longer receive any `availableInvokers` related
+            // `availableInvokers` check is useless now, because `route` method will no longer receive any
-            // with the origin invokers. The getter of invokers reference in directory is locked now, and will return newly invokers
+            // `availableInvokers` related
+            // with the origin invokers. The getter of invokers reference in directory is locked now, and will return
+            // newly invokers
             // once lock released.
             notifyingInvokers.set(null);
         } finally {
@@ -186,7 +192,8 @@ public class RouterChain<T> {
         }
 
         // Refresh main chain.
-        // No one can request to use main chain. `currentChain` is main chain. `route` method cannot access backup chain.
+        // No one can request to use main chain. `currentChain` is main chain. `route` method cannot access backup
+        // chain.
         try {
             // Lock main chain to wait all invocation end
             backupChain.getLock().writeLock().lock();

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/RuleConverter.java

@@ -14,7 +14,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.apache.dubbo.rpc.cluster;
 
 import org.apache.dubbo.common.URL;
@@ -26,5 +25,4 @@ import java.util.List;
 public interface RuleConverter {
 
     List<URL> convert(URL subscribeUrl, Object source);
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/SingleRouterChain.java

@@ -16,13 +16,6 @@
  */
 package org.apache.dubbo.rpc.cluster;
 
-import java.util.ArrayList;
-import java.util.Collections;
-import java.util.LinkedList;
-import java.util.List;
-import java.util.concurrent.locks.ReadWriteLock;
-import java.util.concurrent.locks.ReentrantReadWriteLock;
-
 import org.apache.dubbo.common.URL;
 import org.apache.dubbo.common.Version;
 import org.apache.dubbo.common.logger.ErrorTypeAwareLogger;
@@ -40,6 +33,13 @@ import org.apache.dubbo.rpc.cluster.router.state.BitList;
 import org.apache.dubbo.rpc.cluster.router.state.StateRouter;
 import org.apache.dubbo.rpc.cluster.router.state.TailStateRouter;
 
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.LinkedList;
+import java.util.List;
+import java.util.concurrent.locks.ReadWriteLock;
+import java.util.concurrent.locks.ReentrantReadWriteLock;
+
 import static org.apache.dubbo.common.constants.LoggerCodeConstants.CLUSTER_FAILED_STOP;
 import static org.apache.dubbo.common.constants.LoggerCodeConstants.CLUSTER_NO_VALID_PROVIDER;
 import static org.apache.dubbo.common.constants.LoggerCodeConstants.INTERNAL_ERROR;
@@ -79,7 +79,11 @@ public class SingleRouterChain<T> {
 
     private final ReadWriteLock lock = new ReentrantReadWriteLock();
 
-    public SingleRouterChain(List<Router> routers, List<StateRouter<T>> stateRouters, boolean shouldFailFast, RouterSnapshotSwitcher routerSnapshotSwitcher) {
+    public SingleRouterChain(
+            List<Router> routers,
+            List<StateRouter<T>> stateRouters,
+            boolean shouldFailFast,
+            RouterSnapshotSwitcher routerSnapshotSwitcher) {
         initWithRouters(routers);
 
         initWithStateRouters(stateRouters);
@@ -134,9 +138,12 @@ public class SingleRouterChain<T> {
 
     public List<Invoker<T>> route(URL url, BitList<Invoker<T>> availableInvokers, Invocation invocation) {
         if (invokers.getOriginList() != availableInvokers.getOriginList()) {
-            logger.error(INTERNAL_ERROR, "", "Router's invoker size: " + invokers.getOriginList().size() +
+            logger.error(
-                    " Invocation's invoker size: " + availableInvokers.getOriginList().size(),
+                    INTERNAL_ERROR,
-                "Reject to route, because the invokers has changed.");
+                    "",
+                    "Router's invoker size: " + invokers.getOriginList().size() + " Invocation's invoker size: "
+                            + availableInvokers.getOriginList().size(),
+                    "Reject to route, because the invokers has changed.");
             throw new IllegalStateException("reject to route, because the invokers has changed.");
         }
         if (RpcContext.getServiceContext().isNeedPrintRouterSnapshot()) {
@@ -202,9 +209,10 @@ public class SingleRouterChain<T> {
     /**
      * Build each router's result
      */
-    public RouterSnapshotNode<T> buildRouterSnapshot(URL url, BitList<Invoker<T>> availableInvokers, Invocation invocation) {
+    public RouterSnapshotNode<T> buildRouterSnapshot(
+            URL url, BitList<Invoker<T>> availableInvokers, Invocation invocation) {
         BitList<Invoker<T>> resultInvokers = availableInvokers.clone();
-        RouterSnapshotNode<T> parentNode = new RouterSnapshotNode<T>("Parent", resultInvokers.clone());
+        RouterSnapshotNode<T> parentNode = new RouterSnapshotNode<>("Parent", resultInvokers.clone());
         parentNode.setNodeOutputInvokers(resultInvokers.clone());
 
         // 1. route state router
@@ -219,7 +227,7 @@ public class SingleRouterChain<T> {
             return parentNode;
         }
 
-        RouterSnapshotNode<T> commonRouterNode = new RouterSnapshotNode<T>("CommonRouter", resultInvokers.clone());
+        RouterSnapshotNode<T> commonRouterNode = new RouterSnapshotNode<>("CommonRouter", resultInvokers.clone());
         parentNode.appendNode(commonRouterNode);
         List<Invoker<T>> commonRouterResult = resultInvokers;
 
@@ -228,7 +236,8 @@ public class SingleRouterChain<T> {
             // Copy resultInvokers to a arrayList. BitList not support
             List<Invoker<T>> inputInvokers = new ArrayList<>(commonRouterResult);
 
-            RouterSnapshotNode<T> currentNode = new RouterSnapshotNode<T>(router.getClass().getSimpleName(), inputInvokers);
+            RouterSnapshotNode<T> currentNode =
+                    new RouterSnapshotNode<>(router.getClass().getSimpleName(), inputInvokers);
 
             // append to router node chain
             commonRouterNode.appendNode(currentNode);
@@ -268,24 +277,27 @@ public class SingleRouterChain<T> {
     }
 
     private void logRouterSnapshot(URL url, Invocation invocation, RouterSnapshotNode<T> snapshotNode) {
-        if (snapshotNode.getChainOutputInvokers() == null ||
+        if (snapshotNode.getChainOutputInvokers() == null
-            snapshotNode.getChainOutputInvokers().isEmpty()) {
+                || snapshotNode.getChainOutputInvokers().isEmpty()) {
             if (logger.isWarnEnabled()) {
                 String message = "No provider available after route for the service " + url.getServiceKey()
-                    + " from registry " + url.getAddress()
+                        + " from registry " + url.getAddress()
-                    + " on the consumer " + NetUtils.getLocalHost()
+                        + " on the consumer " + NetUtils.getLocalHost()
-                    + " using the dubbo version " + Version.getVersion() + ". Router snapshot is below: \n" + snapshotNode.toString();
+                        + " using the dubbo version " + Version.getVersion() + ". Router snapshot is below: \n"
+                        + snapshotNode.toString();
                 if (routerSnapshotSwitcher.isEnable()) {
                     routerSnapshotSwitcher.setSnapshot(message);
                 }
-                logger.warn(CLUSTER_NO_VALID_PROVIDER, "No provider available after route for the service", "", message);
+                logger.warn(
+                        CLUSTER_NO_VALID_PROVIDER, "No provider available after route for the service", "", message);
             }
         } else {
             if (logger.isInfoEnabled()) {
                 String message = "Router snapshot service " + url.getServiceKey()
-                    + " from registry " + url.getAddress()
+                        + " from registry " + url.getAddress()
-                    + " on the consumer " + NetUtils.getLocalHost()
+                        + " on the consumer " + NetUtils.getLocalHost()
-                    + " using the dubbo version " + Version.getVersion() + " is below: \n" + snapshotNode.toString();
+                        + " using the dubbo version " + Version.getVersion() + " is below: \n"
+                        + snapshotNode.toString();
                 if (routerSnapshotSwitcher.isEnable()) {
                     routerSnapshotSwitcher.setSnapshot(message);
                 }
@@ -330,7 +342,12 @@ public class SingleRouterChain<T> {
             try {
                 router.stop();
             } catch (Exception e) {
-                logger.error(CLUSTER_FAILED_STOP, "route stop failed", "", "Error trying to stop router " + router.getClass(), e);
+                logger.error(
+                        CLUSTER_FAILED_STOP,
+                        "route stop failed",
+                        "",
+                        "Error trying to stop router " + router.getClass(),
+                        e);
             }
         }
         routers = Collections.emptyList();
@@ -340,7 +357,12 @@ public class SingleRouterChain<T> {
             try {
                 router.stop();
             } catch (Exception e) {
-                logger.error(CLUSTER_FAILED_STOP, "StateRouter stop failed", "", "Error trying to stop StateRouter " + router.getClass(), e);
+                logger.error(
+                        CLUSTER_FAILED_STOP,
+                        "StateRouter stop failed",
+                        "",
+                        "Error trying to stop StateRouter " + router.getClass(),
+                        e);
             }
         }
         stateRouters = Collections.emptyList();

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/configurator/AbstractConfigurator.java

@@ -72,7 +72,10 @@ public abstract class AbstractConfigurator implements Configurator {
     @Override
     public URL configure(URL url) {
         // If override url is not enabled or is invalid, just return.
-        if (!configuratorUrl.getParameter(ENABLED_KEY, true) || configuratorUrl.getHost() == null || url == null || url.getHost() == null) {
+        if (!configuratorUrl.getParameter(ENABLED_KEY, true)
+                || configuratorUrl.getHost() == null
+                || url == null
+                || url.getHost() == null) {
             logger.info("Cannot apply configurator rule, the rule is disabled or is invalid: \n" + configuratorUrl);
             return url;
         }
@@ -98,7 +101,9 @@ public abstract class AbstractConfigurator implements Configurator {
 
     @Deprecated
     private URL configureDeprecated(URL url) {
-        // If override url has port, means it is a provider address. We want to control a specific provider with this override url, it may take effect on the specific provider instance or on consumers holding this provider instance.
+        // If override url has port, means it is a provider address. We want to control a specific provider with this
+        // override url, it may take effect on the specific provider instance or on consumers holding this provider
+        // instance.
         if (configuratorUrl.getPort() != 0) {
             if (url.getPort() == configuratorUrl.getPort()) {
                 return configureIfMatch(url.getHost(), url);
@@ -131,7 +136,8 @@ public abstract class AbstractConfigurator implements Configurator {
                         if (matcher.isMatch(host, url)) {
                             return doConfigure(url, configuratorUrl.removeParameters(conditionKeys));
                         } else {
-                            logger.debug("Cannot apply configurator rule, param mismatch, current params are " + url + ", params in rule is " + matcher);
+                            logger.debug("Cannot apply configurator rule, param mismatch, current params are " + url
+                                    + ", params in rule is " + matcher);
                         }
                     } else {
                         return doConfigure(url, configuratorUrl.removeParameters(conditionKeys));
@@ -141,7 +147,8 @@ public abstract class AbstractConfigurator implements Configurator {
                 }
             }
         } else {
-            logger.debug("Cannot apply configurator rule, host mismatch, current host is " + host + ", host in rule is " + configuratorUrl.getHost());
+            logger.debug("Cannot apply configurator rule, host mismatch, current host is " + host + ", host in rule is "
+                    + configuratorUrl.getHost());
         }
         return url;
     }
@@ -159,15 +166,16 @@ public abstract class AbstractConfigurator implements Configurator {
             String[] providerAddresses = providers.split(CommonConstants.COMMA_SEPARATOR);
             for (String address : providerAddresses) {
                 if (address.equals(url.getAddress())
-                    || address.equals(ANYHOST_VALUE)
+                        || address.equals(ANYHOST_VALUE)
-                    || address.equals(ANYHOST_VALUE + CommonConstants.GROUP_CHAR_SEPARATOR + ANY_VALUE)
+                        || address.equals(ANYHOST_VALUE + CommonConstants.GROUP_CHAR_SEPARATOR + ANY_VALUE)
-                    || address.equals(ANYHOST_VALUE + CommonConstants.GROUP_CHAR_SEPARATOR + url.getPort())
+                        || address.equals(ANYHOST_VALUE + CommonConstants.GROUP_CHAR_SEPARATOR + url.getPort())
-                    || address.equals(url.getHost())) {
+                        || address.equals(url.getHost())) {
                     match = true;
                 }
             }
             if (!match) {
-                logger.debug("Cannot apply configurator rule, provider address mismatch, current address " + url.getAddress() + ", address in rule is " + providers);
+                logger.debug("Cannot apply configurator rule, provider address mismatch, current address "
+                        + url.getAddress() + ", address in rule is " + providers);
                 return false;
             }
         }
@@ -175,17 +183,18 @@ public abstract class AbstractConfigurator implements Configurator {
         String configApplication = configuratorUrl.getApplication(configuratorUrl.getUsername());
         String currentApplication = url.getApplication(url.getUsername());
         if (configApplication != null
-            && !ANY_VALUE.equals(configApplication)
+                && !ANY_VALUE.equals(configApplication)
-            && !configApplication.equals(currentApplication)) {
+                && !configApplication.equals(currentApplication)) {
-            logger.debug("Cannot apply configurator rule, application name mismatch, current application is " + currentApplication + ", application in rule is " + configApplication);
+            logger.debug("Cannot apply configurator rule, application name mismatch, current application is "
+                    + currentApplication + ", application in rule is " + configApplication);
             return false;
         }
 
         String configServiceKey = configuratorUrl.getServiceKey();
         String currentServiceKey = url.getServiceKey();
-        if (!ANY_VALUE.equals(configServiceKey)
+        if (!ANY_VALUE.equals(configServiceKey) && !configServiceKey.equals(currentServiceKey)) {
-            && !configServiceKey.equals(currentServiceKey)) {
+            logger.debug("Cannot apply configurator rule, service mismatch, current service is " + currentServiceKey
-            logger.debug("Cannot apply configurator rule, service mismatch, current service is " + currentServiceKey + ", service in rule is " + configServiceKey);
+                    + ", service in rule is " + configServiceKey);
             return false;
         }
 
@@ -202,8 +211,9 @@ public abstract class AbstractConfigurator implements Configurator {
                 if (startWithTilde) {
                     conditionKeys.add(key);
                 }
-                if (value != null && !ANY_VALUE.equals(value)
+                if (value != null
-                    && !value.equals(url.getParameter(startWithTilde ? key.substring(1) : key))) {
+                        && !ANY_VALUE.equals(value)
+                        && !value.equals(url.getParameter(startWithTilde ? key.substring(1) : key))) {
                     result = false;
                     break;
                 }
@@ -213,7 +223,7 @@ public abstract class AbstractConfigurator implements Configurator {
     }
 
     private Set<String> genConditionKeys() {
-        Set<String> conditionKeys = new HashSet<String>();
+        Set<String> conditionKeys = new HashSet<>();
         conditionKeys.add(CATEGORY_KEY);
         conditionKeys.add(Constants.CHECK_KEY);
         conditionKeys.add(DYNAMIC_KEY);
@@ -233,5 +243,4 @@ public abstract class AbstractConfigurator implements Configurator {
     }
 
     protected abstract URL doConfigure(URL currentUrl, URL configUrl);
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/configurator/absent/AbsentConfigurator.java

@@ -32,5 +32,4 @@ public class AbsentConfigurator extends AbstractConfigurator {
     public URL doConfigure(URL currentUrl, URL configUrl) {
         return currentUrl.addParametersIfAbsent(configUrl.getParameters());
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/configurator/absent/AbsentConfiguratorFactory.java

@@ -30,5 +30,4 @@ public class AbsentConfiguratorFactory implements ConfiguratorFactory {
     public Configurator getConfigurator(URL url) {
         return new AbsentConfigurator(url);
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/configurator/override/OverrideConfigurator.java

@@ -36,5 +36,4 @@ public class OverrideConfigurator extends AbstractConfigurator {
         logger.info("Start overriding url " + currentUrl + " with override url " + configUrl);
         return currentUrl.addParameters(configUrl.getParameters());
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/configurator/override/OverrideConfiguratorFactory.java

@@ -30,5 +30,4 @@ public class OverrideConfiguratorFactory implements ConfiguratorFactory {
     public Configurator getConfigurator(URL url) {
         return new OverrideConfigurator(url);
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/configurator/parser/ConfigParser.java

@@ -23,14 +23,14 @@ import org.apache.dubbo.common.utils.StringUtils;
 import org.apache.dubbo.rpc.cluster.configurator.parser.model.ConfigItem;
 import org.apache.dubbo.rpc.cluster.configurator.parser.model.ConfiguratorConfig;
 
-import org.yaml.snakeyaml.LoaderOptions;
-import org.yaml.snakeyaml.Yaml;
-import org.yaml.snakeyaml.constructor.SafeConstructor;
-
 import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
 
+import org.yaml.snakeyaml.LoaderOptions;
+import org.yaml.snakeyaml.Yaml;
+import org.yaml.snakeyaml.constructor.SafeConstructor;
+
 import static org.apache.dubbo.common.constants.CommonConstants.ANYHOST_VALUE;
 import static org.apache.dubbo.common.constants.CommonConstants.PROVIDER;
 import static org.apache.dubbo.common.constants.RegistryConstants.APP_DYNAMIC_CONFIGURATORS_CATEGORY;
@@ -104,7 +104,12 @@ public class ConfigParser {
             if (CollectionUtils.isNotEmpty(apps)) {
                 apps.forEach(app -> {
                     StringBuilder tmpUrlBuilder = new StringBuilder(urlBuilder);
-                    urls.add(appendMatchCondition(URL.valueOf(tmpUrlBuilder.append("&application=").append(app).toString()), item));
+                    urls.add(appendMatchCondition(
+                            URL.valueOf(tmpUrlBuilder
+                                    .append("&application=")
+                                    .append(app)
+                                    .toString()),
+                            item));
                 });
             } else {
                 urls.add(appendMatchCondition(URL.valueOf(urlBuilder.toString()), item));
@@ -155,8 +160,8 @@ public class ConfigParser {
         }
         Map<String, String> parameters = item.getParameters();
         if (CollectionUtils.isEmptyMap(parameters)) {
-            throw new IllegalStateException("Invalid configurator rule, please specify at least one parameter " +
+            throw new IllegalStateException("Invalid configurator rule, please specify at least one parameter "
-                "you want to change in the rule.");
+                    + "you want to change in the rule.");
         }
 
         parameters.forEach((k, v) -> {

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/configurator/parser/model/ConditionMatch.java

@@ -101,12 +101,11 @@ public class ConditionMatch {
 
     @Override
     public String toString() {
-        return "ConditionMatch{" +
+        return "ConditionMatch{" + "address='"
-            "address='" + address + '\'' +
+                + address + '\'' + "providerAddress='"
-            "providerAddress='" + providerAddress + '\'' +
+                + providerAddress + '\'' + ", service='"
-            ", service='" + service + '\'' +
+                + service + '\'' + ", app='"
-            ", app='" + app + '\'' +
+                + app + '\'' + ", param='"
-            ", param='" + param + '\'' +
+                + param + '\'' + '}';
-            '}';
     }
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/configurator/parser/model/ConfigItem.java

@@ -68,32 +68,33 @@ public class ConfigItem {
 
         Object addresses = map.get(ADDRESSES_KEY);
         if (addresses != null && List.class.isAssignableFrom(addresses.getClass())) {
-            configItem.setAddresses(((List<Object>) addresses).stream()
+            configItem.setAddresses(
-                    .map(String::valueOf).collect(Collectors.toList()));
+                    ((List<Object>) addresses).stream().map(String::valueOf).collect(Collectors.toList()));
         }
 
         Object providerAddresses = map.get(PROVIDER_ADDRESSES_KEY);
         if (providerAddresses != null && List.class.isAssignableFrom(providerAddresses.getClass())) {
-            configItem.setProviderAddresses(((List<Object>) providerAddresses).stream()
+            configItem.setProviderAddresses(((List<Object>) providerAddresses)
-                    .map(String::valueOf).collect(Collectors.toList()));
+                    .stream().map(String::valueOf).collect(Collectors.toList()));
         }
 
         Object services = map.get(SERVICES_KEY);
         if (services != null && List.class.isAssignableFrom(services.getClass())) {
-            configItem.setServices(((List<Object>) services).stream()
+            configItem.setServices(
-                    .map(String::valueOf).collect(Collectors.toList()));
+                    ((List<Object>) services).stream().map(String::valueOf).collect(Collectors.toList()));
         }
 
         Object applications = map.get(APPLICATIONS_KEY);
         if (applications != null && List.class.isAssignableFrom(applications.getClass())) {
-            configItem.setApplications(((List<Object>) applications).stream()
+            configItem.setApplications(
-                .map(String::valueOf).collect(Collectors.toList()));
+                    ((List<Object>) applications).stream().map(String::valueOf).collect(Collectors.toList()));
         }
 
         Object parameters = map.get(PARAMETERS_KEY);
         if (parameters != null && Map.class.isAssignableFrom(parameters.getClass())) {
-            configItem.setParameters(((Map<String, Object>) parameters).entrySet()
+            configItem.setParameters(((Map<String, Object>) parameters)
-                .stream().collect(Collectors.toMap(Map.Entry::getKey, entry -> entry.getValue().toString())));
+                    .entrySet().stream().collect(Collectors.toMap(Map.Entry::getKey, entry -> entry.getValue()
+                            .toString())));
         }
 
         try {
@@ -102,7 +103,12 @@ public class ConfigItem {
                 configItem.setMatch(PojoUtils.mapToPojo((Map<String, Object>) match, ConditionMatch.class));
             }
         } catch (Throwable t) {
-            logger.error(CLUSTER_FAILED_RECEIVE_RULE, " Failed to parse dynamic configuration rule", String.valueOf(map.get(MATCH_KEY)), "Error occurred when parsing rule component.", t);
+            logger.error(
+                    CLUSTER_FAILED_RECEIVE_RULE,
+                    " Failed to parse dynamic configuration rule",
+                    String.valueOf(map.get(MATCH_KEY)),
+                    "Error occurred when parsing rule component.",
+                    t);
         }
 
         configItem.setSide((String) map.get(SIDE_KEY));

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/configurator/parser/model/ConfiguratorConfig.java

@@ -52,8 +52,8 @@ public class ConfiguratorConfig {
 
         Object configs = map.get(CONFIGS_KEY);
         if (configs != null && List.class.isAssignableFrom(configs.getClass())) {
-            configuratorConfig.setConfigs(((List<Map<String, Object>>) configs).stream()
+            configuratorConfig.setConfigs(((List<Map<String, Object>>) configs)
-                    .map(ConfigItem::parseFromMap).collect(Collectors.toList()));
+                    .stream().map(ConfigItem::parseFromMap).collect(Collectors.toList()));
         }
 
         return configuratorConfig;

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/configurator/parser/model/ParamMatch.java

@@ -50,9 +50,6 @@ public class ParamMatch {
 
     @Override
     public String toString() {
-        return "ParamMatch{" +
+        return "ParamMatch{" + "key='" + key + '\'' + ", value='" + value + '\'' + '}';
-            "key='" + key + '\'' +
-            ", value='" + value + '\'' +
-            '}';
     }
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/directory/AbstractDirectory.java

@@ -25,6 +25,7 @@ import org.apache.dubbo.common.logger.ErrorTypeAwareLogger;
 import org.apache.dubbo.common.logger.LoggerFactory;
 import org.apache.dubbo.common.threadpool.manager.FrameworkExecutorRepository;
 import org.apache.dubbo.common.utils.ConcurrentHashSet;
+import org.apache.dubbo.common.utils.LockUtils;
 import org.apache.dubbo.common.utils.NetUtils;
 import org.apache.dubbo.common.utils.StringUtils;
 import org.apache.dubbo.metrics.event.MetricsEventBus;
@@ -56,6 +57,8 @@ import java.util.concurrent.ScheduledFuture;
 import java.util.concurrent.Semaphore;
 import java.util.concurrent.ThreadLocalRandom;
 import java.util.concurrent.TimeUnit;
+import java.util.concurrent.atomic.AtomicBoolean;
+import java.util.concurrent.locks.ReentrantLock;
 import java.util.stream.Collectors;
 
 import static org.apache.dubbo.common.constants.CommonConstants.CONSUMER;
@@ -122,6 +125,8 @@ public abstract class AbstractDirectory<T> implements Directory<T> {
 
     private volatile ScheduledFuture<?> connectivityCheckFuture;
 
+    private final ReentrantLock invokerRefreshLock = new ReentrantLock();
+
     /**
      * The max count of invokers for each reconnect task select to try to reconnect.
      */
@@ -160,18 +165,20 @@ public abstract class AbstractDirectory<T> implements Directory<T> {
 
         // remove some local only parameters
         applicationModel = url.getOrDefaultApplicationModel();
-        this.queryMap = applicationModel.getBeanFactory().getBean(ClusterUtils.class).mergeLocalParams(queryMap);
+        this.queryMap =
+                applicationModel.getBeanFactory().getBean(ClusterUtils.class).mergeLocalParams(queryMap);
 
         if (consumerUrl == null) {
-            String host = isNotEmpty(queryMap.get(REGISTER_IP_KEY)) ? queryMap.get(REGISTER_IP_KEY) : this.url.getHost();
+            String host =
+                    isNotEmpty(queryMap.get(REGISTER_IP_KEY)) ? queryMap.get(REGISTER_IP_KEY) : this.url.getHost();
             String path = isNotEmpty(queryMap.get(PATH_KEY)) ? queryMap.get(PATH_KEY) : queryMap.get(INTERFACE_KEY);
             String consumedProtocol = isNotEmpty(queryMap.get(PROTOCOL_KEY)) ? queryMap.get(PROTOCOL_KEY) : CONSUMER;
 
             URL consumerUrlFrom = this.url
-                .setHost(host)
+                    .setHost(host)
-                .setPort(0)
+                    .setPort(0)
-                .setProtocol(consumedProtocol)
+                    .setProtocol(consumedProtocol)
-                .setPath(path);
+                    .setPath(path);
             if (isUrlFromRegistry) {
                 // reserve parameters if url is already a consumer url
                 consumerUrlFrom = consumerUrlFrom.clearParameters();
@@ -179,19 +186,23 @@ public abstract class AbstractDirectory<T> implements Directory<T> {
             this.consumerUrl = consumerUrlFrom.addParameters(queryMap);
         }
 
-        this.connectivityExecutor = applicationModel.getFrameworkModel().getBeanFactory()
+        this.connectivityExecutor = applicationModel
-            .getBean(FrameworkExecutorRepository.class).getConnectivityScheduledExecutor();
+                .getFrameworkModel()
+                .getBeanFactory()
+                .getBean(FrameworkExecutorRepository.class)
+                .getConnectivityScheduledExecutor();
         Configuration configuration = ConfigurationUtils.getGlobalConfiguration(url.getOrDefaultModuleModel());
         this.reconnectTaskTryCount = configuration.getInt(RECONNECT_TASK_TRY_COUNT, DEFAULT_RECONNECT_TASK_TRY_COUNT);
         this.reconnectTaskPeriod = configuration.getInt(RECONNECT_TASK_PERIOD, DEFAULT_RECONNECT_TASK_PERIOD);
         setRouterChain(routerChain);
-
     }
 
     @Override
     public List<Invoker<T>> list(Invocation invocation) throws RpcException {
         if (destroyed) {
-            throw new RpcException("Directory of type " + this.getClass().getSimpleName() + " already destroyed for service " + getConsumerUrl().getServiceKey() + " from registry " + getUrl());
+            throw new RpcException(
+                    "Directory of type " + this.getClass().getSimpleName() + " already destroyed for service "
+                            + getConsumerUrl().getServiceKey() + " from registry " + getUrl());
         }
 
         BitList<Invoker<T>> availableInvokers;
@@ -222,12 +233,16 @@ public abstract class AbstractDirectory<T> implements Directory<T> {
             if (routedResult.isEmpty()) {
                 // 2-2 - No provider available.
 
-                logger.warn(CLUSTER_NO_VALID_PROVIDER, "provider server or registry center crashed", "",
+                logger.warn(
-                    "No provider available after connectivity filter for the service " + getConsumerUrl().getServiceKey()
+                        CLUSTER_NO_VALID_PROVIDER,
-                        + " All routed invokers' size: " + routedResult.size()
+                        "provider server or registry center crashed",
-                        + " from registry " + this
+                        "",
-                        + " on the consumer " + NetUtils.getLocalHost()
+                        "No provider available after connectivity filter for the service "
-                        + " using the dubbo version " + Version.getVersion() + ".");
+                                + getConsumerUrl().getServiceKey()
+                                + " All routed invokers' size: " + routedResult.size()
+                                + " from registry " + this
+                                + " on the consumer " + NetUtils.getLocalHost()
+                                + " using the dubbo version " + Version.getVersion() + ".");
             }
             return Collections.unmodifiableList(routedResult);
         } finally {
@@ -283,74 +298,109 @@ public abstract class AbstractDirectory<T> implements Directory<T> {
 
     @Override
     public void addInvalidateInvoker(Invoker<T> invoker) {
-        // 1. remove this invoker from validInvokers list, this invoker will not be listed in the next time
+        LockUtils.safeLock(invokerRefreshLock, LockUtils.DEFAULT_TIMEOUT, () -> {
-        if (removeValidInvoker(invoker)) {
+            // 1. remove this invoker from validInvokers list, this invoker will not be listed in the next time
-            // 2. add this invoker to reconnect list
+            if (removeValidInvoker(invoker)) {
-            invokersToReconnect.add(invoker);
+                // 2. add this invoker to reconnect list
-            // 3. try start check connectivity task
+                invokersToReconnect.add(invoker);
-            checkConnectivity();
+                // 3. try start check connectivity task
+                checkConnectivity();
 
-            logger.info("The invoker " + invoker.getUrl() + " has been added to invalidate list due to connectivity problem. " +
+                logger.info("The invoker " + invoker.getUrl()
-                "Will trying to reconnect to it in the background.");
+                        + " has been added to invalidate list due to connectivity problem. "
-        }
+                        + "Will trying to reconnect to it in the background.");
+            }
+        });
     }
 
     public void checkConnectivity() {
         // try to submit task, to ensure there is only one task at most for each directory
         if (checkConnectivityPermit.tryAcquire()) {
-            this.connectivityCheckFuture = connectivityExecutor.schedule(() -> {
+            this.connectivityCheckFuture = connectivityExecutor.schedule(
-                try {
+                    () -> {
-                    if (isDestroyed()) {
+                        try {
-                        return;
+                            if (isDestroyed()) {
-                    }
+                                return;
-                    RpcContext.getServiceContext().setConsumerUrl(getConsumerUrl());
-                    List<Invoker<T>> needDeleteList = new ArrayList<>();
-                    List<Invoker<T>> invokersToTry = new ArrayList<>();
-
-                    // 1. pick invokers from invokersToReconnect
-                    // limit max reconnectTaskTryCount, prevent this task hang up all the connectivityExecutor for long time
-                    if (invokersToReconnect.size() < reconnectTaskTryCount) {
-                        invokersToTry.addAll(invokersToReconnect);
-                    } else {
-                        for (int i = 0; i < reconnectTaskTryCount; i++) {
-                            Invoker<T> tInvoker = invokersToReconnect.get(ThreadLocalRandom.current().nextInt(invokersToReconnect.size()));
-                            if (!invokersToTry.contains(tInvoker)) {
-                                // ignore if is selected, invokersToTry's size is always smaller than reconnectTaskTryCount + 1
-                                invokersToTry.add(tInvoker);
                             }
-                        }
+                            RpcContext.getServiceContext().setConsumerUrl(getConsumerUrl());
-                    }
+                            List<Invoker<T>> needDeleteList = new ArrayList<>();
+                            List<Invoker<T>> invokersToTry = new ArrayList<>();
 
-                    // 2. try to check the invoker's status
+                            // 1. pick invokers from invokersToReconnect
-                    for (Invoker<T> invoker : invokersToTry) {
+                            // limit max reconnectTaskTryCount, prevent this task hang up all the connectivityExecutor
-                        if (invokers.contains(invoker)) {
+                            // for long time
-                            if (invoker.isAvailable()) {
+                            LockUtils.safeLock(invokerRefreshLock, LockUtils.DEFAULT_TIMEOUT, () -> {
-                                needDeleteList.add(invoker);
+                                if (invokersToReconnect.size() < reconnectTaskTryCount) {
+                                    invokersToTry.addAll(invokersToReconnect);
+                                } else {
+                                    for (int i = 0; i < reconnectTaskTryCount; i++) {
+                                        Invoker<T> tInvoker = invokersToReconnect.get(
+                                                ThreadLocalRandom.current().nextInt(invokersToReconnect.size()));
+                                        if (!invokersToTry.contains(tInvoker)) {
+                                            // ignore if is selected, invokersToTry's size is always smaller than
+                                            // reconnectTaskTryCount + 1
+                                            invokersToTry.add(tInvoker);
+                                        }
+                                    }
+                                }
+                            });
+
+                            // 2. try to check the invoker's status
+                            for (Invoker<T> invoker : invokersToTry) {
+                                AtomicBoolean invokerExist = new AtomicBoolean(false);
+                                LockUtils.safeLock(invokerRefreshLock, LockUtils.DEFAULT_TIMEOUT, () -> {
+                                    invokerExist.set(invokers.contains(invoker));
+                                });
+                                // Should not lock here, `invoker.isAvailable` may need some time to check
+                                if (invokerExist.get()) {
+                                    if (invoker.isAvailable()) {
+                                        needDeleteList.add(invoker);
+                                    }
+                                } else {
+                                    needDeleteList.add(invoker);
+                                }
                             }
-                        } else {
-                            needDeleteList.add(invoker);
-                        }
-                    }
 
-                    // 3. recover valid invoker
+                            // 3. recover valid invoker
-                    for (Invoker<T> tInvoker : needDeleteList) {
+                            LockUtils.safeLock(invokerRefreshLock, LockUtils.DEFAULT_TIMEOUT, () -> {
-                        if (invokers.contains(tInvoker)) {
+                                for (Invoker<T> tInvoker : needDeleteList) {
-                            addValidInvoker(tInvoker);
+                                    if (invokers.contains(tInvoker)) {
-                            logger.info("Recover service address: " + tInvoker.getUrl() + "  from invalid list.");
+                                        addValidInvoker(tInvoker);
+                                        logger.info("Recover service address: " + tInvoker.getUrl()
+                                                + "  from invalid list.");
+                                    } else {
+                                        logger.info(
+                                                "The invoker " + tInvoker.getUrl()
+                                                        + " has been removed from invokers list. Will remove it in reconnect list.");
+                                    }
+                                    invokersToReconnect.remove(tInvoker);
+                                }
+                            });
+                        } catch (Throwable t) {
+                            logger.error(
+                                    LoggerCodeConstants.INTERNAL_ERROR,
+                                    "",
+                                    "",
+                                    "Error occurred when check connectivity. ",
+                                    t);
+                        } finally {
+                            checkConnectivityPermit.release();
                         }
-                        invokersToReconnect.remove(tInvoker);
-                    }
-                } finally {
-                    checkConnectivityPermit.release();
-                }
 
-                // 4. submit new task if it has more to recover
+                        // 4. submit new task if it has more to recover
-                if (!invokersToReconnect.isEmpty()) {
+                        LockUtils.safeLock(invokerRefreshLock, LockUtils.DEFAULT_TIMEOUT, () -> {
-                    checkConnectivity();
+                            if (!invokersToReconnect.isEmpty()) {
-                }
+                                checkConnectivity();
-            }, reconnectTaskPeriod, TimeUnit.MILLISECONDS);
+                            }
+                        });
+                        MetricsEventBus.publish(RegistryEvent.refreshDirectoryEvent(
+                                applicationModel, getSummary(), getDirectoryMeta()));
+                    },
+                    reconnectTaskPeriod,
+                    TimeUnit.MILLISECONDS);
         }
-        MetricsEventBus.publish(RegistryEvent.refreshDirectoryEvent(applicationModel, getSummary(), getDirectoryMeta()));
+        MetricsEventBus.publish(
+                RegistryEvent.refreshDirectoryEvent(applicationModel, getSummary(), getDirectoryMeta()));
     }
 
     /**
@@ -361,17 +411,20 @@ public abstract class AbstractDirectory<T> implements Directory<T> {
      * 4. all the invokers disappeared from total invokers should be removed in the disabled invokers list
      */
     public void refreshInvoker() {
-        if (invokersInitialized) {
+        LockUtils.safeLock(invokerRefreshLock, LockUtils.DEFAULT_TIMEOUT, () -> {
-            refreshInvokerInternal();
+            if (invokersInitialized) {
-        }
+                refreshInvokerInternal();
-        MetricsEventBus.publish(RegistryEvent.refreshDirectoryEvent(applicationModel, getSummary(), getDirectoryMeta()));
+            }
+        });
+        MetricsEventBus.publish(
+                RegistryEvent.refreshDirectoryEvent(applicationModel, getSummary(), getDirectoryMeta()));
     }
 
     protected Map<String, String> getDirectoryMeta() {
         return Collections.emptyMap();
     }
 
-    private synchronized void refreshInvokerInternal() {
+    private void refreshInvokerInternal() {
         BitList<Invoker<T>> copiedInvokers = invokers.clone();
         refreshInvokers(copiedInvokers, invokersToReconnect);
         refreshInvokers(copiedInvokers, disabledInvokers);
@@ -392,37 +445,47 @@ public abstract class AbstractDirectory<T> implements Directory<T> {
 
     @Override
     public void addDisabledInvoker(Invoker<T> invoker) {
-        if (invokers.contains(invoker)) {
+        LockUtils.safeLock(invokerRefreshLock, LockUtils.DEFAULT_TIMEOUT, () -> {
-            disabledInvokers.add(invoker);
+            if (invokers.contains(invoker)) {
-            removeValidInvoker(invoker);
+                disabledInvokers.add(invoker);
-            logger.info("Disable service address: " + invoker.getUrl() + ".");
+                removeValidInvoker(invoker);
-        }
+                logger.info("Disable service address: " + invoker.getUrl() + ".");
-        MetricsEventBus.publish(RegistryEvent.refreshDirectoryEvent(applicationModel, getSummary(), getDirectoryMeta()));
+            }
+        });
+        MetricsEventBus.publish(
+                RegistryEvent.refreshDirectoryEvent(applicationModel, getSummary(), getDirectoryMeta()));
     }
 
     @Override
     public void recoverDisabledInvoker(Invoker<T> invoker) {
-        if (disabledInvokers.remove(invoker)) {
+        LockUtils.safeLock(invokerRefreshLock, LockUtils.DEFAULT_TIMEOUT, () -> {
-            try {
+            if (disabledInvokers.remove(invoker)) {
-                addValidInvoker(invoker);
+                try {
-                logger.info("Recover service address: " + invoker.getUrl() + "  from disabled list.");
+                    addValidInvoker(invoker);
-            } catch (Throwable ignore) {
+                    logger.info("Recover service address: " + invoker.getUrl() + "  from disabled list.");
+                } catch (Throwable ignore) {
 
+                }
             }
-        }
+        });
-        MetricsEventBus.publish(RegistryEvent.refreshDirectoryEvent(applicationModel, getSummary(), getDirectoryMeta()));
+        MetricsEventBus.publish(
+                RegistryEvent.refreshDirectoryEvent(applicationModel, getSummary(), getDirectoryMeta()));
     }
 
     protected final void refreshRouter(BitList<Invoker<T>> newlyInvokers, Runnable switchAction) {
         try {
             routerChain.setInvokers(newlyInvokers.clone(), switchAction);
         } catch (Throwable t) {
-            logger.error(LoggerCodeConstants.INTERNAL_ERROR, "", "", "Error occurred when refreshing router chain. " +
+            logger.error(
-                "The addresses from notification: " +
+                    LoggerCodeConstants.INTERNAL_ERROR,
-                newlyInvokers.stream()
+                    "",
-                    .map(Invoker::getUrl)
+                    "",
-                    .map(URL::getAddress)
+                    "Error occurred when refreshing router chain. " + "The addresses from notification: "
-                    .collect(Collectors.joining(", ")), t);
+                            + newlyInvokers.stream()
+                                    .map(Invoker::getUrl)
+                                    .map(URL::getAddress)
+                                    .collect(Collectors.joining(", ")),
+                    t);
 
             throw t;
         }
@@ -463,40 +526,48 @@ public abstract class AbstractDirectory<T> implements Directory<T> {
     }
 
     protected void setInvokers(BitList<Invoker<T>> invokers) {
-        this.invokers = invokers;
+        LockUtils.safeLock(invokerRefreshLock, LockUtils.DEFAULT_TIMEOUT, () -> {
-        refreshInvokerInternal();
+            this.invokers = invokers;
-        this.invokersInitialized = true;
+            refreshInvokerInternal();
+            this.invokersInitialized = true;
+        });
 
-        MetricsEventBus.publish(RegistryEvent.refreshDirectoryEvent(applicationModel, getSummary(), getDirectoryMeta()));
+        MetricsEventBus.publish(
+                RegistryEvent.refreshDirectoryEvent(applicationModel, getSummary(), getDirectoryMeta()));
     }
 
     protected void destroyInvokers() {
         // set empty instead of clearing to support concurrent access.
-        this.invokers = BitList.emptyList();
+        LockUtils.safeLock(invokerRefreshLock, LockUtils.DEFAULT_TIMEOUT, () -> {
-        this.validInvokers = BitList.emptyList();
+            this.invokers = BitList.emptyList();
-        this.invokersInitialized = false;
+            this.validInvokers = BitList.emptyList();
+            this.invokersInitialized = false;
+        });
     }
 
     private boolean addValidInvoker(Invoker<T> invoker) {
-        boolean result;
+        AtomicBoolean result = new AtomicBoolean(false);
-        synchronized (this.validInvokers) {
+        LockUtils.safeLock(invokerRefreshLock, LockUtils.DEFAULT_TIMEOUT, () -> {
-            result = this.validInvokers.add(invoker);
+            result.set(this.validInvokers.add(invoker));
-        }
+        });
-        MetricsEventBus.publish(RegistryEvent.refreshDirectoryEvent(applicationModel, getSummary(), getDirectoryMeta()));
+        MetricsEventBus.publish(
-        return result;
+                RegistryEvent.refreshDirectoryEvent(applicationModel, getSummary(), getDirectoryMeta()));
+        return result.get();
     }
 
     private boolean removeValidInvoker(Invoker<T> invoker) {
-        boolean result;
+        AtomicBoolean result = new AtomicBoolean(false);
-        synchronized (this.validInvokers) {
+        LockUtils.safeLock(invokerRefreshLock, LockUtils.DEFAULT_TIMEOUT, () -> {
-            result = this.validInvokers.remove(invoker);
+            result.set(this.validInvokers.remove(invoker));
-        }
+        });
-        MetricsEventBus.publish(RegistryEvent.refreshDirectoryEvent(applicationModel, getSummary(), getDirectoryMeta()));
+        MetricsEventBus.publish(
-        return result;
+                RegistryEvent.refreshDirectoryEvent(applicationModel, getSummary(), getDirectoryMeta()));
+        return result.get();
     }
 
-    protected abstract List<Invoker<T>> doList(SingleRouterChain<T> singleRouterChain,
+    protected abstract List<Invoker<T>> doList(
-                                               BitList<Invoker<T>> invokers, Invocation invocation) throws RpcException;
+            SingleRouterChain<T> singleRouterChain, BitList<Invoker<T>> invokers, Invocation invocation)
+            throws RpcException;
 
     protected String joinValidInvokerAddresses() {
         BitList<Invoker<T>> validInvokers = getValidInvokers().clone();
@@ -504,10 +575,10 @@ public abstract class AbstractDirectory<T> implements Directory<T> {
             return "empty";
         }
         return validInvokers.stream()
-            .limit(5)
+                .limit(5)
-            .map(Invoker::getUrl)
+                .map(Invoker::getUrl)
-            .map(URL::getAddress)
+                .map(URL::getAddress)
-            .collect(Collectors.joining(","));
+                .collect(Collectors.joining(","));
     }
 
     private Map<MetricsKey, Map<String, Integer>> getSummary() {
@@ -526,25 +597,27 @@ public abstract class AbstractDirectory<T> implements Directory<T> {
 
     @Override
     public String toString() {
-        return "Directory(" +
+        return "Directory(" + "invokers: "
-            "invokers: " + invokers.size() + "[" +
+                + invokers.size() + "["
-            invokers.stream()
+                + invokers.stream()
-                .map(Invoker::getUrl)
+                        .map(Invoker::getUrl)
-                .map(URL::getAddress)
+                        .map(URL::getAddress)
-                .limit(3)
+                        .limit(3)
-                .collect(Collectors.joining(", ")) + "]" +
+                        .collect(Collectors.joining(", "))
-            ", validInvokers: " + validInvokers.size() + "[" +
+                + "]" + ", validInvokers: "
-            validInvokers.stream()
+                + validInvokers.size() + "["
-                .map(Invoker::getUrl)
+                + validInvokers.stream()
-                .map(URL::getAddress)
+                        .map(Invoker::getUrl)
-                .limit(3)
+                        .map(URL::getAddress)
-                .collect(Collectors.joining(", ")) + "]" +
+                        .limit(3)
-            ", invokersToReconnect: " + invokersToReconnect.size() + "[" +
+                        .collect(Collectors.joining(", "))
-            invokersToReconnect.stream()
+                + "]" + ", invokersToReconnect: "
-                .map(Invoker::getUrl)
+                + invokersToReconnect.size() + "["
-                .map(URL::getAddress)
+                + invokersToReconnect.stream()
-                .limit(3)
+                        .map(Invoker::getUrl)
-                .collect(Collectors.joining(", ")) + "]" +
+                        .map(URL::getAddress)
-            ')';
+                        .limit(3)
+                        .collect(Collectors.joining(", "))
+                + "]" + ')';
     }
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/directory/StaticDirectory.java

@@ -55,7 +55,12 @@ public class StaticDirectory<T> extends AbstractDirectory<T> {
     }
 
     public StaticDirectory(URL url, List<Invoker<T>> invokers, RouterChain<T> routerChain) {
-        super(url == null && CollectionUtils.isNotEmpty(invokers) ? invokers.get(0).getUrl() : url, routerChain, false);
+        super(
+                url == null && CollectionUtils.isNotEmpty(invokers)
+                        ? invokers.get(0).getUrl()
+                        : url,
+                routerChain,
+                false);
         if (CollectionUtils.isEmpty(invokers)) {
             throw new IllegalArgumentException("invokers == null");
         }
@@ -101,28 +106,34 @@ public class StaticDirectory<T> extends AbstractDirectory<T> {
 
     public void buildRouterChain() {
         RouterChain<T> routerChain = RouterChain.buildChain(getInterface(), getUrl());
-        routerChain.setInvokers(getInvokers(), () -> {
+        routerChain.setInvokers(getInvokers(), () -> {});
-        });
         this.setRouterChain(routerChain);
     }
 
     public void notify(List<Invoker<T>> invokers) {
         BitList<Invoker<T>> bitList = new BitList<>(invokers);
         if (routerChain != null) {
-            refreshRouter(bitList.clone(),  () -> this.setInvokers(bitList));
+            refreshRouter(bitList.clone(), () -> this.setInvokers(bitList));
         } else {
             this.setInvokers(bitList);
         }
     }
 
     @Override
-    protected List<Invoker<T>> doList(SingleRouterChain<T> singleRouterChain, BitList<Invoker<T>> invokers, Invocation invocation) throws RpcException {
+    protected List<Invoker<T>> doList(
+            SingleRouterChain<T> singleRouterChain, BitList<Invoker<T>> invokers, Invocation invocation)
+            throws RpcException {
         if (singleRouterChain != null) {
             try {
                 List<Invoker<T>> finalInvokers = singleRouterChain.route(getConsumerUrl(), invokers, invocation);
                 return finalInvokers == null ? BitList.emptyList() : finalInvokers;
             } catch (Throwable t) {
-                logger.error(CLUSTER_FAILED_SITE_SELECTION, "Failed to execute router", "", "Failed to execute router: " + getUrl() + ", cause: " + t.getMessage(), t);
+                logger.error(
+                        CLUSTER_FAILED_SITE_SELECTION,
+                        "Failed to execute router",
+                        "",
+                        "Failed to execute router: " + getUrl() + ", cause: " + t.getMessage(),
+                        t);
                 return BitList.emptyList();
             }
         }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/filter/ClusterFilter.java

@@ -21,5 +21,4 @@ import org.apache.dubbo.common.extension.SPI;
 import org.apache.dubbo.rpc.BaseFilter;
 
 @SPI(scope = ExtensionScope.MODULE)
-public interface ClusterFilter extends BaseFilter {
+public interface ClusterFilter extends BaseFilter {}
-}

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/filter/DefaultFilterChainBuilder.java

@@ -47,12 +47,14 @@ public class DefaultFilterChainBuilder implements FilterChainBuilder {
         List<ModuleModel> moduleModels = getModuleModelsFromUrl(url);
         List<Filter> filters;
         if (moduleModels != null && moduleModels.size() == 1) {
-            filters = ScopeModelUtil.getExtensionLoader(Filter.class, moduleModels.get(0)).getActivateExtension(url, key, group);
+            filters = ScopeModelUtil.getExtensionLoader(Filter.class, moduleModels.get(0))
+                    .getActivateExtension(url, key, group);
         } else if (moduleModels != null && moduleModels.size() > 1) {
             filters = new ArrayList<>();
             List<ExtensionDirector> directors = new ArrayList<>();
             for (ModuleModel moduleModel : moduleModels) {
-                List<Filter> tempFilters = ScopeModelUtil.getExtensionLoader(Filter.class, moduleModel).getActivateExtension(url, key, group);
+                List<Filter> tempFilters = ScopeModelUtil.getExtensionLoader(Filter.class, moduleModel)
+                        .getActivateExtension(url, key, group);
                 filters.addAll(tempFilters);
                 directors.add(moduleModel.getExtensionDirector());
             }
@@ -62,7 +64,6 @@ public class DefaultFilterChainBuilder implements FilterChainBuilder {
             filters = ScopeModelUtil.getExtensionLoader(Filter.class, null).getActivateExtension(url, key, group);
         }
 
-
         if (!CollectionUtils.isEmpty(filters)) {
             for (int i = filters.size() - 1; i >= 0; i--) {
                 final Filter filter = filters.get(i);
@@ -79,25 +80,29 @@ public class DefaultFilterChainBuilder implements FilterChainBuilder {
      * build consumer cluster filter chain
      */
     @Override
-    public <T> ClusterInvoker<T> buildClusterInvokerChain(final ClusterInvoker<T> originalInvoker, String key, String group) {
+    public <T> ClusterInvoker<T> buildClusterInvokerChain(
+            final ClusterInvoker<T> originalInvoker, String key, String group) {
         ClusterInvoker<T> last = originalInvoker;
         URL url = originalInvoker.getUrl();
         List<ModuleModel> moduleModels = getModuleModelsFromUrl(url);
         List<ClusterFilter> filters;
         if (moduleModels != null && moduleModels.size() == 1) {
-            filters = ScopeModelUtil.getExtensionLoader(ClusterFilter.class, moduleModels.get(0)).getActivateExtension(url, key, group);
+            filters = ScopeModelUtil.getExtensionLoader(ClusterFilter.class, moduleModels.get(0))
+                    .getActivateExtension(url, key, group);
         } else if (moduleModels != null && moduleModels.size() > 1) {
             filters = new ArrayList<>();
             List<ExtensionDirector> directors = new ArrayList<>();
             for (ModuleModel moduleModel : moduleModels) {
-                List<ClusterFilter> tempFilters = ScopeModelUtil.getExtensionLoader(ClusterFilter.class, moduleModel).getActivateExtension(url, key, group);
+                List<ClusterFilter> tempFilters = ScopeModelUtil.getExtensionLoader(ClusterFilter.class, moduleModel)
+                        .getActivateExtension(url, key, group);
                 filters.addAll(tempFilters);
                 directors.add(moduleModel.getExtensionDirector());
             }
             filters = sortingAndDeduplication(filters, directors);
 
         } else {
-            filters = ScopeModelUtil.getExtensionLoader(ClusterFilter.class, null).getActivateExtension(url, key, group);
+            filters =
+                    ScopeModelUtil.getExtensionLoader(ClusterFilter.class, null).getActivateExtension(url, key, group);
         }
 
         if (!CollectionUtils.isEmpty(filters)) {
@@ -141,5 +146,4 @@ public class DefaultFilterChainBuilder implements FilterChainBuilder {
         }
         return moduleModels;
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/filter/FilterChainBuilder.java

@@ -92,7 +92,8 @@ public interface FilterChainBuilder {
         public Result invoke(Invocation invocation) throws RpcException {
             Result asyncResult;
             try {
-                InvocationProfilerUtils.enterDetailProfiler(invocation, () -> "Filter " + filter.getClass().getName() + " invoke.");
+                InvocationProfilerUtils.enterDetailProfiler(
+                        invocation, () -> "Filter " + filter.getClass().getName() + " invoke.");
                 asyncResult = filter.invoke(nextNode, invocation);
             } catch (Exception e) {
                 InvocationProfilerUtils.releaseDetailProfiler(invocation);
@@ -159,12 +160,11 @@ public interface FilterChainBuilder {
      * @param <TYPE>
      */
     class ClusterFilterChainNode<T, TYPE extends ClusterInvoker<T>, FILTER extends BaseFilter>
-        extends FilterChainNode<T, TYPE, FILTER> implements ClusterInvoker<T> {
+            extends FilterChainNode<T, TYPE, FILTER> implements ClusterInvoker<T> {
         public ClusterFilterChainNode(TYPE originalInvoker, Invoker<T> nextNode, FILTER filter) {
             super(originalInvoker, nextNode, filter);
         }
 
-
         @Override
         public URL getRegistryUrl() {
             return getOriginalInvoker().getRegistryUrl();
@@ -182,7 +182,8 @@ public interface FilterChainBuilder {
     }
 
     class CallbackRegistrationInvoker<T, FILTER extends BaseFilter> implements Invoker<T> {
-        private static final ErrorTypeAwareLogger LOGGER = LoggerFactory.getErrorTypeAwareLogger(CallbackRegistrationInvoker.class);
+        private static final ErrorTypeAwareLogger LOGGER =
+                LoggerFactory.getErrorTypeAwareLogger(CallbackRegistrationInvoker.class);
         final Invoker<T> filterInvoker;
         final List<FILTER> filters;
 
@@ -223,9 +224,20 @@ public interface FilterChainBuilder {
                             }
                         }
                     } catch (RuntimeException runtimeException) {
-                        LOGGER.error(CLUSTER_EXECUTE_FILTER_EXCEPTION, "the custom filter is abnormal", "", String.format("Exception occurred while executing the %s filter named %s.", i, filter.getClass().getSimpleName()));
+                        LOGGER.error(
+                                CLUSTER_EXECUTE_FILTER_EXCEPTION,
+                                "the custom filter is abnormal",
+                                "",
+                                String.format(
+                                        "Exception occurred while executing the %s filter named %s.",
+                                        i, filter.getClass().getSimpleName()));
                         if (LOGGER.isDebugEnabled()) {
-                            LOGGER.debug(String.format("Whole filter list is: %s", filters.stream().map(tmpFilter -> tmpFilter.getClass().getSimpleName()).collect(Collectors.toList())));
+                            LOGGER.debug(String.format(
+                                    "Whole filter list is: %s",
+                                    filters.stream()
+                                            .map(tmpFilter ->
+                                                    tmpFilter.getClass().getSimpleName())
+                                            .collect(Collectors.toList())));
                         }
                         filterRuntimeException = runtimeException;
                         t = runtimeException;
@@ -264,11 +276,12 @@ public interface FilterChainBuilder {
         }
     }
 
-    class ClusterCallbackRegistrationInvoker<T, FILTER extends BaseFilter> extends CallbackRegistrationInvoker<T, FILTER>
+    class ClusterCallbackRegistrationInvoker<T, FILTER extends BaseFilter>
-        implements ClusterInvoker<T> {
+            extends CallbackRegistrationInvoker<T, FILTER> implements ClusterInvoker<T> {
         private ClusterInvoker<T> originalInvoker;
 
-        public ClusterCallbackRegistrationInvoker(ClusterInvoker<T> originalInvoker, Invoker<T> filterInvoker, List<FILTER> filters) {
+        public ClusterCallbackRegistrationInvoker(
+                ClusterInvoker<T> originalInvoker, Invoker<T> filterInvoker, List<FILTER> filters) {
             super(filterInvoker, filters);
             this.originalInvoker = originalInvoker;
         }
@@ -293,10 +306,11 @@ public interface FilterChainBuilder {
         }
     }
 
-
+    @Experimental(
-    @Experimental("Works for the same purpose as FilterChainNode, replace FilterChainNode with this one when proved stable enough")
+            "Works for the same purpose as FilterChainNode, replace FilterChainNode with this one when proved stable enough")
     class CopyOfFilterChainNode<T, TYPE extends Invoker<T>, FILTER extends BaseFilter> implements Invoker<T> {
-        private static final ErrorTypeAwareLogger LOGGER = LoggerFactory.getErrorTypeAwareLogger(CopyOfFilterChainNode.class);
+        private static final ErrorTypeAwareLogger LOGGER =
+                LoggerFactory.getErrorTypeAwareLogger(CopyOfFilterChainNode.class);
         TYPE originalInvoker;
         Invoker<T> nextNode;
         FILTER filter;
@@ -330,11 +344,14 @@ public interface FilterChainBuilder {
         public Result invoke(Invocation invocation) throws RpcException {
             Result asyncResult;
             try {
-                InvocationProfilerUtils.enterDetailProfiler(invocation, () -> "Filter " + filter.getClass().getName() + " invoke.");
+                InvocationProfilerUtils.enterDetailProfiler(
+                        invocation, () -> "Filter " + filter.getClass().getName() + " invoke.");
                 asyncResult = filter.invoke(nextNode, invocation);
                 if (!(asyncResult instanceof AsyncRpcResult)) {
-                    String msg = "The result of filter invocation must be AsyncRpcResult. (If you want to recreate a result, please use AsyncRpcResult.newDefaultAsyncResult.) " +
+                    String msg =
-                        "Filter class: " + filter.getClass().getName() + ". Result class: " + asyncResult.getClass().getName() + ".";
+                            "The result of filter invocation must be AsyncRpcResult. (If you want to recreate a result, please use AsyncRpcResult.newDefaultAsyncResult.) "
+                                    + "Filter class: " + filter.getClass().getName() + ". Result class: "
+                                    + asyncResult.getClass().getName() + ".";
                     LOGGER.error(INTERNAL_ERROR, "", "", msg);
                     throw new RpcException(msg);
                 }
@@ -372,14 +389,14 @@ public interface FilterChainBuilder {
         }
     }
 
-    @Experimental("Works for the same purpose as ClusterFilterChainNode, replace ClusterFilterChainNode with this one when proved stable enough")
+    @Experimental(
+            "Works for the same purpose as ClusterFilterChainNode, replace ClusterFilterChainNode with this one when proved stable enough")
     class CopyOfClusterFilterChainNode<T, TYPE extends ClusterInvoker<T>, FILTER extends BaseFilter>
-        extends CopyOfFilterChainNode<T, TYPE, FILTER> implements ClusterInvoker<T> {
+            extends CopyOfFilterChainNode<T, TYPE, FILTER> implements ClusterInvoker<T> {
         public CopyOfClusterFilterChainNode(TYPE originalInvoker, Invoker<T> nextNode, FILTER filter) {
             super(originalInvoker, nextNode, filter);
         }
 
-
         @Override
         public URL getRegistryUrl() {
             return getOriginalInvoker().getRegistryUrl();

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/filter/ProtocolFilterWrapper.java

@@ -62,7 +62,8 @@ public class ProtocolFilterWrapper implements Protocol {
     }
 
     private <T> FilterChainBuilder getFilterChainBuilder(URL url) {
-        return ScopeModelUtil.getExtensionLoader(FilterChainBuilder.class, url.getScopeModel()).getDefaultExtension();
+        return ScopeModelUtil.getExtensionLoader(FilterChainBuilder.class, url.getScopeModel())
+                .getDefaultExtension();
     }
 
     @Override
@@ -83,5 +84,4 @@ public class ProtocolFilterWrapper implements Protocol {
     public List<ProtocolServer> getServers() {
         return protocol.getServers();
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/filter/support/ConsumerClassLoaderFilter.java

@@ -35,8 +35,8 @@ public class ConsumerClassLoaderFilter implements ClusterFilter {
         ClassLoader originClassLoader = Thread.currentThread().getContextClassLoader();
         try {
             Optional.ofNullable(invocation.getServiceModel())
-                .map(ServiceModel::getClassLoader)
+                    .map(ServiceModel::getClassLoader)
-                .ifPresent(Thread.currentThread()::setContextClassLoader);
+                    .ifPresent(Thread.currentThread()::setContextClassLoader);
             return invoker.invoke(invocation);
         } finally {
             Thread.currentThread().setContextClassLoader(originClassLoader);

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/filter/support/ConsumerContextFilter.java

@@ -19,7 +19,6 @@ package org.apache.dubbo.rpc.cluster.filter.support;
 import org.apache.dubbo.common.extension.Activate;
 import org.apache.dubbo.common.extension.ExtensionLoader;
 import org.apache.dubbo.common.utils.CollectionUtils;
-import org.apache.dubbo.common.utils.NetUtils;
 import org.apache.dubbo.rpc.AsyncRpcResult;
 import org.apache.dubbo.rpc.Filter;
 import org.apache.dubbo.rpc.Invocation;
@@ -56,16 +55,14 @@ public class ConsumerContextFilter implements ClusterFilter, ClusterFilter.Liste
     private Set<PenetrateAttachmentSelector> supportedSelectors;
 
     public ConsumerContextFilter(ApplicationModel applicationModel) {
-        ExtensionLoader<PenetrateAttachmentSelector> selectorExtensionLoader = applicationModel.getExtensionLoader(PenetrateAttachmentSelector.class);
+        ExtensionLoader<PenetrateAttachmentSelector> selectorExtensionLoader =
+                applicationModel.getExtensionLoader(PenetrateAttachmentSelector.class);
         supportedSelectors = selectorExtensionLoader.getSupportedExtensionInstances();
     }
 
     @Override
     public Result invoke(Invoker<?> invoker, Invocation invocation) throws RpcException {
-        RpcContext.getServiceContext()
+        RpcContext.getServiceContext().setInvoker(invoker).setInvocation(invocation);
-            .setInvoker(invoker)
-            .setInvocation(invocation)
-            .setLocalAddress(NetUtils.getLocalHost(), 0);
 
         RpcContext context = RpcContext.getClientAttachment();
         context.setAttachment(REMOTE_APPLICATION_KEY, invoker.getUrl().getApplication());
@@ -75,15 +72,18 @@ public class ConsumerContextFilter implements ClusterFilter, ClusterFilter.Liste
 
         if (CollectionUtils.isNotEmpty(supportedSelectors)) {
             for (PenetrateAttachmentSelector supportedSelector : supportedSelectors) {
-                Map<String, Object> selected = supportedSelector.select(invocation, RpcContext.getClientAttachment(), RpcContext.getServerAttachment());
+                Map<String, Object> selected = supportedSelector.select(
+                        invocation, RpcContext.getClientAttachment(), RpcContext.getServerAttachment());
                 if (CollectionUtils.isNotEmptyMap(selected)) {
                     ((RpcInvocation) invocation).addObjectAttachments(selected);
                 }
             }
         } else {
-            ((RpcInvocation) invocation).addObjectAttachments(RpcContext.getServerAttachment().getObjectAttachments());
+            ((RpcInvocation) invocation)
+                    .addObjectAttachments(RpcContext.getServerAttachment().getObjectAttachments());
         }
-        Map<String, Object> contextAttachments = RpcContext.getClientAttachment().getObjectAttachments();
+        Map<String, Object> contextAttachments =
+                RpcContext.getClientAttachment().getObjectAttachments();
         if (CollectionUtils.isNotEmptyMap(contextAttachments)) {
             /**
              * invocation.addAttachmentsIfAbsent(context){@link RpcInvocation#addAttachmentsIfAbsent(Map)}should not be used here,
@@ -106,9 +106,12 @@ public class ConsumerContextFilter implements ClusterFilter, ClusterFilter.Liste
 
                 TimeoutCountDown timeoutCountDown = (TimeoutCountDown) countDown;
                 if (timeoutCountDown.isExpired()) {
-                    return AsyncRpcResult.newDefaultAsyncResult(new RpcException(RpcException.TIMEOUT_TERMINATE,
+                    return AsyncRpcResult.newDefaultAsyncResult(
-                        "No time left for making the following call: " + invocation.getServiceName() + "."
+                            new RpcException(
-                            + RpcUtils.getMethodName(invocation) + ", terminate directly."), invocation);
+                                    RpcException.TIMEOUT_TERMINATE,
+                                    "No time left for making the following call: " + invocation.getServiceName() + "."
+                                            + RpcUtils.getMethodName(invocation) + ", terminate directly."),
+                            invocation);
                 }
             }
         }
@@ -135,14 +138,15 @@ public class ConsumerContextFilter implements ClusterFilter, ClusterFilter.Liste
             RpcInvocation rpcInvocation = (RpcInvocation) invocation;
             if (rpcInvocation.getInvokeMode() != null) {
                 // clear service context if not in sync mode
-                if (rpcInvocation.getInvokeMode() == InvokeMode.ASYNC || rpcInvocation.getInvokeMode() == InvokeMode.FUTURE) {
+                if (rpcInvocation.getInvokeMode() == InvokeMode.ASYNC
+                        || rpcInvocation.getInvokeMode() == InvokeMode.FUTURE) {
                     RpcContext.removeServiceContext();
                 }
             }
         }
         // server context must not be removed because user might use it on callback.
-        // So the clear of is delayed til the start of the next rpc call, see RpcContext.removeServerContext(); in invoke() above
+        // So the clear of is delayed til the start of the next rpc call, see RpcContext.removeServerContext(); in
+        // invoke() above
         // RpcContext.removeServerContext();
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/filter/support/MetricsClusterFilter.java

@@ -14,15 +14,14 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.apache.dubbo.rpc.cluster.filter.support;
 
-
 import org.apache.dubbo.common.extension.Activate;
 import org.apache.dubbo.metrics.collector.DefaultMetricsCollector;
 import org.apache.dubbo.metrics.event.MetricsDispatcher;
 import org.apache.dubbo.metrics.event.MetricsEventBus;
 import org.apache.dubbo.metrics.event.RequestEvent;
+import org.apache.dubbo.metrics.model.MethodMetric;
 import org.apache.dubbo.rpc.BaseFilter;
 import org.apache.dubbo.rpc.Invocation;
 import org.apache.dubbo.rpc.Invoker;
@@ -42,6 +41,7 @@ public class MetricsClusterFilter implements ClusterFilter, BaseFilter.Listener,
     private DefaultMetricsCollector collector;
     private String appName;
     private MetricsDispatcher metricsDispatcher;
+    private boolean serviceLevel;
 
     @Override
     public void setApplicationModel(ApplicationModel applicationModel) {
@@ -49,6 +49,7 @@ public class MetricsClusterFilter implements ClusterFilter, BaseFilter.Listener,
         this.collector = applicationModel.getBeanFactory().getBean(DefaultMetricsCollector.class);
         this.appName = applicationModel.tryGetApplicationName();
         this.metricsDispatcher = applicationModel.getBeanFactory().getBean(MetricsDispatcher.class);
+        this.serviceLevel = MethodMetric.isServiceLevel(applicationModel);
     }
 
     @Override
@@ -73,9 +74,15 @@ public class MetricsClusterFilter implements ClusterFilter, BaseFilter.Listener,
         if (t instanceof RpcException) {
             RpcException e = (RpcException) t;
             if (e.isForbidden()) {
-                MetricsEventBus.publish(RequestEvent.toRequestErrorEvent(applicationModel, appName, metricsDispatcher, invocation, CONSUMER_SIDE, e.getCode()));
+                MetricsEventBus.publish(RequestEvent.toRequestErrorEvent(
+                        applicationModel,
+                        appName,
+                        metricsDispatcher,
+                        invocation,
+                        CONSUMER_SIDE,
+                        e.getCode(),
+                        serviceLevel));
             }
         }
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/filter/support/MetricsConsumerFilter.java

@@ -27,10 +27,11 @@ import org.apache.dubbo.rpc.cluster.filter.ClusterFilter;
 
 import static org.apache.dubbo.common.constants.CommonConstants.CONSUMER;
 
-@Activate(group = {CONSUMER}, order = Integer.MIN_VALUE + 100)
+@Activate(
+        group = {CONSUMER},
+        order = Integer.MIN_VALUE + 100)
 public class MetricsConsumerFilter extends MetricsFilter implements ClusterFilter, BaseFilter.Listener {
-    public MetricsConsumerFilter() {
+    public MetricsConsumerFilter() {}
-    }
 
     @Override
     public Result invoke(Invoker<?> invoker, Invocation invocation) throws RpcException {

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/filter/support/ObservationSenderFilter.java

@@ -50,7 +50,8 @@ public class ObservationSenderFilter implements ClusterFilter, BaseFilter.Listen
         applicationModel.getApplicationConfigManager().getTracing().ifPresent(cfg -> {
             if (Boolean.TRUE.equals(cfg.getEnabled())) {
                 observationRegistry = applicationModel.getBeanFactory().getBean(ObservationRegistry.class);
-                clientObservationConvention = applicationModel.getBeanFactory().getBean(DubboClientObservationConvention.class);
+                clientObservationConvention =
+                        applicationModel.getBeanFactory().getBean(DubboClientObservationConvention.class);
             }
         });
     }
@@ -64,7 +65,8 @@ public class ObservationSenderFilter implements ClusterFilter, BaseFilter.Listen
         final Observation observation = DubboObservationDocumentation.CLIENT.observation(
                 this.clientObservationConvention,
                 DefaultDubboClientObservationConvention.getInstance(),
-                () -> senderContext, observationRegistry);
+                () -> senderContext,
+                observationRegistry);
         invocation.put(Observation.class, observation.start());
         return observation.scoped(() -> invoker.invoke(invocation));
     }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/governance/DefaultGovernanceRuleRepositoryImpl.java

@@ -56,5 +56,4 @@ public class DefaultGovernanceRuleRepositoryImpl implements GovernanceRuleReposi
     private DynamicConfiguration getDynamicConfiguration() {
         return moduleModel.modelEnvironment().getDynamicConfiguration().orElse(null);
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/governance/GovernanceRuleRepository.java

@@ -36,7 +36,6 @@ public interface GovernanceRuleRepository {
         addListener(key, DEFAULT_GROUP, listener);
     }
 
-
     /**
      * {@link #removeListener(String, String, ConfigurationListener)}
      *

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/loadbalance/AbstractLoadBalance.java

@@ -47,7 +47,7 @@ public abstract class AbstractLoadBalance implements LoadBalance {
      * @return weight which takes warmup into account
      */
     static int calculateWarmupWeight(int uptime, int warmup, int weight) {
-        int ww = (int) ( uptime / ((float) warmup / weight));
+        int ww = (int) (uptime / ((float) warmup / weight));
         return ww < 1 ? 1 : (Math.min(ww, weight));
     }
 
@@ -64,7 +64,6 @@ public abstract class AbstractLoadBalance implements LoadBalance {
 
     protected abstract <T> Invoker<T> doSelect(List<Invoker<T>> invokers, URL url, Invocation invocation);
 
-
     /**
      * Get the weight of the invoker's invocation which takes warmup time into account
      * if the uptime is within the warmup time, the weight will be reduce proportionally
@@ -94,7 +93,7 @@ public abstract class AbstractLoadBalance implements LoadBalance {
                     }
                     int warmup = invoker.getUrl().getParameter(WARMUP_KEY, DEFAULT_WARMUP);
                     if (uptime > 0 && uptime < warmup) {
-                        weight = calculateWarmupWeight((int)uptime, warmup, weight);
+                        weight = calculateWarmupWeight((int) uptime, warmup, weight);
                     }
                 }
             }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/loadbalance/AdaptiveLoadBalance.java

@@ -41,36 +41,36 @@ public class AdaptiveLoadBalance extends AbstractLoadBalance {
 
     public static final String NAME = "adaptive";
 
-    //default key
+    // default key
     private String attachmentKey = "mem,load";
 
     private final AdaptiveMetrics adaptiveMetrics;
 
-    public AdaptiveLoadBalance(ApplicationModel scopeModel){
+    public AdaptiveLoadBalance(ApplicationModel scopeModel) {
         adaptiveMetrics = scopeModel.getBeanFactory().getBean(AdaptiveMetrics.class);
     }
 
     @Override
     protected <T> Invoker<T> doSelect(List<Invoker<T>> invokers, URL url, Invocation invocation) {
-        Invoker<T> invoker = selectByP2C(invokers,invocation);
+        Invoker<T> invoker = selectByP2C(invokers, invocation);
-        invocation.setAttachment(Constants.ADAPTIVE_LOADBALANCE_ATTACHMENT_KEY,attachmentKey);
+        invocation.setAttachment(Constants.ADAPTIVE_LOADBALANCE_ATTACHMENT_KEY, attachmentKey);
         long startTime = System.currentTimeMillis();
-        invocation.getAttributes().put(Constants.ADAPTIVE_LOADBALANCE_START_TIME,startTime);
+        invocation.getAttributes().put(Constants.ADAPTIVE_LOADBALANCE_START_TIME, startTime);
-        invocation.getAttributes().put(LOADBALANCE_KEY,LoadbalanceRules.ADAPTIVE);
+        invocation.getAttributes().put(LOADBALANCE_KEY, LoadbalanceRules.ADAPTIVE);
-        adaptiveMetrics.addConsumerReq(getServiceKey(invoker,invocation));
+        adaptiveMetrics.addConsumerReq(getServiceKey(invoker, invocation));
-        adaptiveMetrics.setPickTime(getServiceKey(invoker,invocation),startTime);
+        adaptiveMetrics.setPickTime(getServiceKey(invoker, invocation), startTime);
 
         return invoker;
     }
 
-    private <T> Invoker<T> selectByP2C(List<Invoker<T>> invokers, Invocation invocation){
+    private <T> Invoker<T> selectByP2C(List<Invoker<T>> invokers, Invocation invocation) {
         int length = invokers.size();
-        if(length == 1) {
+        if (length == 1) {
             return invokers.get(0);
         }
 
-        if(length == 2) {
+        if (length == 2) {
-            return chooseLowLoadInvoker(invokers.get(0),invokers.get(1),invocation);
+            return chooseLowLoadInvoker(invokers.get(0), invokers.get(1), invocation);
         }
 
         int pos1 = ThreadLocalRandom.current().nextInt(length);
@@ -79,22 +79,22 @@ public class AdaptiveLoadBalance extends AbstractLoadBalance {
             pos2 = pos2 + 1;
         }
 
-        return chooseLowLoadInvoker(invokers.get(pos1),invokers.get(pos2),invocation);
+        return chooseLowLoadInvoker(invokers.get(pos1), invokers.get(pos2), invocation);
     }
 
-    private String getServiceKey(Invoker<?> invoker,Invocation invocation){
+    private String getServiceKey(Invoker<?> invoker, Invocation invocation) {
 
         String key = (String) invocation.getAttributes().get(invoker);
-        if (StringUtils.isNotEmpty(key)){
+        if (StringUtils.isNotEmpty(key)) {
             return key;
         }
 
-        key = buildServiceKey(invoker,invocation);
+        key = buildServiceKey(invoker, invocation);
-        invocation.getAttributes().put(invoker,key);
+        invocation.getAttributes().put(invoker, key);
         return key;
     }
 
-    private String buildServiceKey(Invoker<?> invoker,Invocation invocation){
+    private String buildServiceKey(Invoker<?> invoker, Invocation invocation) {
         URL url = invoker.getUrl();
         StringBuilder sb = new StringBuilder(128);
         sb.append(url.getAddress()).append(":").append(invocation.getProtocolServiceKey());
@@ -104,16 +104,19 @@ public class AdaptiveLoadBalance extends AbstractLoadBalance {
     private int getTimeout(Invoker<?> invoker, Invocation invocation) {
         URL url = invoker.getUrl();
         String methodName = RpcUtils.getMethodName(invocation);
-        return (int) RpcUtils.getTimeout(url,methodName, RpcContext.getClientAttachment(),invocation, DEFAULT_TIMEOUT);
+        return (int)
+                RpcUtils.getTimeout(url, methodName, RpcContext.getClientAttachment(), invocation, DEFAULT_TIMEOUT);
     }
 
-    private <T> Invoker<T> chooseLowLoadInvoker(Invoker<T> invoker1,Invoker<T> invoker2,Invocation invocation){
+    private <T> Invoker<T> chooseLowLoadInvoker(Invoker<T> invoker1, Invoker<T> invoker2, Invocation invocation) {
         int weight1 = getWeight(invoker1, invocation);
         int weight2 = getWeight(invoker2, invocation);
         int timeout1 = getTimeout(invoker1, invocation);
         int timeout2 = getTimeout(invoker2, invocation);
-        long load1 = Double.doubleToLongBits(adaptiveMetrics.getLoad(getServiceKey(invoker1,invocation),weight1,timeout1 ));
+        long load1 = Double.doubleToLongBits(
-        long load2 = Double.doubleToLongBits(adaptiveMetrics.getLoad(getServiceKey(invoker2,invocation),weight2,timeout2 ));
+                adaptiveMetrics.getLoad(getServiceKey(invoker1, invocation), weight1, timeout1));
+        long load2 = Double.doubleToLongBits(
+                adaptiveMetrics.getLoad(getServiceKey(invoker2, invocation), weight2, timeout2));
 
         if (load1 == load2) {
             // The sum of weights
@@ -129,5 +132,4 @@ public class AdaptiveLoadBalance extends AbstractLoadBalance {
         }
         return load1 > load2 ? invoker2 : invoker1;
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/loadbalance/ConsistentHashLoadBalance.java

@@ -46,7 +46,7 @@ public class ConsistentHashLoadBalance extends AbstractLoadBalance {
      */
     public static final String HASH_ARGUMENTS = "hash.arguments";
 
-    private final ConcurrentMap<String, ConsistentHashSelector<?>> selectors = new ConcurrentHashMap<String, ConsistentHashSelector<?>>();
+    private final ConcurrentMap<String, ConsistentHashSelector<?>> selectors = new ConcurrentHashMap<>();
 
     @SuppressWarnings("unchecked")
     @Override
@@ -74,7 +74,7 @@ public class ConsistentHashLoadBalance extends AbstractLoadBalance {
         private final int[] argumentIndex;
 
         ConsistentHashSelector(List<Invoker<T>> invokers, String methodName, int identityHashCode) {
-            this.virtualInvokers = new TreeMap<Long, Invoker<T>>();
+            this.virtualInvokers = new TreeMap<>();
             this.identityHashCode = identityHashCode;
             URL url = invokers.get(0).getUrl();
             this.replicaNumber = url.getMethodParameter(methodName, HASH_NODES, 160);
@@ -96,12 +96,10 @@ public class ConsistentHashLoadBalance extends AbstractLoadBalance {
         }
 
         public Invoker<T> select(Invocation invocation) {
-            byte[] digest = Bytes.getMD5(RpcUtils.getMethodName(invocation));
+            String key = toKey(RpcUtils.getArguments(invocation));
-            return selectForKey(hash(digest, 0));
-        }
 
-        private String toKey(Object[] args, boolean isGeneric) {
+            byte[] digest = Bytes.getMD5(key);
-            return isGeneric ? toKey((Object[]) args[1]) : toKey(args);
+            return selectForKey(hash(digest, 0));
         }
 
         private String toKey(Object[] args) {
@@ -124,11 +122,10 @@ public class ConsistentHashLoadBalance extends AbstractLoadBalance {
 
         private long hash(byte[] digest, int number) {
             return (((long) (digest[3 + number * 4] & 0xFF) << 24)
-                    | ((long) (digest[2 + number * 4] & 0xFF) << 16)
+                            | ((long) (digest[2 + number * 4] & 0xFF) << 16)
-                    | ((long) (digest[1 + number * 4] & 0xFF) << 8)
+                            | ((long) (digest[1 + number * 4] & 0xFF) << 8)
-                    | (digest[number * 4] & 0xFF))
+                            | (digest[number * 4] & 0xFF))
                     & 0xFFFFFFFFL;
         }
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/loadbalance/LeastActiveLoadBalance.java

@@ -56,17 +56,18 @@ public class LeastActiveLoadBalance extends AbstractLoadBalance {
         // Every least active invoker has the same weight value?
         boolean sameWeight = true;
 
-
         // Filter out all the least active invokers
         for (int i = 0; i < length; i++) {
             Invoker<T> invoker = invokers.get(i);
             // Get the active number of the invoker
-            int active = RpcStatus.getStatus(invoker.getUrl(), RpcUtils.getMethodName(invocation)).getActive();
+            int active = RpcStatus.getStatus(invoker.getUrl(), RpcUtils.getMethodName(invocation))
+                    .getActive();
             // Get the weight of the invoker's configuration. The default value is 100.
             int afterWarmup = getWeight(invoker, invocation);
             // save for later use
             weights[i] = afterWarmup;
-            // If it is the first invoker or the active number of the invoker is less than the current least active number
+            // If it is the first invoker or the active number of the invoker is less than the current least active
+            // number
             if (leastActive == -1 || active < leastActive) {
                 // Reset the active number of the current invoker to the least active number
                 leastActive = active;

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/loadbalance/RandomLoadBalance.java

@@ -78,7 +78,8 @@ public class RandomLoadBalance extends AbstractLoadBalance {
             }
         }
         if (totalWeight > 0 && !sameWeight) {
-            // If (not every invoker has the same weight & at least one invoker's weight>0), select randomly based on totalWeight.
+            // If (not every invoker has the same weight & at least one invoker's weight>0), select randomly based on
+            // totalWeight.
             int offset = ThreadLocalRandom.current().nextInt(totalWeight);
             // Return an invoker based on the random value.
             if (length <= 4) {
@@ -92,7 +93,7 @@ public class RandomLoadBalance extends AbstractLoadBalance {
                 if (i < 0) {
                     i = -i - 1;
                 } else {
-                    while (weights[i+1] == offset) {
+                    while (weights[i + 1] == offset) {
                         i++;
                     }
                     i++;

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/loadbalance/RoundRobinLoadBalance.java

@@ -37,7 +37,8 @@ public class RoundRobinLoadBalance extends AbstractLoadBalance {
 
     private static final int RECYCLE_PERIOD = 60000;
 
-    private final ConcurrentMap<String, ConcurrentMap<String, WeightedRoundRobin>> methodWeightMap = new ConcurrentHashMap<>();
+    private final ConcurrentMap<String, ConcurrentMap<String, WeightedRoundRobin>> methodWeightMap =
+            new ConcurrentHashMap<>();
 
     protected static class WeightedRoundRobin {
         private int weight;
@@ -91,7 +92,8 @@ public class RoundRobinLoadBalance extends AbstractLoadBalance {
     @Override
     protected <T> Invoker<T> doSelect(List<Invoker<T>> invokers, URL url, Invocation invocation) {
         String key = invokers.get(0).getUrl().getServiceKey() + "." + RpcUtils.getMethodName(invocation);
-        ConcurrentMap<String, WeightedRoundRobin> map = ConcurrentHashMapUtils.computeIfAbsent(methodWeightMap, key, k -> new ConcurrentHashMap<>());
+        ConcurrentMap<String, WeightedRoundRobin> map =
+                ConcurrentHashMapUtils.computeIfAbsent(methodWeightMap, key, k -> new ConcurrentHashMap<>());
         int totalWeight = 0;
         long maxCurrent = Long.MIN_VALUE;
         long now = System.currentTimeMillis();
@@ -107,7 +109,7 @@ public class RoundRobinLoadBalance extends AbstractLoadBalance {
             });
 
             if (weight != weightedRoundRobin.getWeight()) {
-                //weight changed
+                // weight changed
                 weightedRoundRobin.setWeight(weight);
             }
             long cur = weightedRoundRobin.increaseCurrent();
@@ -129,5 +131,4 @@ public class RoundRobinLoadBalance extends AbstractLoadBalance {
         // should not happen here
         return invokers.get(0);
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/loadbalance/ShortestResponseLoadBalance.java

@@ -59,9 +59,15 @@ public class ShortestResponseLoadBalance extends AbstractLoadBalance implements
 
     @Override
     public void setApplicationModel(ApplicationModel applicationModel) {
-        slidePeriod = applicationModel.modelEnvironment().getConfiguration().getInt(Constants.SHORTEST_RESPONSE_SLIDE_PERIOD, 30_000);
+        slidePeriod = applicationModel
-        executorService = applicationModel.getFrameworkModel().getBeanFactory()
+                .modelEnvironment()
-            .getBean(FrameworkExecutorRepository.class).getSharedExecutor();
+                .getConfiguration()
+                .getInt(Constants.SHORTEST_RESPONSE_SLIDE_PERIOD, 30_000);
+        executorService = applicationModel
+                .getFrameworkModel()
+                .getBeanFactory()
+                .getBean(FrameworkExecutorRepository.class)
+                .getSharedExecutor();
     }
 
     protected static class SlideWindowData {
@@ -118,9 +124,11 @@ public class ShortestResponseLoadBalance extends AbstractLoadBalance implements
         for (int i = 0; i < length; i++) {
             Invoker<T> invoker = invokers.get(i);
             RpcStatus rpcStatus = RpcStatus.getStatus(invoker.getUrl(), RpcUtils.getMethodName(invocation));
-            SlideWindowData slideWindowData = ConcurrentHashMapUtils.computeIfAbsent(methodMap, rpcStatus, SlideWindowData::new);
+            SlideWindowData slideWindowData =
+                    ConcurrentHashMapUtils.computeIfAbsent(methodMap, rpcStatus, SlideWindowData::new);
 
-            // Calculate the estimated response time from the product of active connections and succeeded average elapsed time.
+            // Calculate the estimated response time from the product of active connections and succeeded average
+            // elapsed time.
             long estimateResponse = slideWindowData.getEstimateResponse();
             int afterWarmup = getWeight(invoker, invocation);
             weights[i] = afterWarmup;
@@ -135,16 +143,15 @@ public class ShortestResponseLoadBalance extends AbstractLoadBalance implements
             } else if (estimateResponse == shortestResponse) {
                 shortestIndexes[shortestCount++] = i;
                 totalWeight += afterWarmup;
-                if (sameWeight && i > 0
+                if (sameWeight && i > 0 && afterWarmup != firstWeight) {
-                    && afterWarmup != firstWeight) {
                     sameWeight = false;
                 }
             }
         }
 
         if (System.currentTimeMillis() - lastUpdateTime > slidePeriod
-            && onResetSlideWindow.compareAndSet(false, true)) {
+                && onResetSlideWindow.compareAndSet(false, true)) {
-            //reset slideWindowData in async way
+            // reset slideWindowData in async way
             executorService.execute(() -> {
                 methodMap.values().forEach(SlideWindowData::reset);
                 lastUpdateTime = System.currentTimeMillis();

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/merger/BooleanArrayMerger.java

@@ -14,7 +14,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.apache.dubbo.rpc.cluster.merger;
 
 import org.apache.dubbo.common.utils.ArrayUtils;
@@ -43,5 +42,4 @@ public class BooleanArrayMerger implements Merger<boolean[]> {
         }
         return result;
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/merger/ByteArrayMerger.java

@@ -14,7 +14,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.apache.dubbo.rpc.cluster.merger;
 
 import org.apache.dubbo.common.utils.ArrayUtils;
@@ -43,5 +42,4 @@ public class ByteArrayMerger implements Merger<byte[]> {
         }
         return result;
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/merger/CharArrayMerger.java

@@ -14,7 +14,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.apache.dubbo.rpc.cluster.merger;
 
 import org.apache.dubbo.common.utils.ArrayUtils;

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/merger/DoubleArrayMerger.java

@@ -14,7 +14,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.apache.dubbo.rpc.cluster.merger;
 
 import org.apache.dubbo.common.utils.ArrayUtils;
@@ -30,7 +29,8 @@ public class DoubleArrayMerger implements Merger<double[]> {
         if (ArrayUtils.isEmpty(items)) {
             return new double[0];
         }
-        return Arrays.stream(items).filter(Objects::nonNull)
+        return Arrays.stream(items)
+                .filter(Objects::nonNull)
                 .flatMapToDouble(Arrays::stream)
                 .toArray();
     }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/merger/FloatArrayMerger.java

@@ -14,7 +14,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.apache.dubbo.rpc.cluster.merger;
 
 import org.apache.dubbo.common.utils.ArrayUtils;

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/merger/IntArrayMerger.java

@@ -14,7 +14,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.apache.dubbo.rpc.cluster.merger;
 
 import org.apache.dubbo.common.utils.ArrayUtils;
@@ -30,9 +29,9 @@ public class IntArrayMerger implements Merger<int[]> {
         if (ArrayUtils.isEmpty(items)) {
             return new int[0];
         }
-        return Arrays.stream(items).filter(Objects::nonNull)
+        return Arrays.stream(items)
+                .filter(Objects::nonNull)
                 .flatMapToInt(Arrays::stream)
                 .toArray();
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/merger/ListMerger.java

@@ -14,7 +14,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.apache.dubbo.rpc.cluster.merger;
 
 import org.apache.dubbo.common.utils.ArrayUtils;
@@ -34,10 +33,9 @@ public class ListMerger implements Merger<List<?>> {
         if (ArrayUtils.isEmpty(items)) {
             return Collections.emptyList();
         }
-        return Stream.of(items).filter(Objects::nonNull)
+        return Stream.of(items)
+                .filter(Objects::nonNull)
                 .flatMap(Collection::stream)
                 .collect(Collectors.toList());
-
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/merger/LongArrayMerger.java

@@ -14,7 +14,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.apache.dubbo.rpc.cluster.merger;
 
 import org.apache.dubbo.common.utils.ArrayUtils;
@@ -30,7 +29,8 @@ public class LongArrayMerger implements Merger<long[]> {
         if (ArrayUtils.isEmpty(items)) {
             return new long[0];
         }
-        return Arrays.stream(items).filter(Objects::nonNull)
+        return Arrays.stream(items)
+                .filter(Objects::nonNull)
                 .flatMapToLong(Arrays::stream)
                 .toArray();
     }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/merger/MapMerger.java

@@ -32,9 +32,8 @@ public class MapMerger implements Merger<Map<?, ?>> {
         if (ArrayUtils.isEmpty(items)) {
             return Collections.emptyMap();
         }
-        Map<Object, Object> result = new HashMap<Object, Object>();
+        Map<Object, Object> result = new HashMap<>();
         Stream.of(items).filter(Objects::nonNull).forEach(result::putAll);
         return result;
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/merger/MergerFactory.java

@@ -14,7 +14,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.apache.dubbo.rpc.cluster.merger;
 
 import org.apache.dubbo.common.logger.ErrorTypeAwareLogger;
@@ -37,7 +36,7 @@ public class MergerFactory implements ScopeModelAware {
 
     private static final ErrorTypeAwareLogger logger = LoggerFactory.getErrorTypeAwareLogger(MergerFactory.class);
 
-    private ConcurrentMap<Class<?>, Merger<?>> MERGER_CACHE = new ConcurrentHashMap<Class<?>, Merger<?>>();
+    private ConcurrentMap<Class<?>, Merger<?>> MERGER_CACHE = new ConcurrentHashMap<>();
     private ScopeModel scopeModel;
 
     @Override
@@ -69,13 +68,17 @@ public class MergerFactory implements ScopeModelAware {
     }
 
     private void loadMergers() {
-        Set<String> names = scopeModel.getExtensionLoader(Merger.class)
+        Set<String> names = scopeModel.getExtensionLoader(Merger.class).getSupportedExtensions();
-            .getSupportedExtensions();
         for (String name : names) {
             Merger m = scopeModel.getExtensionLoader(Merger.class).getExtension(name);
             Class<?> actualTypeArg = getActualTypeArgument(m.getClass());
             if (actualTypeArg == null) {
-                logger.warn(CLUSTER_FAILED_LOAD_MERGER,"load merger config failed","","Failed to get actual type argument from merger " + m.getClass().getName());
+                logger.warn(
+                        CLUSTER_FAILED_LOAD_MERGER,
+                        "load merger config failed",
+                        "",
+                        "Failed to get actual type argument from merger "
+                                + m.getClass().getName());
                 continue;
             }
             MERGER_CACHE.putIfAbsent(actualTypeArg, m);
@@ -94,7 +97,7 @@ public class MergerFactory implements ScopeModelAware {
             ParameterizedType mergerType;
             for (Type it : interfaceTypes) {
                 if (it instanceof ParameterizedType
-                    && (mergerType = ((ParameterizedType) it)).getRawType() == Merger.class) {
+                        && (mergerType = ((ParameterizedType) it)).getRawType() == Merger.class) {
                     Type typeArg = mergerType.getActualTypeArguments()[0];
                     return TypeUtils.getRawClass(typeArg);
                 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/merger/SetMerger.java

@@ -32,7 +32,7 @@ public class SetMerger implements Merger<Set<?>> {
         if (ArrayUtils.isEmpty(items)) {
             return Collections.emptySet();
         }
-        Set<Object> result = new HashSet<Object>();
+        Set<Object> result = new HashSet<>();
         Stream.of(items).filter(Objects::nonNull).forEach(result::addAll);
         return result;
     }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/merger/ShortArrayMerger.java

@@ -14,7 +14,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.apache.dubbo.rpc.cluster.merger;
 
 import org.apache.dubbo.common.utils.ArrayUtils;

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/AbstractRouter.java

@@ -28,12 +28,13 @@ public abstract class AbstractRouter implements Router {
     private GovernanceRuleRepository ruleRepository;
 
     public AbstractRouter(URL url) {
-        this.ruleRepository = url.getOrDefaultModuleModel().getExtensionLoader(GovernanceRuleRepository.class).getDefaultExtension();
+        this.ruleRepository = url.getOrDefaultModuleModel()
+                .getExtensionLoader(GovernanceRuleRepository.class)
+                .getDefaultExtension();
         this.url = url;
     }
 
-    public AbstractRouter() {
+    public AbstractRouter() {}
-    }
 
     @Override
     public URL getUrl() {
@@ -67,8 +68,7 @@ public abstract class AbstractRouter implements Router {
         this.priority = priority;
     }
 
-    public GovernanceRuleRepository getRuleRepository(){
+    public GovernanceRuleRepository getRuleRepository() {
         return this.ruleRepository;
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/RouterSnapshotFilter.java

@@ -34,7 +34,7 @@ import static org.apache.dubbo.common.constants.CommonConstants.CONSUMER;
 public class RouterSnapshotFilter implements ClusterFilter, BaseFilter.Listener {
 
     private final RouterSnapshotSwitcher switcher;
-    private final static Logger logger = LoggerFactory.getLogger(RouterSnapshotFilter.class);
+    private static final Logger logger = LoggerFactory.getLogger(RouterSnapshotFilter.class);
 
     public RouterSnapshotFilter(FrameworkModel frameworkModel) {
         this.switcher = frameworkModel.getBeanFactory().getBean(RouterSnapshotSwitcher.class);

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/RouterSnapshotNode.java

@@ -114,43 +114,55 @@ public class RouterSnapshotNode<T> {
 
     public String toString(int level) {
         StringBuilder stringBuilder = new StringBuilder();
-        stringBuilder.append("[ ")
+        stringBuilder
-            .append(name)
+                .append("[ ")
-            .append(' ')
+                .append(name)
-            .append("(Input: ").append(beforeSize).append(") ")
+                .append(' ')
-            .append("(Current Node Output: ").append(nodeOutputSize).append(") ")
+                .append("(Input: ")
-            .append("(Chain Node Output: ").append(chainOutputSize).append(')')
+                .append(beforeSize)
-            .append(routerMessage == null ? "" : " Router message: ")
+                .append(") ")
-            .append(routerMessage == null ? "" : routerMessage)
+                .append("(Current Node Output: ")
-            .append(" ] ");
+                .append(nodeOutputSize)
+                .append(") ")
+                .append("(Chain Node Output: ")
+                .append(chainOutputSize)
+                .append(')')
+                .append(routerMessage == null ? "" : " Router message: ")
+                .append(routerMessage == null ? "" : routerMessage)
+                .append(" ] ");
         if (level == 1) {
-            stringBuilder.append("Input: ")
+            stringBuilder
-                .append(CollectionUtils.isEmpty(inputInvokers) ? "Empty" :
+                    .append("Input: ")
-                        inputInvokers.subList(0, Math.min(5, inputInvokers.size()))
+                    .append(
-                            .stream()
+                            CollectionUtils.isEmpty(inputInvokers)
-                            .map(Invoker::getUrl)
+                                    ? "Empty"
-                            .map(URL::getAddress)
+                                    : inputInvokers.subList(0, Math.min(5, inputInvokers.size())).stream()
-                            .collect(Collectors.joining(",")))
+                                            .map(Invoker::getUrl)
-                .append(" -> ");
+                                            .map(URL::getAddress)
+                                            .collect(Collectors.joining(",")))
+                    .append(" -> ");
 
-            stringBuilder.append("Chain Node Output: ")
+            stringBuilder
-                .append(CollectionUtils.isEmpty(chainOutputInvokers) ? "Empty" :
+                    .append("Chain Node Output: ")
-                    chainOutputInvokers.subList(0, Math.min(5, chainOutputInvokers.size()))
+                    .append(
-                        .stream()
+                            CollectionUtils.isEmpty(chainOutputInvokers)
-                        .map(Invoker::getUrl)
+                                    ? "Empty"
-                        .map(URL::getAddress)
+                                    : chainOutputInvokers.subList(0, Math.min(5, chainOutputInvokers.size())).stream()
-                        .collect(Collectors.joining(",")));
+                                            .map(Invoker::getUrl)
+                                            .map(URL::getAddress)
+                                            .collect(Collectors.joining(",")));
         } else {
-            stringBuilder.append("Current Node Output: ")
+            stringBuilder
-                .append(CollectionUtils.isEmpty(nodeOutputInvokers) ? "Empty" :
+                    .append("Current Node Output: ")
-                    nodeOutputInvokers.subList(0, Math.min(5, nodeOutputInvokers.size()))
+                    .append(
-                        .stream()
+                            CollectionUtils.isEmpty(nodeOutputInvokers)
-                        .map(Invoker::getUrl)
+                                    ? "Empty"
-                        .map(URL::getAddress)
+                                    : nodeOutputInvokers.subList(0, Math.min(5, nodeOutputInvokers.size())).stream()
-                        .collect(Collectors.joining(",")));
+                                            .map(Invoker::getUrl)
+                                            .map(URL::getAddress)
+                                            .collect(Collectors.joining(",")));
         }
 
-
         if (nodeOutputInvokers != null && nodeOutputInvokers.size() > 5) {
             stringBuilder.append("...");
         }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/RouterSnapshotSwitcher.java

@@ -26,7 +26,7 @@ public class RouterSnapshotSwitcher {
     private volatile boolean enable;
     private final Set<String> enabledService = new ConcurrentHashSet<>();
 
-    private final static int MAX_LENGTH = 1 << 5; // 2 ^ 5 = 31
+    private static final int MAX_LENGTH = 1 << 5; // 2 ^ 5 = 31
 
     private final AtomicInteger offset = new AtomicInteger(0);
     private volatile String[] recentSnapshot = new String[MAX_LENGTH];

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/condition/ConditionStateRouter.java

@@ -84,7 +84,8 @@ public class ConditionStateRouter<T> extends AbstractStateRouter<T> {
         super(url);
         this.setForce(force);
         this.enabled = enabled;
-        matcherFactories = moduleModel.getExtensionLoader(ConditionMatcherFactory.class).getActivateExtensions();
+        matcherFactories =
+                moduleModel.getExtensionLoader(ConditionMatcherFactory.class).getActivateExtensions();
         if (enabled) {
             this.init(rule);
         }
@@ -94,7 +95,8 @@ public class ConditionStateRouter<T> extends AbstractStateRouter<T> {
         super(url);
         this.setUrl(url);
         this.setForce(url.getParameter(FORCE_KEY, false));
-        matcherFactories = moduleModel.getExtensionLoader(ConditionMatcherFactory.class).getActivateExtensions();
+        matcherFactories =
+                moduleModel.getExtensionLoader(ConditionMatcherFactory.class).getActivateExtensions();
         this.enabled = url.getParameter(ENABLED_KEY, true);
         if (enabled) {
             init(url.getParameterAndDecoded(RULE_KEY));
@@ -110,8 +112,10 @@ public class ConditionStateRouter<T> extends AbstractStateRouter<T> {
             int i = rule.indexOf("=>");
             String whenRule = i < 0 ? null : rule.substring(0, i).trim();
             String thenRule = i < 0 ? rule.trim() : rule.substring(i + 2).trim();
-            Map<String, ConditionMatcher> when = StringUtils.isBlank(whenRule) || "true".equals(whenRule) ? new HashMap<>() : parseRule(whenRule);
+            Map<String, ConditionMatcher> when =
-            Map<String, ConditionMatcher> then = StringUtils.isBlank(thenRule) || "false".equals(thenRule) ? null : parseRule(thenRule);
+                    StringUtils.isBlank(whenRule) || "true".equals(whenRule) ? new HashMap<>() : parseRule(whenRule);
+            Map<String, ConditionMatcher> then =
+                    StringUtils.isBlank(thenRule) || "false".equals(thenRule) ? null : parseRule(thenRule);
             // NOTE: It should be determined on the business level whether the `When condition` can be empty or not.
             this.whenCondition = when;
             this.thenCondition = then;
@@ -120,8 +124,7 @@ public class ConditionStateRouter<T> extends AbstractStateRouter<T> {
         }
     }
 
-    private Map<String, ConditionMatcher> parseRule(String rule)
+    private Map<String, ConditionMatcher> parseRule(String rule) throws ParseException {
-        throws ParseException {
         Map<String, ConditionMatcher> condition = new HashMap<>();
         if (StringUtils.isBlank(rule)) {
             return condition;
@@ -151,10 +154,12 @@ public class ConditionStateRouter<T> extends AbstractStateRouter<T> {
             // The Value in the KV part.
             else if ("=".equals(separator)) {
                 if (matcherPair == null) {
-                    throw new ParseException("Illegal route rule \""
+                    throw new ParseException(
-                        + rule + "\", The error char '" + separator
+                            "Illegal route rule \""
-                        + "' at index " + matcher.start() + " before \""
+                                    + rule + "\", The error char '" + separator
-                        + content + "\".", matcher.start());
+                                    + "' at index " + matcher.start() + " before \""
+                                    + content + "\".",
+                            matcher.start());
                 }
 
                 values = matcherPair.getMatches();
@@ -163,10 +168,12 @@ public class ConditionStateRouter<T> extends AbstractStateRouter<T> {
             // The Value in the KV part.
             else if ("!=".equals(separator)) {
                 if (matcherPair == null) {
-                    throw new ParseException("Illegal route rule \""
+                    throw new ParseException(
-                        + rule + "\", The error char '" + separator
+                            "Illegal route rule \""
-                        + "' at index " + matcher.start() + " before \""
+                                    + rule + "\", The error char '" + separator
-                        + content + "\".", matcher.start());
+                                    + "' at index " + matcher.start() + " before \""
+                                    + content + "\".",
+                            matcher.start());
                 }
 
                 values = matcherPair.getMismatches();
@@ -175,25 +182,34 @@ public class ConditionStateRouter<T> extends AbstractStateRouter<T> {
             // The Value in the KV part, if Value have more than one items.
             else if (",".equals(separator)) { // Should be separated by ','
                 if (values == null || values.isEmpty()) {
-                    throw new ParseException("Illegal route rule \""
+                    throw new ParseException(
-                            + rule + "\", The error char '" + separator
+                            "Illegal route rule \""
-                            + "' at index " + matcher.start() + " before \""
+                                    + rule + "\", The error char '" + separator
-                            + content + "\".", matcher.start());
+                                    + "' at index " + matcher.start() + " before \""
+                                    + content + "\".",
+                            matcher.start());
                 }
                 values.add(content);
             } else {
-                throw new ParseException("Illegal route rule \"" + rule
+                throw new ParseException(
-                        + "\", The error char '" + separator + "' at index "
+                        "Illegal route rule \"" + rule
-                        + matcher.start() + " before \"" + content + "\".", matcher.start());
+                                + "\", The error char '" + separator + "' at index "
+                                + matcher.start() + " before \"" + content + "\".",
+                        matcher.start());
             }
         }
         return condition;
     }
 
     @Override
-    protected BitList<Invoker<T>> doRoute(BitList<Invoker<T>> invokers, URL url, Invocation invocation,
+    protected BitList<Invoker<T>> doRoute(
-                                          boolean needToPrintMessage, Holder<RouterSnapshotNode<T>> nodeHolder,
+            BitList<Invoker<T>> invokers,
-                                          Holder<String> messageHolder) throws RpcException {
+            URL url,
+            Invocation invocation,
+            boolean needToPrintMessage,
+            Holder<RouterSnapshotNode<T>> nodeHolder,
+            Holder<String> messageHolder)
+            throws RpcException {
         if (!enabled) {
             if (needToPrintMessage) {
                 messageHolder.set("Directly return. Reason: ConditionRouter disabled.");
@@ -215,7 +231,13 @@ public class ConditionStateRouter<T> extends AbstractStateRouter<T> {
                 return invokers;
             }
             if (thenCondition == null) {
-                logger.warn(CLUSTER_CONDITIONAL_ROUTE_LIST_EMPTY,"condition state router thenCondition is empty","","The current consumer in the service blacklist. consumer: " + NetUtils.getLocalHost() + ", service: " + url.getServiceKey());                if (needToPrintMessage) {
+                logger.warn(
+                        CLUSTER_CONDITIONAL_ROUTE_LIST_EMPTY,
+                        "condition state router thenCondition is empty",
+                        "",
+                        "The current consumer in the service blocklist. consumer: " + NetUtils.getLocalHost()
+                                + ", service: " + url.getServiceKey());
+                if (needToPrintMessage) {
                     messageHolder.set("Empty return. Reason: ThenCondition is empty.");
                 }
                 return BitList.emptyList();
@@ -229,14 +251,26 @@ public class ConditionStateRouter<T> extends AbstractStateRouter<T> {
                 }
                 return result;
             } else if (this.isForce()) {
-                logger.warn(CLUSTER_CONDITIONAL_ROUTE_LIST_EMPTY,"execute condition state router result list is empty. and force=true","","The route result is empty and force execute. consumer: " + NetUtils.getLocalHost() + ", service: " + url.getServiceKey() + ", router: " + url.getParameterAndDecoded(RULE_KEY));
+                logger.warn(
+                        CLUSTER_CONDITIONAL_ROUTE_LIST_EMPTY,
+                        "execute condition state router result list is empty. and force=true",
+                        "",
+                        "The route result is empty and force execute. consumer: " + NetUtils.getLocalHost()
+                                + ", service: " + url.getServiceKey() + ", router: "
+                                + url.getParameterAndDecoded(RULE_KEY));
                 if (needToPrintMessage) {
                     messageHolder.set("Empty return. Reason: Empty result from condition and condition is force.");
                 }
                 return result;
             }
         } catch (Throwable t) {
-            logger.error(CLUSTER_FAILED_EXEC_CONDITION_ROUTER,"execute condition state router exception","","Failed to execute condition router rule: " + getUrl() + ", invokers: " + invokers + ", cause: " + t.getMessage(),t);
+            logger.error(
+                    CLUSTER_FAILED_EXEC_CONDITION_ROUTER,
+                    "execute condition state router exception",
+                    "",
+                    "Failed to execute condition router rule: " + getUrl() + ", invokers: " + invokers + ", cause: "
+                            + t.getMessage(),
+                    t);
         }
         if (needToPrintMessage) {
             messageHolder.set("Directly return. Reason: Error occurred ( or result is empty ).");
@@ -247,7 +281,7 @@ public class ConditionStateRouter<T> extends AbstractStateRouter<T> {
     @Override
     public boolean isRuntime() {
         // We always return true for previously defined Router, that is, old Router doesn't support cache anymore.
-//        return true;
+        //        return true;
         return this.getUrl().getParameter(RUNTIME_KEY, false);
     }
 
@@ -257,7 +291,10 @@ public class ConditionStateRouter<T> extends AbstractStateRouter<T> {
                 return factory.createMatcher(key, moduleModel);
             }
         }
-        return moduleModel.getExtensionLoader(ConditionMatcherFactory.class).getExtension("param").createMatcher(key, moduleModel);
+        return moduleModel
+                .getExtensionLoader(ConditionMatcherFactory.class)
+                .getExtension("param")
+                .createMatcher(key, moduleModel);
     }
 
     boolean matchWhen(URL url, Invocation invocation) {
@@ -276,7 +313,12 @@ public class ConditionStateRouter<T> extends AbstractStateRouter<T> {
         return doMatch(url, param, null, thenCondition, false);
     }
 
-    private boolean doMatch(URL url, URL param, Invocation invocation, Map<String, ConditionMatcher> conditions, boolean isWhenCondition) {
+    private boolean doMatch(
+            URL url,
+            URL param,
+            Invocation invocation,
+            Map<String, ConditionMatcher> conditions,
+            boolean isWhenCondition) {
         Map<String, String> sample = url.toOriginalMap();
         for (Map.Entry<String, ConditionMatcher> entry : conditions.entrySet()) {
             ConditionMatcher matchPair = entry.getValue();

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/condition/ConditionStateRouterFactory.java

@@ -30,6 +30,6 @@ public class ConditionStateRouterFactory extends CacheableStateRouterFactory {
 
     @Override
     protected <T> StateRouter<T> createRouter(Class<T> interfaceClass, URL url) {
-        return new ConditionStateRouter<T>(url);
+        return new ConditionStateRouter<>(url);
     }
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/condition/config/ListenableStateRouter.java

@@ -50,7 +50,8 @@ public abstract class ListenableStateRouter<T> extends AbstractStateRouter<T> im
     public static final String NAME = "LISTENABLE_ROUTER";
     public static final String RULE_SUFFIX = ".condition-router";
 
-    private static final ErrorTypeAwareLogger logger = LoggerFactory.getErrorTypeAwareLogger(ListenableStateRouter.class);
+    private static final ErrorTypeAwareLogger logger =
+            LoggerFactory.getErrorTypeAwareLogger(ListenableStateRouter.class);
     private volatile ConditionRouterRule routerRule;
     private volatile List<ConditionStateRouter<T>> conditionRouters = Collections.emptyList();
     private final String ruleKey;
@@ -65,8 +66,8 @@ public abstract class ListenableStateRouter<T> extends AbstractStateRouter<T> im
     @Override
     public synchronized void process(ConfigChangedEvent event) {
         if (logger.isInfoEnabled()) {
-            logger.info("Notification of condition rule, change type is: " + event.getChangeType() +
+            logger.info("Notification of condition rule, change type is: " + event.getChangeType() + ", raw rule is:\n "
-                    ", raw rule is:\n " + event.getContent());
+                    + event.getContent());
         }
 
         if (event.getChangeType().equals(ConfigChangeType.DELETED)) {
@@ -77,18 +78,31 @@ public abstract class ListenableStateRouter<T> extends AbstractStateRouter<T> im
                 routerRule = ConditionRuleParser.parse(event.getContent());
                 generateConditions(routerRule);
             } catch (Exception e) {
-                logger.error(CLUSTER_FAILED_RULE_PARSING,"Failed to parse the raw condition rule","","Failed to parse the raw condition rule and it will not take effect, please check " +
+                logger.error(
-                    "if the condition rule matches with the template, the raw rule is:\n " + event.getContent(),e);
+                        CLUSTER_FAILED_RULE_PARSING,
+                        "Failed to parse the raw condition rule",
+                        "",
+                        "Failed to parse the raw condition rule and it will not take effect, please check "
+                                + "if the condition rule matches with the template, the raw rule is:\n "
+                                + event.getContent(),
+                        e);
             }
         }
     }
 
     @Override
-    public BitList<Invoker<T>> doRoute(BitList<Invoker<T>> invokers, URL url, Invocation invocation,
+    public BitList<Invoker<T>> doRoute(
-                                       boolean needToPrintMessage, Holder<RouterSnapshotNode<T>> nodeHolder, Holder<String> messageHolder) throws RpcException {
+            BitList<Invoker<T>> invokers,
+            URL url,
+            Invocation invocation,
+            boolean needToPrintMessage,
+            Holder<RouterSnapshotNode<T>> nodeHolder,
+            Holder<String> messageHolder)
+            throws RpcException {
         if (CollectionUtils.isEmpty(invokers) || conditionRouters.size() == 0) {
             if (needToPrintMessage) {
-                messageHolder.set("Directly return. Reason: Invokers from previous router is empty or conditionRouters is empty.");
+                messageHolder.set(
+                        "Directly return. Reason: Invokers from previous router is empty or conditionRouters is empty.");
             }
             return invokers;
         }
@@ -123,9 +137,9 @@ public abstract class ListenableStateRouter<T> extends AbstractStateRouter<T> im
 
     private void generateConditions(ConditionRouterRule rule) {
         if (rule != null && rule.isValid()) {
-            this.conditionRouters = rule.getConditions()
+            this.conditionRouters = rule.getConditions().stream()
-                    .stream()
+                    .map(condition ->
-                    .map(condition -> new ConditionStateRouter<T>(getUrl(), condition, rule.isForce(), rule.isEnabled()))
+                            new ConditionStateRouter<T>(getUrl(), condition, rule.isForce(), rule.isEnabled()))
                     .collect(Collectors.toList());
             for (ConditionStateRouter<T> conditionRouter : this.conditionRouters) {
                 conditionRouter.setNextRouter(TailStateRouter.getInstance());

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/condition/config/ProviderAppStateRouter.java

@@ -33,7 +33,8 @@ import static org.apache.dubbo.common.utils.StringUtils.isEmpty;
  * Application level router, "application.condition-router"
  */
 public class ProviderAppStateRouter<T> extends ListenableStateRouter<T> {
-    private static final ErrorTypeAwareLogger logger = LoggerFactory.getErrorTypeAwareLogger(ListenableStateRouter.class);
+    private static final ErrorTypeAwareLogger logger =
+            LoggerFactory.getErrorTypeAwareLogger(ListenableStateRouter.class);
     public static final String NAME = "PROVIDER_APP_ROUTER";
     private String application;
     private final String currentApplication;
@@ -55,7 +56,11 @@ public class ProviderAppStateRouter<T> extends ListenableStateRouter<T> {
 
         // provider application is empty or equals with the current application
         if (isEmpty(providerApplication)) {
-            logger.warn(CLUSTER_TAG_ROUTE_EMPTY, "condition router get providerApplication is empty, will not subscribe to provider app rules.", "", "");
+            logger.warn(
+                    CLUSTER_TAG_ROUTE_EMPTY,
+                    "condition router get providerApplication is empty, will not subscribe to provider app rules.",
+                    "",
+                    "");
             return;
         }
         if (providerApplication.equals(currentApplication)) {

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/condition/config/ServiceStateRouterFactory.java

@@ -32,7 +32,6 @@ public class ServiceStateRouterFactory extends CacheableStateRouterFactory {
 
     @Override
     protected <T> StateRouter<T> createRouter(Class<T> interfaceClass, URL url) {
-        return new ServiceStateRouter<T>(url);
+        return new ServiceStateRouter<>(url);
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/condition/config/model/ConditionRouterRule.java

@@ -34,15 +34,14 @@ public class ConditionRouterRule extends AbstractRouterRule {
 
         Object conditions = map.get(CONDITIONS_KEY);
         if (conditions != null && List.class.isAssignableFrom(conditions.getClass())) {
-            conditionRouterRule.setConditions(((List<Object>) conditions).stream()
+            conditionRouterRule.setConditions(
-                    .map(String::valueOf).collect(Collectors.toList()));
+                    ((List<Object>) conditions).stream().map(String::valueOf).collect(Collectors.toList()));
         }
 
         return conditionRouterRule;
     }
 
-    public ConditionRouterRule() {
+    public ConditionRouterRule() {}
-    }
 
     public List<String> getConditions() {
         return conditions;

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/condition/config/model/ConditionRuleParser.java

@@ -18,12 +18,12 @@ package org.apache.dubbo.rpc.cluster.router.condition.config.model;
 
 import org.apache.dubbo.common.utils.CollectionUtils;
 
+import java.util.Map;
+
 import org.yaml.snakeyaml.LoaderOptions;
 import org.yaml.snakeyaml.Yaml;
 import org.yaml.snakeyaml.constructor.SafeConstructor;
 
-import java.util.Map;
-
 /**
  * %YAML1.2
  *
@@ -51,5 +51,4 @@ public class ConditionRuleParser {
 
         return rule;
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/condition/matcher/AbstractConditionMatcher.java

@@ -38,7 +38,8 @@ import static org.apache.dubbo.common.constants.LoggerCodeConstants.CLUSTER_FAIL
  * provides the common match logics.
  */
 public abstract class AbstractConditionMatcher implements ConditionMatcher {
-    public static final ErrorTypeAwareLogger logger = LoggerFactory.getErrorTypeAwareLogger(AbstractConditionMatcher.class);
+    public static final ErrorTypeAwareLogger logger =
+            LoggerFactory.getErrorTypeAwareLogger(AbstractConditionMatcher.class);
     public static final String DOES_NOT_FOUND_VALUE = "dubbo_internal_not_found_argument_condition_value";
     final Set<String> matches = new HashSet<>();
     final Set<String> mismatches = new HashSet<>();
@@ -52,9 +53,10 @@ public abstract class AbstractConditionMatcher implements ConditionMatcher {
         this.valueMatchers = model.getExtensionLoader(ValuePattern.class).getActivateExtensions();
     }
 
-    public static String getSampleValueFromUrl(String conditionKey, Map<String, String> sample, URL param, Invocation invocation) {
+    public static String getSampleValueFromUrl(
+            String conditionKey, Map<String, String> sample, URL param, Invocation invocation) {
         String sampleValue;
-        //get real invoked method name from invocation
+        // get real invoked method name from invocation
         if (invocation != null && (METHOD_KEY.equals(conditionKey) || METHODS_KEY.equals(conditionKey))) {
             sampleValue = RpcUtils.getMethodName(invocation);
         } else {
@@ -67,7 +69,8 @@ public abstract class AbstractConditionMatcher implements ConditionMatcher {
     public boolean isMatch(Map<String, String> sample, URL param, Invocation invocation, boolean isWhenCondition) {
         String value = getValue(sample, param, invocation);
         if (value == null) {
-            // if key does not present in whichever of url, invocation or attachment based on the matcher type, then return false.
+            // if key does not present in whichever of url, invocation or attachment based on the matcher type, then
+            // return false.
             return false;
         }
 
@@ -90,7 +93,7 @@ public abstract class AbstractConditionMatcher implements ConditionMatcher {
         }
 
         if (!matches.isEmpty() && !mismatches.isEmpty()) {
-            //when both mismatches and matches contain the same value, then using mismatches first
+            // when both mismatches and matches contain the same value, then using mismatches first
             for (String mismatch : mismatches) {
                 if (doPatternMatch(mismatch, value, param, invocation, isWhenCondition)) {
                     return false;
@@ -117,16 +120,23 @@ public abstract class AbstractConditionMatcher implements ConditionMatcher {
     }
 
     // range, equal or other methods
-    protected boolean doPatternMatch(String pattern, String value, URL url, Invocation invocation, boolean isWhenCondition) {
+    protected boolean doPatternMatch(
+            String pattern, String value, URL url, Invocation invocation, boolean isWhenCondition) {
         for (ValuePattern valueMatcher : valueMatchers) {
             if (valueMatcher.shouldMatch(pattern)) {
                 return valueMatcher.match(pattern, value, url, invocation, isWhenCondition);
             }
         }
         // this should never happen.
-        logger.error(CLUSTER_FAILED_EXEC_CONDITION_ROUTER, "Executing condition rule value match expression error.", "pattern is " + pattern + ", value is " + value + ", condition type " + (isWhenCondition ? "when" : "then"), "There should at least has one ValueMatcher instance that applies to all patterns, will force to use wildcard matcher now.");
+        logger.error(
+                CLUSTER_FAILED_EXEC_CONDITION_ROUTER,
+                "Executing condition rule value match expression error.",
+                "pattern is " + pattern + ", value is " + value + ", condition type "
+                        + (isWhenCondition ? "when" : "then"),
+                "There should at least has one ValueMatcher instance that applies to all patterns, will force to use wildcard matcher now.");
 
-        ValuePattern paramValueMatcher = model.getExtensionLoader(ValuePattern.class).getExtension("wildcard");
+        ValuePattern paramValueMatcher =
+                model.getExtensionLoader(ValuePattern.class).getExtension("wildcard");
         return paramValueMatcher.match(pattern, value, url, invocation, isWhenCondition);
     }
 
@@ -135,5 +145,4 @@ public abstract class AbstractConditionMatcher implements ConditionMatcher {
      * This makes condition rule possible to check values in any place of a request.
      */
     protected abstract String getValue(Map<String, String> sample, URL url, Invocation invocation);
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/condition/matcher/argument/ArgumentConditionMatcher.java

@@ -38,7 +38,8 @@ import static org.apache.dubbo.common.constants.LoggerCodeConstants.CLUSTER_FAIL
  */
 @Activate
 public class ArgumentConditionMatcher extends AbstractConditionMatcher {
-    private static final ErrorTypeAwareLogger logger = LoggerFactory.getErrorTypeAwareLogger(ArgumentConditionMatcher.class);
+    private static final ErrorTypeAwareLogger logger =
+            LoggerFactory.getErrorTypeAwareLogger(ArgumentConditionMatcher.class);
     private static final Pattern ARGUMENTS_PATTERN = Pattern.compile("arguments\\[([0-9]+)\\]");
 
     public ArgumentConditionMatcher(String key, ModuleModel model) {
@@ -56,16 +57,21 @@ public class ArgumentConditionMatcher extends AbstractConditionMatcher {
                 return DOES_NOT_FOUND_VALUE;
             }
 
-            //extract the argument index
+            // extract the argument index
             int index = Integer.parseInt(matcher.group(1));
             if (index < 0 || index > invocation.getArguments().length) {
                 return DOES_NOT_FOUND_VALUE;
             }
 
-            //extract the argument value
+            // extract the argument value
             return String.valueOf(invocation.getArguments()[index]);
         } catch (Exception e) {
-            logger.warn(CLUSTER_FAILED_EXEC_CONDITION_ROUTER, "Parse argument match condition failed", "", "Invalid , will ignore., ", e);
+            logger.warn(
+                    CLUSTER_FAILED_EXEC_CONDITION_ROUTER,
+                    "Parse argument match condition failed",
+                    "",
+                    "Invalid , will ignore., ",
+                    e);
         }
         return DOES_NOT_FOUND_VALUE;
     }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/condition/matcher/attachment/AttachmentConditionMatcher.java

@@ -38,7 +38,8 @@ import static org.apache.dubbo.common.constants.LoggerCodeConstants.CLUSTER_FAIL
  */
 @Activate
 public class AttachmentConditionMatcher extends AbstractConditionMatcher {
-    private static final ErrorTypeAwareLogger logger = LoggerFactory.getErrorTypeAwareLogger(AttachmentConditionMatcher.class);
+    private static final ErrorTypeAwareLogger logger =
+            LoggerFactory.getErrorTypeAwareLogger(AttachmentConditionMatcher.class);
     private static final Pattern ATTACHMENTS_PATTERN = Pattern.compile("attachments\\[(.+)\\]");
 
     public AttachmentConditionMatcher(String key, ModuleModel model) {
@@ -56,20 +57,21 @@ public class AttachmentConditionMatcher extends AbstractConditionMatcher {
                 return DOES_NOT_FOUND_VALUE;
             }
 
-            //extract the argument index
+            // extract the argument index
             String attachmentKey = matcher.group(1);
             if (StringUtils.isEmpty(attachmentKey)) {
                 return DOES_NOT_FOUND_VALUE;
             }
 
-            //extract the argument value
+            // extract the argument value
             return invocation.getAttachment(attachmentKey);
         } catch (Exception e) {
-            logger.warn(CLUSTER_FAILED_EXEC_CONDITION_ROUTER,
+            logger.warn(
-                "condition state router attachment match failed",
+                    CLUSTER_FAILED_EXEC_CONDITION_ROUTER,
-                "",
+                    "condition state router attachment match failed",
-                "Invalid match condition: " + key,
+                    "",
-                e);
+                    "Invalid match condition: " + key,
+                    e);
         }
         return DOES_NOT_FOUND_VALUE;
     }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/condition/matcher/param/UrlParamConditionMatcher.java

@@ -37,5 +37,4 @@ public class UrlParamConditionMatcher extends AbstractConditionMatcher {
     protected String getValue(Map<String, String> sample, URL url, Invocation invocation) {
         return getSampleValueFromUrl(key, sample, url, invocation);
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/condition/matcher/pattern/range/RangeValuePattern.java

@@ -46,7 +46,11 @@ public class RangeValuePattern implements ValuePattern {
 
             String[] arr = pattern.split("~");
             if (arr.length < 2) {
-                logger.error(CLUSTER_FAILED_EXEC_CONDITION_ROUTER, "", "", "Invalid condition rule " + pattern + " or value " + value + ", will ignore.");
+                logger.error(
+                        CLUSTER_FAILED_EXEC_CONDITION_ROUTER,
+                        "",
+                        "",
+                        "Invalid condition rule " + pattern + " or value " + value + ", will ignore.");
                 return defaultValue;
             }
 
@@ -75,7 +79,12 @@ public class RangeValuePattern implements ValuePattern {
                 }
             }
         } catch (Exception e) {
-            logger.error(CLUSTER_FAILED_EXEC_CONDITION_ROUTER, "Parse integer error", "", "Invalid condition rule " + pattern + " or value " + value + ", will ignore.", e);
+            logger.error(
+                    CLUSTER_FAILED_EXEC_CONDITION_ROUTER,
+                    "Parse integer error",
+                    "",
+                    "Invalid condition rule " + pattern + " or value " + value + ", will ignore.",
+                    e);
             return defaultValue;
         }
 

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/file/FileStateRouterFactory.java

@@ -46,7 +46,9 @@ public class FileStateRouterFactory implements StateRouterFactory {
         try {
             // Transform File URL into Script Route URL, and Load
             // file:///d:/path/to/route.js?router=script ==> script:///d:/path/to/route.js?type=js&rule=<file-content>
-            String protocol = url.getParameter(ROUTER_KEY, ScriptStateRouterFactory.NAME); // Replace original protocol (maybe 'file') with 'script'
+            String protocol = url.getParameter(
+                    ROUTER_KEY,
+                    ScriptStateRouterFactory.NAME); // Replace original protocol (maybe 'file') with 'script'
             String type = null; // Use file suffix to config script type, e.g., js, groovy ...
             String path = url.getPath();
             if (path != null) {
@@ -71,5 +73,4 @@ public class FileStateRouterFactory implements StateRouterFactory {
             throw new IllegalStateException(e.getMessage(), e);
         }
     }
-
 }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/mesh/route/MeshAppRuleListener.java

@@ -14,7 +14,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.apache.dubbo.rpc.cluster.router.mesh.route;
 
 import org.apache.dubbo.common.config.configcenter.ConfigChangeType;
@@ -26,24 +25,23 @@ import org.apache.dubbo.common.utils.CollectionUtils;
 import org.apache.dubbo.rpc.cluster.router.mesh.util.MeshRuleDispatcher;
 import org.apache.dubbo.rpc.cluster.router.mesh.util.MeshRuleListener;
 
-import org.yaml.snakeyaml.DumperOptions;
-import org.yaml.snakeyaml.LoaderOptions;
-import org.yaml.snakeyaml.Yaml;
-import org.yaml.snakeyaml.constructor.SafeConstructor;
-import org.yaml.snakeyaml.representer.Representer;
-
 import java.text.MessageFormat;
 import java.util.HashMap;
 import java.util.LinkedList;
 import java.util.List;
 import java.util.Map;
 
+import org.yaml.snakeyaml.DumperOptions;
+import org.yaml.snakeyaml.LoaderOptions;
+import org.yaml.snakeyaml.Yaml;
+import org.yaml.snakeyaml.constructor.SafeConstructor;
+import org.yaml.snakeyaml.representer.Representer;
+
 import static org.apache.dubbo.common.constants.LoggerCodeConstants.CLUSTER_FAILED_RECEIVE_RULE;
 import static org.apache.dubbo.rpc.cluster.router.mesh.route.MeshRuleConstants.METADATA_KEY;
 import static org.apache.dubbo.rpc.cluster.router.mesh.route.MeshRuleConstants.NAME_KEY;
 import static org.apache.dubbo.rpc.cluster.router.mesh.route.MeshRuleConstants.STANDARD_ROUTER_KEY;
 
-
 public class MeshAppRuleListener implements ConfigurationListener {
 
     private static final ErrorTypeAwareLogger logger = LoggerFactory.getErrorTypeAwareLogger(MeshAppRuleListener.class);
@@ -62,8 +60,7 @@ public class MeshAppRuleListener implements ConfigurationListener {
     @SuppressWarnings("unchecked")
     public void receiveConfigInfo(String configInfo) {
         if (logger.isDebugEnabled()) {
-            logger.debug(MessageFormat.format("[MeshAppRule] Received rule for app [{0}]: {1}.",
+            logger.debug(MessageFormat.format("[MeshAppRule] Received rule for app [{0}]: {1}.", appName, configInfo));
-                appName, configInfo));
         }
         try {
             Map<String, List<Map<String, Object>>> groupMap = new HashMap<>();
@@ -79,19 +76,34 @@ public class MeshAppRuleListener implements ConfigurationListener {
 
                     String ruleType = computeRuleType(resultMap);
                     if (ruleType != null) {
-                        groupMap.computeIfAbsent(ruleType, (k)-> new LinkedList<>()).add(resultMap);
+                        groupMap.computeIfAbsent(ruleType, (k) -> new LinkedList<>())
+                                .add(resultMap);
                     } else {
-                        logger.error(CLUSTER_FAILED_RECEIVE_RULE,"receive mesh app route rule is invalid","","Unable to get rule type from raw rule. " +
+                        logger.error(
-                            "Probably the metadata.name is absent. App Name: " + appName + " RawRule: " + configInfo);
+                                CLUSTER_FAILED_RECEIVE_RULE,
+                                "receive mesh app route rule is invalid",
+                                "",
+                                "Unable to get rule type from raw rule. "
+                                        + "Probably the metadata.name is absent. App Name: " + appName + " RawRule: "
+                                        + configInfo);
                     }
                 } else {
-                    logger.error(CLUSTER_FAILED_RECEIVE_RULE,"receive mesh app route rule is invalid","","Rule format is unacceptable. App Name: " + appName + " RawRule: " + configInfo);
+                    logger.error(
+                            CLUSTER_FAILED_RECEIVE_RULE,
+                            "receive mesh app route rule is invalid",
+                            "",
+                            "Rule format is unacceptable. App Name: " + appName + " RawRule: " + configInfo);
                 }
             }
 
             ruleMapHolder = groupMap;
         } catch (Exception e) {
-            logger.error(CLUSTER_FAILED_RECEIVE_RULE,"failed to receive mesh app route rule","","[MeshAppRule] parse failed: " + configInfo,e);
+            logger.error(
+                    CLUSTER_FAILED_RECEIVE_RULE,
+                    "failed to receive mesh app route rule",
+                    "",
+                    "[MeshAppRule] parse failed: " + configInfo,
+                    e);
         }
         if (ruleMapHolder != null) {
             meshRuleDispatcher.post(ruleMapHolder);
@@ -123,7 +135,6 @@ public class MeshAppRuleListener implements ConfigurationListener {
         meshRuleDispatcher.register(subscriber);
     }
 
-
     public <T> void unregister(MeshRuleListener subscriber) {
         meshRuleDispatcher.unregister(subscriber);
     }

dubbo-cluster/src/main/java/org/apache/dubbo/rpc/cluster/router/mesh/route/MeshEnvListener.java

@@ -14,7 +14,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 package org.apache.dubbo.rpc.cluster.router.mesh.route;
 
 /**