apache / celeborn

Apache Celeborn is an elastic and high-performance service for shuffle and spilled data.
https://celeborn.apache.org/
Apache License 2.0
863 stars 351 forks source link

[CELEBORN-1541] Enhance the readable address for internal port #2659

Closed turboFei closed 1 month ago

turboFei commented 1 month ago

What changes were proposed in this pull request?

If the internal port is not defined, do not show the internal port(-1) in the readable address.

Why are the changes needed?

If the workerInfo is applied for the RESTful request, such as curl /api/v1/workers/exclude, the internal port is always -1.

It is not necessary to show the -1 in the readable address.

Does this PR introduce any user-facing change?

Just reduce the unnecessary info.

How was this patch tested?

Not needed.