[ 
https://issues.apache.org/jira/browse/SCB-859?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16598068#comment-16598068
 ] 

ASF GitHub Bot commented on SCB-859:
------------------------------------

WillemJiang commented on a change in pull request #885:  [SCB-859] Generate 
traceid
URL: 
https://github.com/apache/incubator-servicecomb-java-chassis/pull/885#discussion_r214217261
 
 

 ##########
 File path: core/src/main/java/org/apache/servicecomb/core/Invocation.java
 ##########
 @@ -188,11 +209,46 @@ public String getMicroserviceQualifiedName() {
     return operationMeta.getMicroserviceQualifiedName();
   }
 
+  protected void initTraceId() {
+    for (TraceIdGenerator traceIdGenerator : TRACE_ID_GENERATORS) {
+      initTraceId(traceIdGenerator);
 
 Review comment:
   If there are lots of transaction Id generators, they may override the 
transaction id each other.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


> generate traceId when create invocation for consumer/producer/edge
> ------------------------------------------------------------------
>
>                 Key: SCB-859
>                 URL: https://issues.apache.org/jira/browse/SCB-859
>             Project: Apache ServiceComb
>          Issue Type: Task
>          Components: Java-Chassis
>            Reporter: wujimin
>            Assignee: wujimin
>            Priority: Major
>             Fix For: java-chassis-1.1.0
>
>
> current generate traceId is a filter, and only for producer, it's not enough.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to