wu-sheng opened a new pull request #4570: [WIP][Inventory Design Change] Remove 
endpoint register and endpoint inventory
URL: https://github.com/apache/skywalking/pull/4570
 
 
   If you are familiar with the latest OAP core, you will know, we register id 
for the endpoint. But this design has the following issues
   1. The register has low performance as it can't be run in parral even we 
have clustering OAP. Even worse, in the cluster mode, the register is slower
   2. The ID is designed as an integer, it could be run out in some time. We 
don't have good idea to restore or extend it.
   3. Endpoint including the parameter(such as, inside the URI) is always an 
issue, and could only be improved in the agent plugin, but never could be fixed.
   
   The new changes are
   1. Use `EndpointTraffic` to replace `EndpointInventory`. EndpointTraffic is 
a manual metrics, not a register inventory.
   2. ID register would not happen, and service_id + base64(endpoint name) will 
be used as the entity id.
   3. EndpointTraffic also keeps the time series feature like all other 
metrics, so it could be removed by TTL.
   4. EndpointRelation follows the changes of EndpointInevntory removal. It 
keeps the source-service_id + base64(source endpoint name) and dest-service_id 
+ base64(dest endpoint name).
   
   - The Endpoint_Traffic looks like this
   
![image](https://user-images.githubusercontent.com/5441976/77605518-17779080-6f50-11ea-88df-de2e6e98e357.png)
   
   - The Endpoint* metrics, such as endpoint_sla. It is wider than before, 
because of entity_id format changed.
   
![image](https://user-images.githubusercontent.com/5441976/77605598-50176a00-6f50-11ea-9194-348487b654ba.png)
   

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


With regards,
Apache Git Services

Reply via email to