[GitHub] [skywalking-satellite] EvanLjp commented on a change in pull request #8: Add main structure

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

EvanLjp commented on a change in pull request #8:
URL: https://github.com/apache/skywalking-satellite/pull/8#discussion_r541832314



##########
File path: docs/design/module_structure.md
##########
@@ -1,18 +1,25 @@
 # Module structure
 
+## Overview
 Module is the core workers in Satellite. Module is constituted by the specific extension plugins.
-There are four modules in Satellite, which is ClientManager, Gatherer, Sender, and Processor.
+There are 3 modules in one agent, which are Gatherer, Processor, and Sender.
 
-Responsibilities:
+- The Gatherer module is responsible for fetching or receiving data and pushing the data to Queue. So there are 2 kinds of Gatherer, which are ReceiverGatherer and FetcherGatherer.
+- The Processor module is responsible for reading data from the queue and processing data by a series of filter chains.
+- The Sender module is responsible for async processing and forwarding the data to the external services in the batch mode. After sending success, Sender would also acknowledge the offset of Queue in Gatherer.
 
-- ClientManager: Maintain connection and monitor connection status
-- Sender: Sender data to the external services, such as Kafka and OAP
-- Gatherer: Gather the APM data from the other systems, such as fetch prometheus metrics.
-- Processor: Data processing to create new metrics data.
+```
+                            Agent

Review comment:
       Agent is a whole concept:   Each namespace has one agent to collect the telemetry data. (module_design.md)




----------------------------------------------------------------
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:
users@infra.apache.org