[GitHub] [incubator-celeborn] pan3793 commented on a diff in pull request #1664: [CELEBORN-751] Rename remain rss related class name and filenames etc...

["pan3793 (via GitHub)" <gi...@apache.org>]

pan3793 commented on code in PR #1664:
URL: https://github.com/apache/incubator-celeborn/pull/1664#discussion_r1250352872


##########
docs/configuration/worker.md:
##########
@@ -93,7 +93,7 @@ license: |
 | celeborn.worker.storage.dirs | <undefined> | Directory list to store shuffle data. It's recommended to configure one directory on each disk. Storage size limit can be set for each directory. For the sake of performance, there should be no more than 2 flush threads on the same disk partition if you are using HDD, and should be 8 or more flush threads on the same disk partition if you are using SSD. For example: `dir1[:capacity=][:disktype=][:flushthread=],dir2[:capacity=][:disktype=][:flushthread=]` | 0.2.0 | 
 | celeborn.worker.storage.disk.reserve.size | 5G | Celeborn worker reserved space for each disk. | 0.3.0 | 
 | celeborn.worker.storage.hdfs.dir | <undefined> | HDFS dir configuration for Celeborn to access HDFS. | 0.2.0 | 
-| celeborn.worker.storage.workingDir | rss-worker/shuffle_data | Worker's working dir path name. | 0.3.0 | 
+| celeborn.worker.storage.workingDir | celeborn-worker/shuffle_data | Worker's working dir path name. | 0.3.0 | 

Review Comment:
   this may need to upgrade migration guide



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