Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[WIP] Track the progress of PIP-149-Making the REST Admin API fully async #15043

Closed
26 tasks done
mattisonchao opened this issue Apr 6, 2022 · 12 comments
Closed
26 tasks done

[WIP] Track the progress of PIP-149-Making the REST Admin API fully async #15043

mattisonchao opened this issue Apr 6, 2022 · 12 comments
Assignees
@Technoboy- @mattisonchao
Labels
Stale

Comments

@mattisonchao
Copy link
Member

mattisonchao commented Apr 6, 2022 •
edited by Technoboy-
Loading

Motivation

This issue is used to track the progress of PIP-149 Making the REST Admin API fully async.

We should check this PIP first to know our purpose and then push PR to refactor.

Each group, consisting of 2~4 methods, referring to the rules of PIP-149 to submit only one patch to fix.

Progress

BrokersBase

ClustersBase

Namespaces

Tenants

ResourceQuotas

SchemasResource
@Technoboy- Technoboy- changed the title [WIP] Track the progress of PIP-149( Making the REST Admin API fully async ) [WIP] Track the progress of PIP-149-Making the REST Admin API fully async Apr 6, 2022
@sijie sijie mentioned this issue Apr 6, 2022
Open
26 tasks
This was referenced Apr 23, 2022
Merged
Merged
Merged
Merged
@mattisonchao mattisonchao mentioned this issue May 4, 2022
Merged
2 tasks
@Technoboy- Technoboy- mentioned this issue May 10, 2022
Merged
1 task
@mattisonchao mattisonchao mentioned this issue May 10, 2022
Merged
2 tasks
@shibd shibd mentioned this issue May 13, 2022
Merged
1 task
codelipenghui pushed a commit that referenced this issue May 14, 2022
@Technoboy-
[improve][broker] Make some methods in NamespacesBase async. (#15518)
138ea35 
### Motivation
See PIP #14365  and change tracker #15043. 

 Make `NamespacesBase` `getTenantNamespaces / createNamespace / getTopics / getPolicies / getPermissions` methods to pure async.
This was referenced May 14, 2022
Merged
Merged
This was referenced May 15, 2022
Merged
Merged
@gaoyanliang
Copy link
Contributor

I'll take care of （Namespaces - deleteSubscribeRate / setSubscribeRate / getSubscribeRate）, please assign to me

@gaoyanliang gaoyanliang mentioned this issue May 18, 2022
Merged
1 task
@mattisonchao mattisonchao mentioned this issue May 20, 2022
Merged
2 tasks
@shibd shibd mentioned this issue May 24, 2022
Merged
1 task
@Technoboy- Technoboy- mentioned this issue May 28, 2022
Merged
1 task
@gaoyanliang
Copy link
Contributor

image

please assign these tasks to me .

@mattisonchao mattisonchao mentioned this issue May 30, 2022
Merged
2 tasks
@shibd shibd mentioned this issue May 31, 2022
Closed
5 tasks
@leizhiyuan
Copy link
Contributor

@gaoyanliang

maybe I can help for

setSubscriptionDispatchRate / getSubscriptionDispatchRate / deleteSubscriptionDispatchRate @gaoyanliang

@gaoyanliang
Copy link
Contributor

gaoyanliang commented Jun 1, 2022 •
edited
Loading

@leizhiyuan This task will be completed soon and a review will be launched later. maybe you can help for
image

@zhangqian321
Copy link
Contributor

我可以帮助完成 removePublishRate / getPublishRate / getDispatchRate / setDispatchRate / deleteDispatchRate @leizhiyuan @Technoboy- @gaoyanliang

@Technoboy-
Copy link
Contributor

我可以帮助完成 removePublishRate / getPublishRate / getDispatchRate / setDispatchRate / deleteDispatchRate @leizhiyuan @Technoboy- @gaoyanliang

you can choose one that is not assigned.

@gaoyanliang gaoyanliang mentioned this issue Jun 1, 2022
Merged
1 task
@gaoyanliang
Copy link
Contributor

setSubscriptionDispatchRate / getSubscriptionDispatchRate / deleteSubscriptionDispatchRate this task has been completed, and review request has been launched. @Technoboy- @leizhiyuan @zqcece

Please @leizhiyuan @zqcece help to complete the following tasks:

  • splitNamespaceBundle / getTopicHashPositions / setPublishRate
  • removePublishRate / getPublishRate / getDispatchRate / setDispatchRate / deleteDispatchRate

congbobo184 pushed a commit that referenced this issue Jun 2, 2022
@Technoboy-
[improve][broker] Make some methods in SchemasResourceBase async. (#1…
8bcb921 
…5821)

### Motivation
See PIP #14365  and change tracker #15043. 

 Make `getSchema` /  `deleteSchema` / `postSchema` / `getVersionBySchema` / `testCompatibility` methods in SchemasResourceBase to pure async.
@leizhiyuan
Copy link
Contributor

setSubscriptionDispatchRate / getSubscriptionDispatchRate / deleteSubscriptionDispatchRate this task has been completed, and review request has been launched. @Technoboy- @leizhiyuan @zqcece

Please @leizhiyuan @zqcece help to complete the following tasks:

  • splitNamespaceBundle / getTopicHashPositions / setPublishRate
  • removePublishRate / getPublishRate / getDispatchRate / setDispatchRate / deleteDispatchRate

ok

I will work for splitNamespaceBundle / getTopicHashPositions / setPublishRate

@leizhiyuan leizhiyuan mentioned this issue Jun 2, 2022
Closed
5 tasks
@zhangqian321
Copy link
Contributor

setSubscriptionDispatchRate / getSubscriptionDispatchRate / deleteSubscriptionDispatchRate this task has been completed, and review request has been launched. @Technoboy- @leizhiyuan @zqcece

Please @leizhiyuan @zqcece help to complete the following tasks:

  • splitNamespaceBundle / getTopicHashPositions / setPublishRate
  • removePublishRate / getPublishRate / getDispatchRate / setDispatchRate / deleteDispatchRate

thx~ i will work for removePublishRate / getPublishRate / getDispatchRate / setDispatchRate / deleteDispatchRate

@leizhiyuan
Copy link
Contributor

setSubscriptionDispatchRate / getSubscriptionDispatchRate / deleteSubscriptionDispatchRate this task has been completed, and review request has been launched. @Technoboy- @leizhiyuan @zqcece
Please @leizhiyuan @zqcece help to complete the following tasks:

  • splitNamespaceBundle / getTopicHashPositions / setPublishRate
  • removePublishRate / getPublishRate / getDispatchRate / setDispatchRate / deleteDispatchRate

thx~ i will work for removePublishRate / getPublishRate / getDispatchRate / setDispatchRate / deleteDispatchRate

setPublishRate is same with these work, you can do it together

@leizhiyuan leizhiyuan mentioned this issue Jun 2, 2022
Closed
5 tasks
@mattisonchao mattisonchao mentioned this issue Jun 3, 2022
Closed
2 tasks
@zhangqian321 zhangqian321 mentioned this issue Jun 6, 2022
Merged
1 task
@Technoboy- Technoboy- mentioned this issue Jul 1, 2022
Merged
1 task
This was referenced Jul 1, 2022
Merged
Merged
Merged
Merged
@HQebupt HQebupt mentioned this issue Jul 14, 2022
Closed
1 task
@github-actions
Copy link

github-actions bot commented Aug 1, 2022

The issue had no activity for 30 days, mark with Stale label.

@github-actions github-actions bot added the Stale label Aug 1, 2022
@gaozhangmin gaozhangmin mentioned this issue Aug 23, 2022
Merged
4 tasks
@lhotari lhotari mentioned this issue Dec 21, 2022
Merged
4 tasks
@Technoboy-
Copy link
Contributor

Close as we have done all these trackings

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Technoboy- Technoboy-

@mattisonchao mattisonchao

Labels
Stale
Projects
None yet
Milestone
No milestone
5 participants
@leizhiyuan @Technoboy- @zhangqian321 @gaoyanliang @mattisonchao