wu-sheng commented on issue #5023:
URL: https://github.com/apache/skywalking/issues/5023#issuecomment-655815846


   > i read thePR ,i don't think parse time from traceId is a good idea.for 
example the traceId in nginx lua agent.
   
   No, that is not a good idea.
   
   > Basically, you need to consider when you do trace_id query, there is no 
timestamp.
   
   Because finding the timestamp may be an issue. When we inject the trace id 
into the logs or even in codes/database, there may be no timestamp there. And 
the trace query page supports `?traceId=xxx` to get the result directly.
   
   > if the advice2 is not implement, Short term plan: if have 7 segment index, 
from 7.1 to 7.7 ,we can search every index from 7.7 to 7.1 .if hits some 
data,we also to search the prefix one (when cross day) .
   
   This is as same as using Alias query, which is better because of parallel 
query.
   
   -------
   Basically, in `trace_id` query case, I think we still need the current query 
mode. If you want to optimize the query when timestamps exist, I think it is 
possible.


----------------------------------------------------------------
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]


Reply via email to