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

Tracking issue for the HTTP SDK #7300

Open
5 of 7 tasks
JmPotato opened this issue Nov 2, 2023 · 0 comments
Open
5 of 7 tasks

Tracking issue for the HTTP SDK #7300

JmPotato opened this issue Nov 2, 2023 · 0 comments
Assignees
@JmPotato @HuSharp
Labels
component/api HTTP API. type/development The issue belongs to a development tasks

Comments

@JmPotato
Copy link
Member

JmPotato commented Nov 2, 2023 •
edited by HuSharp

Development Task

As pingcap/tidb#35319 mentioned, providing a unified PD HTTP API SDK has become an urgent need. With this means, it could be easier for us to control and observe the use of the PD HTTP API in general.
@JmPotato JmPotato added component/api HTTP API. type/development The issue belongs to a development tasks labels Nov 2, 2023
This was referenced Nov 2, 2023
Merged
Merged
Merged
ti-chi-bot bot added a commit that referenced this issue Nov 13, 2023
@JmPotato @ti-chi-bot
client: introduce the HTTP client (#7304)
7dbe607 
ref #7300

Introduce the HTTP client.

Signed-off-by: JmPotato <ghzpotato@gmail.com>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
This was referenced Nov 13, 2023
Merged
Merged
Merged
Merged
ti-chi-bot bot pushed a commit that referenced this issue Nov 17, 2023
@JmPotato
client/http: implement more HTTP APIs (#7371)
dda748a 
ref #7300

- Implement more HTTP APIs.
- Use consts more in `Rule` structure.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
@JmPotato JmPotato mentioned this issue Nov 20, 2023
Merged
ti-chi-bot bot added a commit that referenced this issue Nov 21, 2023
@JmPotato @ti-chi-bot
client/http: encapsulate rule-related PD HTTP interfaces (#7397)
2349f01 
ref #7300

Expand the PD HTTP interfaces to include more encapsulation, particularly focusing on the rule-related interfaces.

Signed-off-by: JmPotato <ghzpotato@gmail.com>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
This was referenced Nov 21, 2023
Merged
Merged
ti-chi-bot bot pushed a commit that referenced this issue Nov 21, 2023
@JmPotato
client/http: pass res interface into respHandler (#7404)
8919bc1 
ref #7300

Pass `res` interface into `respHandler`.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
@lance6716 lance6716 mentioned this issue Nov 21, 2023
Closed
This was referenced Nov 23, 2023
Merged
Merged
ti-chi-bot bot pushed a commit that referenced this issue Nov 27, 2023
@JmPotato
client/http: implement more rule and batch related interfaces (#7430)
58e9b20 
ref #7300

- Implement more rule and batch related interfaces.
- Add more types and methods.
- Refine the tests.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
@JmPotato JmPotato mentioned this issue Nov 27, 2023
Merged
ti-chi-bot bot pushed a commit that referenced this issue Nov 27, 2023
@JmPotato
client/http: use KeyRange to differentiate the encoding format of the…
9f4803d 
… key (#7449)

ref #7300

Use `KeyRange` to differentiate the encoding format of the key.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
This was referenced Nov 27, 2023
Merged
Merged
ti-chi-bot bot pushed a commit that referenced this issue Nov 29, 2023
@JmPotato
client/http: add GetHistoryHotRegions interface and more tests (#7453)
7fbe69d 
ref #7300

- Add `GetHistoryHotRegions` interface.
- Add more tests.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
@JmPotato JmPotato mentioned this issue Nov 29, 2023
Merged
2 tasks
ti-chi-bot bot pushed a commit that referenced this issue Nov 30, 2023
@JmPotato
client/http: implement the marshaler interfaces for Rule/RuleOp (#7462)
862eee1 
ref #7300

Implement the marshaler interfaces for `Rule` and `RuleOP` to make sure we could set/get the correct start/end key.

Ref https://github.com/pingcap/tidb/blob/46d4231c8b0ade353b98572e7c2a015bddf940f4/pkg/ddl/placement/rule.go#L76-L91.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
@JmPotato JmPotato mentioned this issue Dec 4, 2023
Merged
ti-chi-bot bot pushed a commit that referenced this issue Jan 9, 2024
@JmPotato
client/http, retry: support backoffer in HTTP client (#7680)
f18de09 
ref #7300

Support backoffer in HTTP client.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
@JmPotato JmPotato mentioned this issue Jan 9, 2024
Merged
ti-chi-bot bot pushed a commit that referenced this issue Jan 9, 2024
@JmPotato
http/client, retry: copy a new Backoffer for each request (#7684)
dd8df25 
ref #7300

Copy a new `Backoffer` for each request to prevent race.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
@lance6716 lance6716 mentioned this issue Jan 11, 2024
Merged
ti-chi-bot bot pushed a commit that referenced this issue Jan 15, 2024
@lance6716
http: add Admin API and Config with TTL API (#7699)
91bca92 
ref #7300

Signed-off-by: lance6716 <lance6716@gmail.com>
pingandb pushed a commit to pingandb/pd that referenced this issue Jan 18, 2024
@HuSharp @ti-chi-bot @pingandb
gomod: only support one gomod in tools and integrations respectively (t…
09e600e 
…ikv#7635)

ref tikv#7300

nowadays we have too many gomods which are redundant

remove redundant gomods, support only one go mod in integrations/tools/client/server respectively

Signed-off-by: husharp <jinhao.hu@pingcap.com>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Signed-off-by: pingandb <songge102@pingan.com.cn>
pingandb pushed a commit to pingandb/pd that referenced this issue Jan 18, 2024
@lance6716 @ti-chi-bot
@HuSharp @pingandb
http: add GetVersion API (tikv#7648)
9c9a07e 
ref tikv#7300

Signed-off-by: lance6716 <lance6716@gmail.com>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Co-authored-by: Hu# <ihusharp@gmail.com>
Signed-off-by: pingandb <songge102@pingan.com.cn>
pingandb pushed a commit to pingandb/pd that referenced this issue Jan 18, 2024
@HuSharp @ti-chi-bot @pingandb
ctl: replace cluster info with HTTP SDK (tikv#7605)
164e3f4 
ref tikv#7300

Signed-off-by: husharp <jinhao.hu@pingcap.com>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
Signed-off-by: pingandb <songge102@pingan.com.cn>
pingandb pushed a commit to pingandb/pd that referenced this issue Jan 18, 2024
@lance6716 @pingandb
http: add GetEmptyRegions API (tikv#7660)
2f0d6bb 
ref tikv#7300

Signed-off-by: lance6716 <lance6716@gmail.com>
Signed-off-by: pingandb <songge102@pingan.com.cn>
pingandb pushed a commit to pingandb/pd that referenced this issue Jan 18, 2024
@JmPotato @pingandb
client/http: do not retry the 4XX code returned by leader (tikv#7667)
87fab1a 
ref tikv#7300

When the PD leader returns a 404 status code, it no longer continues to retry but returns directly.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
Signed-off-by: pingandb <songge102@pingan.com.cn>
pingandb pushed a commit to pingandb/pd that referenced this issue Jan 18, 2024
@JmPotato @pingandb
client/http, retry: support backoffer in HTTP client (tikv#7680)
d3acab5 
ref tikv#7300

Support backoffer in HTTP client.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
Signed-off-by: pingandb <songge102@pingan.com.cn>
pingandb pushed a commit to pingandb/pd that referenced this issue Jan 18, 2024
@JmPotato @pingandb
http/client, retry: copy a new Backoffer for each request (tikv#7684)
503a22c 
ref tikv#7300

Copy a new `Backoffer` for each request to prevent race.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
Signed-off-by: pingandb <songge102@pingan.com.cn>
pingandb pushed a commit to pingandb/pd that referenced this issue Jan 18, 2024
@lance6716 @pingandb
http: add Admin API and Config with TTL API (tikv#7699)
4f413b1 
ref tikv#7300

Signed-off-by: lance6716 <lance6716@gmail.com>
Signed-off-by: pingandb <songge102@pingan.com.cn>
JmPotato added a commit to JmPotato/pd that referenced this issue Jan 25, 2024
@JmPotato
client/http, retry: support backoffer in HTTP client (tikv#7680)
b94e74e 
ref tikv#7300

Support backoffer in HTTP client.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
JmPotato added a commit to JmPotato/pd that referenced this issue Jan 25, 2024
@JmPotato
http/client, retry: copy a new Backoffer for each request (tikv#7684)
1da8d9d 
ref tikv#7300

Copy a new `Backoffer` for each request to prevent race.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
@JmPotato JmPotato mentioned this issue Feb 5, 2024
Merged
ti-chi-bot bot added a commit that referenced this issue Feb 5, 2024
@JmPotato @ti-chi-bot
pd-ctl: avoid initializing a new PD client for each cmd execution (#7804
ddf810b 
)

ref #7300

Avoid initializing a new PD client for each cmd execution. Now we will only initialize a new one when the cluster changes.

Signed-off-by: JmPotato <ghzpotato@gmail.com>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@HuSharp HuSharp mentioned this issue Feb 21, 2024
Open
@CabinfeverB CabinfeverB mentioned this issue Feb 23, 2024
Open
@JmPotato JmPotato mentioned this issue Mar 6, 2024
Merged
ti-chi-bot bot added a commit that referenced this issue Mar 7, 2024
@JmPotato @ti-chi-bot
client/http, tests: utilize a mock PD service discovery during testing (
721b6fb 
#7883)

ref #7300

Utilize a mock PD service discovery to prevent blocking of the HTTP request during testing.

Signed-off-by: JmPotato <ghzpotato@gmail.com>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@rleungx rleungx mentioned this issue Mar 11, 2024
Merged
ti-chi-bot bot added a commit that referenced this issue Mar 12, 2024
@rleungx @ti-chi-bot
client: add status http interface (#7903)
efffcc5 
ref #7300

Signed-off-by: Ryan Leung <rleungx@gmail.com>

Co-authored-by: ti-chi-bot[bot] <108142056+ti-chi-bot[bot]@users.noreply.github.com>
@JmPotato JmPotato mentioned this issue Mar 22, 2024
Merged
ti-chi-bot bot pushed a commit that referenced this issue Mar 22, 2024
@JmPotato
client/http: fix the panic when merging empty RegionsInfo (#7970)
c2eac4b 
ref #7300

Check if the source and target `*RegionsInfo` are nil before merging.

Signed-off-by: JmPotato <ghzpotato@gmail.com>
@ti-chi-bot ti-chi-bot mentioned this issue Mar 27, 2024
Closed
12 tasks
@okJiang okJiang mentioned this issue May 23, 2024
Merged
@HuSharp HuSharp mentioned this issue May 24, 2024
Merged
ti-chi-bot bot pushed a commit that referenced this issue May 24, 2024
@HuSharp
tools/ctl: add caller ID for pd-ctl (#8214)
7cc3b4e 
ref #7300

Signed-off-by: husharp <jinhao.hu@pingcap.com>
ti-chi-bot bot pushed a commit that referenced this issue May 27, 2024
@okJiang
ctl: replace doRequest with HTTP client to get health status (#8212)
dd7f2a7 
ref #7300

Signed-off-by: okJiang <819421878@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@JmPotato JmPotato

@HuSharp HuSharp

Labels
component/api HTTP API. type/development The issue belongs to a development tasks
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JmPotato @HuSharp