Skip to content

Comments

[CELEBORN-1810] Using Operation description instead of ApiResponse description for RESTful APIs#3038

Closed
turboFei wants to merge 1 commit intoapache:mainfrom
turboFei:api_desc
Closed

[CELEBORN-1810] Using Operation description instead of ApiResponse description for RESTful APIs#3038
turboFei wants to merge 1 commit intoapache:mainfrom
turboFei:api_desc

Conversation

@turboFei
Copy link
Member

@turboFei turboFei commented Dec 29, 2024

What changes were proposed in this pull request?

Using Operation description instead of ApiResponse description for RESTful APIs.

Why are the changes needed?

Make the API description in correct place.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Before:
image

After:
image

@turboFei turboFei changed the title Api desc Using Operation description instead of ApiResponse description Dec 29, 2024
@turboFei turboFei marked this pull request as draft December 29, 2024 01:25
@turboFei turboFei changed the title Using Operation description instead of ApiResponse description Using Operation description instead of ApiResponse description for RESTful APIs Dec 29, 2024
@turboFei turboFei changed the title Using Operation description instead of ApiResponse description for RESTful APIs [CELEBORN-1477][FOLLOWUP] Using Operation description instead of ApiResponse description for RESTful APIs Dec 29, 2024
@turboFei turboFei changed the title [CELEBORN-1477][FOLLOWUP] Using Operation description instead of ApiResponse description for RESTful APIs [CELEBORN-1810] Using Operation description instead of ApiResponse description for RESTful APIs Dec 29, 2024
@turboFei
Copy link
Member Author

Will change the description for exclude worker after #3023 merged.

ApiResponse description to Operation description
@turboFei turboFei marked this pull request as ready for review January 13, 2025 05:02
@turboFei turboFei requested a review from SteNicholas January 13, 2025 06:27
@RexXiong RexXiong closed this in f28ba6e Jan 23, 2025
@RexXiong
Copy link
Contributor

merge to main(v0.6.0)

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