Skip to content

[CELEBORN-1629][FOLLOWUP] Fix broken RESTful api link#2828

Closed
turboFei wants to merge 1 commit into
apache:mainfrom
turboFei:ratis_docs_link
Closed

[CELEBORN-1629][FOLLOWUP] Fix broken RESTful api link#2828
turboFei wants to merge 1 commit into
apache:mainfrom
turboFei:ratis_docs_link

Conversation

@turboFei
Copy link
Copy Markdown
Member

@turboFei turboFei commented Oct 19, 2024

What changes were proposed in this pull request?

Fix the broken link.

Why are the changes needed?

Followup for #2779.
The RESTful api docs was renamed from webapi.md to restapi.md in #2775.

And due these two PRs were merged in sequence nearly, so I did not aware this change.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

image image

@turboFei turboFei requested a review from SteNicholas October 19, 2024 06:01
Copy link
Copy Markdown
Contributor

@FMX FMX left a comment

Choose a reason for hiding this comment

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

LGTM. Merged into main(v0.6.0).

@FMX FMX closed this in bcb4318 Oct 21, 2024
zaynt4606 pushed a commit to zaynt4606/celeborn that referenced this pull request Oct 21, 2024
### What changes were proposed in this pull request?
Fix the broken link.

### Why are the changes needed?
Followup for apache#2779.
The RESTful api docs was renamed from webapi.md to restapi.md in apache#2775.

And due these two PRs were merged in sequence nearly, so I did not aware this change.

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

<img width="1255" alt="image" src="https://github.com/user-attachments/assets/a09aecf8-6e7e-458b-871d-f8dd5a0ac6b2">
<img width="937" alt="image" src="https://github.com/user-attachments/assets/bcefeecf-7a24-4616-9f5e-f2a11f464769">

Closes apache#2828 from turboFei/ratis_docs_link.

Authored-by: Wang, Fei <fwang12@ebay.com>
Signed-off-by: mingji <fengmingxiao.fmx@alibaba-inc.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants