Thesharing commented on a change in pull request #17284:
URL: https://github.com/apache/flink/pull/17284#discussion_r710089049



##########
File path: docs/content.zh/docs/deployment/config.md
##########
@@ -398,7 +398,7 @@ With the introduction of 
`state.backend.rocksdb.memory.managed` and `state.backe
 
 **Blob Server**
 
-The Blob Server is a component in the JobManager. It is used for distribution 
of objects that are too large to be attached to a RPC message and that benefit 
from caching (like Jar files or large serialized code objects).
+The Blob Server is a component in the JobManager and the TaskManager. It is 
used for distribution of objects that are too large to be attached to an RPC 
message and that benefit from caching (like Jar files or large serialized code 
objects).

Review comment:
       Thank you for contributing to Flink. As far as I know, the Blob Server 
is only located in the JobManager. The Blobs in the TaskManager are stored in 
BlobCacheService. 
   
   However, I think it's a bit of confusing that the configuration of 
BlobServer, BlobCacheService and BlobClient are all located at 
`BlobServerOptions.java` in the code and `Blob Server` section in the 
documentation. Should we just rename `BlobServerOptions.java` to 
`BlobOptions.java` and rename `Blob Server` section into `Blob`?




-- 
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-unsubscr...@flink.apache.org

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


Reply via email to