Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@KiteSoar
Copy link
Collaborator

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

What is the purpose of the change

it closes #12979

Brief changelog

XX

Verifying this change

XXXX

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

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

@github-actions
Copy link

Thanks for your this PR. 🙏
Please check again for your PR changes whether contains any usage/api/configuration change such as Add new API , Add new configuration, Change default value of configuration.
If so, please add or update documents(markdown type) in docs/next/ for repository nacos-group/nacos-group.github.io


感谢您提交的PR。 🙏
请再次查看您的PR内容,确认是否包含任何使用方式/API/配置参数的变更,如:新增API新增配置参数修改默认配置等操作。
如果是,请确保在提交之前,在仓库nacos-group/nacos-group.github.io中的docs/next/目录下添加或更新文档(markdown格式)。

@KiteSoar KiteSoar changed the title [ISSUE#12979] Develop maintainer sdk. [ISSUE#12979] Develop a maintainer sdk. Jan 22, 2025
console/pom.xml Outdated
</dependency>
<dependency>
<groupId>com.alibaba.nacos</groupId>
<artifactId>maintainer-client</artifactId>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be nacos-maintainer-client?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

console/pom.xml Outdated
<dependency>
<groupId>com.alibaba.nacos</groupId>
<artifactId>maintainer-client</artifactId>
<version>0.0.1-SNAPSHOT</version>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

version follow parent

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done


@Configuration
@ConditionalOnProperty(value = Constants.NACOS_DEPLOYMENT_TYPE, havingValue = Constants.NACOS_DEPLOYMENT_TYPE_CONSOLE)
public class NacosRemoteHandlerConfiguration {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Console相关的改动可以放在之后的PR里,先添加maintainer client,可以按照console 所需要的API先行添加。然后再补充一些基础的

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

* this is only use in nacos, Custom request type, essentially a GET request, Mainly used for GET request parameters
* are relatively large,can not be placed on the URL, so it needs to be placed in the body.
*/
public static final String GET_LARGE = "GET-LARGE";
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个类型现在还在用吗?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maintainer sdk目前没有用到这个

* @author nkorange
* @since 0.8.0
*/
public class HttpMethod {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个基础类是从nacos-client copy出来的吗?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这是common模块copy出来的,现在已经改成依赖common模块了

@alibaba alibaba deleted a comment from Sarahbella4040 Jan 24, 2025
@KiteSoar KiteSoar marked this pull request as ready for review February 5, 2025 04:29
@KomachiSion KomachiSion added the version/3.x Nacos 3.0 Architecture Evolution label Feb 5, 2025
@KomachiSion KomachiSion added this to the 3.0.0 milestone Feb 5, 2025
@KomachiSion KomachiSion merged commit de4a811 into alibaba:v3.0-develop Feb 5, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

version/3.x Nacos 3.0 Architecture Evolution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants