[GitHub] [rocketmq-externals] StyleTang commented on pull request #744: [ISSUE #743] Implement RocketMQ message trace UI

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

StyleTang commented on pull request #744:
URL: https://github.com/apache/rocketmq-externals/pull/744#issuecomment-877801383


   Commit ca96adb contains some of the improvements for the trace UI.
   
   * Implement Features
     * support transaction message trace
     * support only producer trace enabled case
     * support only consumer trace enabled case
   * add(fix) trace type (message type)
   * use rocketmq.version = 4.9.0
   
   Test evidence are as follows:
   1. when only consumer is trace enabled
   ![image](https://user-images.githubusercontent.com/5286751/125197030-bb449700-e28e-11eb-986b-3358a9bcb4f5.png)
   2. when only producer is trace enabled
   ![image](https://user-images.githubusercontent.com/5286751/125197051-cb5c7680-e28e-11eb-9273-6212d7c0ea2f.png)
   
   3.  executeLocalTransaction fail but checkLocalTransaction succeed 
   ![image](https://user-images.githubusercontent.com/5286751/125197090-ef1fbc80-e28e-11eb-8ef6-acbc9b6e210d.png)
   ![image](https://user-images.githubusercontent.com/5286751/125197105-fe066f00-e28e-11eb-9f06-c325ee27d4f8.png)
   
   (we can not get the costTime and beginTime of the transaction message trace, for now I just use 50ms to draw the graph, this can be optimized in the future)
   
   Please help review when you are available. 
   (No other new features will be added to this PR, if there is, I will open a new PR)
   
   Thanks.


-- 
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: dev-unsubscribe@rocketmq.apache.org

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