You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@celeborn.apache.org by "cxzl25 (via GitHub)" <gi...@apache.org> on 2023/09/18 10:20:37 UTC

[GitHub] [incubator-celeborn] cxzl25 commented on pull request #1919: [CELEBORN-983] Rename PrometheusMetric configuration

cxzl25 commented on PR #1919:
URL: https://github.com/apache/incubator-celeborn/pull/1919#issuecomment-1723125834

   > I don't think there is a need to replace these configurations, otherwise there will be compatibility issues.
   
   `celeborn.master.http.port` has automatic fallback of the original parameters `celeborn.metrics.master.prometheus.port`.
   
   The current parameter is more ambiguous. It is not just a port opened for prometheus.
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@celeborn.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org