[GitHub] [incubator-celeborn] AngersZhuuuu commented on a diff in pull request #1083: [CELEBORN-139][BUG] Fix read wrong yaml file format when loading config

[GitBox <gi...@apache.org>]

AngersZhuuuu commented on code in PR #1083:
URL: https://github.com/apache/incubator-celeborn/pull/1083#discussion_r1048261920


##########
common/src/main/scala/org/apache/celeborn/common/CelebornConf.scala:
##########
@@ -2362,7 +2362,8 @@ object CelebornConf extends Logging {
     buildConf("celeborn.quota.configuration.path")
       .withAlternative("rss.quota.configuration.path")
       .categories("quota")
-      .doc("Quota configuration file path.")
+      .doc("Quota configuration file path. The file format should be yaml. Quota configuration file template can be " +
+        "found under conf directory.")

Review Comment:
   Also show defauklt path is `$RSS_HOME/quota.yaml`



-- 
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