This is an automated email from the ASF dual-hosted git repository.

zhouky pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/incubator-celeborn.git


The following commit(s) were added to refs/heads/main by this push:
     new 92777c3ff [CELEBORN-927][DOC] Correct 
celeborn.metrics.conf.*.sink.csv.class configuration example for a CSV sink
92777c3ff is described below

commit 92777c3ff2866010772b72d1c4ddd9663c11c56f
Author: SteNicholas <[email protected]>
AuthorDate: Wed Aug 30 16:11:03 2023 +0800

    [CELEBORN-927][DOC] Correct celeborn.metrics.conf.*.sink.csv.class 
configuration example for a CSV sink
    
    ### What changes were proposed in this pull request?
    
    Correct `celeborn.metrics.conf.*.sink.csv.class` configuration example for 
a CSV sink.
    
    ### Why are the changes needed?
    
    `celeborn.metrics.conf.*.sink.csv.class` configuration example for a CSV 
sink is wrong, which value should be 
`org.apache.celeborn.common.metrics.sink.CsvSink`.
    
    ### Does this PR introduce _any_ user-facing change?
    
    No.
    
    ### How was this patch tested?
    
    None.
    
    Closes #1865 from SteNicholas/CELEBORN-927.
    
    Authored-by: SteNicholas <[email protected]>
    Signed-off-by: zky.zhoukeyong <[email protected]>
---
 docs/monitoring.md | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/docs/monitoring.md b/docs/monitoring.md
index 0cdbe66e3..1263879e3 100644
--- a/docs/monitoring.md
+++ b/docs/monitoring.md
@@ -57,7 +57,7 @@ details, i.e. the parameters take the following form:
 `celeborn.metrics.conf.[instance|*].sink.[sink_name].[parameter_name]`.
 This example shows a list of Celeborn configuration parameters for a CSV sink:
 ```
-"celeborn.metrics.conf.*.sink.csv.class"="org.apache.celeborn.common.metrics.sink.GraphiteSink"
+"celeborn.metrics.conf.*.sink.csv.class"="org.apache.celeborn.common.metrics.sink.CsvSink"
 "celeborn.metrics.conf.*.sink.csv.period"="1"
 "celeborn.metrics.conf.*.sink.csv.unit"=minutes
 "celeborn.metrics.conf.*.sink.csv.directory"=/tmp/

Reply via email to