weixiang1862 commented on PR #13068: URL: https://github.com/apache/skywalking/pull/13068#issuecomment-2683747116
> I have some doubts > > 1. Metrics cache is metric only. How does this affect `trace latency`? Do you write the title wrong? > 2. Metrics cache is only added after `onInsertCompleted`. Do you mean the JDBC storage plugin isn't aware of the timeout? 1. Metrics cache is metric only. How does this affect `trace latency`? Do you write the title wrong? I means metrics generate by OAL from trace segment analyse, endpoint high latency caused trace segment finish delayed, finaly caused metrics persistent delay. 2. Metrics cache is only added after `onInsertCompleted`. Do you mean the JDBC storage plugin isn't aware of the timeout? Metrics cache may also loaded by `MetricsPersistentWorker#loadFromStorage` before persistent. This problem also exist in other storage modules, not just JDBC implmation. -- 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: notifications-unsubscr...@skywalking.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
