You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@celeborn.apache.org by et...@apache.org on 2023/02/13 07:35:40 UTC

[incubator-celeborn] 01/02: [CELEBORN-284] fix typo in CelebornConf (#1218)

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

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

commit 78c132dcf284a871bb1ead782b61a8528c17d207
Author: jiaoqingbo <11...@qq.com>
AuthorDate: Fri Feb 10 14:59:36 2023 +0800

    [CELEBORN-284] fix typo in CelebornConf (#1218)
    
    Co-authored-by: jiaoqb <ji...@asiainfo.com>
    (cherry picked from commit 3a92b0d911984cc85411c6be68edff90cefd65ce)
---
 common/src/main/scala/org/apache/celeborn/common/CelebornConf.scala | 4 ++--
 docs/configuration/client.md                                        | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/common/src/main/scala/org/apache/celeborn/common/CelebornConf.scala b/common/src/main/scala/org/apache/celeborn/common/CelebornConf.scala
index 31827e42b..f44651ec6 100644
--- a/common/src/main/scala/org/apache/celeborn/common/CelebornConf.scala
+++ b/common/src/main/scala/org/apache/celeborn/common/CelebornConf.scala
@@ -1206,7 +1206,7 @@ object CelebornConf extends Logging {
       .categories("client")
       .doc("Celeborn supports the following kind of shuffle writers. 1. hash: hash-based shuffle writer " +
         "works fine when shuffle partition count is normal; 2. sort: sort-based shuffle writer works fine " +
-        "when memory pressure is high or shuffle partition count it huge.")
+        "when memory pressure is high or shuffle partition count is huge.")
       .version("0.2.0")
       .stringConf
       .transform(_.toUpperCase(Locale.ROOT))
@@ -1305,7 +1305,7 @@ object CelebornConf extends Logging {
     buildConf("celeborn.test.fetchFailure")
       .categories("client")
       .version("0.2.0")
-      .doc("Wheter to test fetch chunk failure")
+      .doc("Whether to test fetch chunk failure")
       .booleanConf
       .createWithDefault(false)
 
diff --git a/docs/configuration/client.md b/docs/configuration/client.md
index b58fac84e..a0cbb8a20 100644
--- a/docs/configuration/client.md
+++ b/docs/configuration/client.md
@@ -64,11 +64,11 @@ license: |
 | celeborn.shuffle.rangeReadFilter.enabled | false | If a spark application have skewed partition, this value can set to true to improve performance. | 0.2.0 | 
 | celeborn.shuffle.register.maxRetries | 3 | Max retry times for client to register shuffle. | 0.2.0 | 
 | celeborn.shuffle.register.retryWait | 3s | Wait time before next retry if register shuffle failed. | 0.2.0 | 
-| celeborn.shuffle.writer | HASH | Celeborn supports the following kind of shuffle writers. 1. hash: hash-based shuffle writer works fine when shuffle partition count is normal; 2. sort: sort-based shuffle writer works fine when memory pressure is high or shuffle partition count it huge. | 0.2.0 | 
+| celeborn.shuffle.writer | HASH | Celeborn supports the following kind of shuffle writers. 1. hash: hash-based shuffle writer works fine when shuffle partition count is normal; 2. sort: sort-based shuffle writer works fine when memory pressure is high or shuffle partition count is huge. | 0.2.0 | 
 | celeborn.slots.reserve.maxRetries | 3 | Max retry times for client to reserve slots. | 0.2.0 | 
 | celeborn.slots.reserve.retryWait | 3s | Wait time before next retry if reserve slots failed. | 0.2.0 | 
 | celeborn.storage.hdfs.dir | &lt;undefined&gt; | HDFS dir configuration for Celeborn to access HDFS. | 0.2.0 | 
-| celeborn.test.fetchFailure | false | Wheter to test fetch chunk failure | 0.2.0 | 
+| celeborn.test.fetchFailure | false | Whether to test fetch chunk failure | 0.2.0 | 
 | celeborn.test.retryCommitFiles | false | Fail commitFile request for test | 0.2.0 | 
 | celeborn.worker.excluded.checkInterval | 30s | Interval for client to refresh excluded worker list. | 0.2.0 | 
 | celeborn.worker.excluded.expireTimeout | 600s | Timeout time for LifecycleManager to clear reserved excluded worker. | 0.2.0 |