celeborn icon indicating copy to clipboard operation
celeborn copied to clipboard

[CELEBORN-1534] Order the API v1 response sequences filed

Open turboFei opened this issue 1 year ago • 4 comments

What changes were proposed in this pull request?

Make the results returned by the API v1 in order.

Why are the changes needed?

For customer interface friendly.

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Not needed.

turboFei avatar Jul 31 '24 18:07 turboFei

it might be costly for sorting shuffle_id and application_id listing API, cc @AngersZhuuuu

pan3793 avatar Aug 06 '24 02:08 pan3793

it might be costly for sorting shuffle_id and application_id listing API If that, I can remove the order for them

turboFei avatar Aug 06 '24 03:08 turboFei

maybe we can sort the response in client side.

Waiting for the celeborn-CLI.

turboFei avatar Aug 06 '24 18:08 turboFei

it might be costly for sorting shuffle_id and application_id listing API, cc @AngersZhuuuu

Perhaps we could add an option to choose between sorted and unsorted results?

RexXiong avatar Aug 08 '24 14:08 RexXiong

This PR is stale because it has been open 20 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Aug 29 '24 08:08 github-actions[bot]

This PR is stale because it has been open 20 days with no activity. Remove stale label or comment or this will be closed in 10 days.

github-actions[bot] avatar Sep 19 '24 08:09 github-actions[bot]

This issue was closed because it has been staled for 10 days with no activity.

github-actions[bot] avatar Sep 30 '24 08:09 github-actions[bot]