-
Notifications
You must be signed in to change notification settings - Fork 69
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RFC: Introduce HTTP API V2 in PD #88
Open
rleungx
wants to merge
3
commits into
tikv:master
Choose a base branch
from
rleungx:pd-apiv2
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
3 commits
Select commit
Hold shift + click to select a range
File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,72 @@ | ||
# PD HTTP API V2 | ||
|
||
## Motivation | ||
|
||
The V2 API aims to provide a standard RESTful API for PD. The current HTTP API doesn't follow the RESTful API conventions. We have summarized some problems of the current status: | ||
|
||
- mixed-use of `_` and `-` in the path | ||
- some methods are not accurate, e.g., we do not distinguish between PUT and POST | ||
- the resource should use a noun | ||
- query parameter should not occur in the path | ||
- mixed-use of singular and plural nouns | ||
|
||
Besides the above problems, we are going to refine some internal structures and reduce some unnecessary APIs which are never been used before. | ||
|
||
## Detailed Design | ||
|
||
We can use gin as our HTTP web framework in the V2 API, which is more popular and has a better ecosystem. | ||
|
||
Here is a basic example for stores: | ||
|
||
```go | ||
router := gin.New() | ||
router.Use(middlewares.Redirector()) | ||
root := router.Group(apiV2Prefix) | ||
meta := root.Group("meta") | ||
meta.Use(middlewares.BootstrapChecker()) | ||
meta.GET("/stores", handlers.GetStores()) | ||
meta.GET("/stores/:id", handlers.GetStoreByID()) | ||
meta.DELETE("/stores/:id", handlers.DeleteStoreByID()) | ||
meta.PATCH("/stores/:id", handlers.UpdateStoreByID()) | ||
``` | ||
|
||
### Middleware | ||
|
||
We can easily add the middleware through the `Use` function and the middleware we defined only needs to return [gin.HandlerFunc](https://github.com/gin-gonic/gin/blob/v1.7.7/gin.go#L34). This also can be applied in a specified API group. Gin has provided many [ready-made middleware](https://github.com/gin-contrib) which is convenient. | ||
|
||
### Path definition | ||
|
||
#### Resource | ||
|
||
Since the key abstraction of information in RESTful API is a resource, it can be any non-virtual object. We use plural form to represent a set of objects, e.g., `/pd/api/v2/stores` and use the plural form with a unique field to represent a singular object, e.g., `/pd/api/v2/stores/:id`. There should not be a verb or any query parameter existing in the path. The previous path like `/pd/api/v1/store/{id}/state` should be rewritten to `/pd/api/v2/stores/:id` with `state` data in JSON format as an input. | ||
|
||
#### Method | ||
|
||
Here is a simple guide about how to use choose the correct method: | ||
|
||
- GET: retrieve resources only | ||
- POST: create resources or do a custom action. We should avoid using it on a single resource for a creating purpose. | ||
- PUT: replace resources or collections | ||
- PATCH: make a partial update on a resource | ||
- DELETE: delete the resources | ||
|
||
#### Word delimiter | ||
|
||
The hyphen will be recommended to use as the word delimiter. | ||
|
||
#### Group | ||
|
||
For those middlewares that only need to be applied in a set of APIs, we can use API group. API group also can be used to divide our APIs into different sets according to their purpose and share the same middleware. The previous `/pd/api/v1/admin/*` can be put into `admin` group. For stores itself, we can put them into `meta` group, so the previous `/pd/api/v1/stores` or `/pd/api/v1/store` become to `/pd/api/v2/meta/stores`. Currently, the V1 API can be divided into the following groups: | ||
|
||
- member: something about etcd itself | ||
- meta: those resources related to PD meta info, the entity in PD core package, like regions, stores, cluster | ||
- scheduling: use to control the scheduling behaviors, e.g., schedulers, checkers, operators | ||
- admin: control the system behavior, e.g., log, ping | ||
- debug: debug information through go pprof | ||
- extension: some extra features required by other components | ||
|
||
There are some other APIs that don't belong to any group mentioned above. We can decide to use an individual group according to middleware usage. | ||
|
||
#### Custom action | ||
|
||
We have many custom actions in API V1, such as `/pd/api/v1/regions/split` or `/pd/api/v1/regions/scatter`, etc. In V2, We recommend using `/pd/api/v2/regions/:action` with `POST` method. There may be [segment conflicts with existing wildcard](https://github.com/gin-gonic/gin/issues/1301) when using gin as the web framework. But fortunately, we don't have this conflict problem after we change existed V1 API to V2. |
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we introduce the store state refine? It's may influence downstream.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I will add another section to introduce some refinement of the internal structure.