jihoonson commented on a change in pull request #7306: Reconcile terminology 
and method naming to 'used/unused segments'; Rename MetadataSegmentManager to 
MetadataSegments
URL: https://github.com/apache/druid/pull/7306#discussion_r369333178
 
 

 ##########
 File path: docs/design/coordinator.md
 ##########
 @@ -51,7 +64,12 @@ Segments can be automatically loaded and dropped from the 
cluster based on a set
 
 ### Cleaning up segments
 
-Each run, the Druid coordinator compares the list of available database 
segments in the database with the current segments in the cluster. Segments 
that are not in the database but are still being served in the cluster are 
flagged and appended to a removal list. Segments that are overshadowed (their 
versions are too old and their data has been replaced by newer segments) are 
also dropped.
+On each run, the Druid Coordinator compares the set of used segments in the 
database with the segments served by some
+nodes in the cluster. Coordinator sends requests to Historical nodes to unload 
unused segments or segments the are
 
 Review comment:
   Suggest `some nodes` -> `historical processes`.

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to