diff --git a/CHANGES.md b/CHANGES.md index b13efb519269fc423ae5c330c496e93e7ca0d178..ee757accf19119d4b1cda04baa36d6e467e86f8d 100644 --- a/CHANGES.md +++ b/CHANGES.md @@ -61,6 +61,7 @@ Release Notes. * Make metrics exporter still work even when storage layer failed. * Fix Jetty HTTP `TRACE` issue, disable HTTP methods except `POST`. * CVE: upgrade snakeyaml to prevent [billion laughs attack](https://en.wikipedia.org/wiki/Billion_laughs#Variations) in dynamic configuration. +* polish debug logging avoids null value when the segment ignored. #### UI * Add logo for kong plugin. diff --git a/oap-server/analyzer/agent-analyzer/src/main/java/org/apache/skywalking/oap/server/analyzer/provider/trace/parser/listener/SegmentAnalysisListener.java b/oap-server/analyzer/agent-analyzer/src/main/java/org/apache/skywalking/oap/server/analyzer/provider/trace/parser/listener/SegmentAnalysisListener.java index 6fc5a76f732b426903735ed8ea480cfb9b16e709..4c8c96d447a017bee0e77ad5e6fedf3dbc2b9ee5 100644 --- a/oap-server/analyzer/agent-analyzer/src/main/java/org/apache/skywalking/oap/server/analyzer/provider/trace/parser/listener/SegmentAnalysisListener.java +++ b/oap-server/analyzer/agent-analyzer/src/main/java/org/apache/skywalking/oap/server/analyzer/provider/trace/parser/listener/SegmentAnalysisListener.java @@ -170,14 +170,17 @@ public class SegmentAnalysisListener implements FirstAnalysisListener, EntryAnal @Override public void build() { - if (log.isDebugEnabled()) { - log.debug("segment listener build, segment id: {}", segment.getSegmentId()); - } - if (sampleStatus.equals(SAMPLE_STATUS.IGNORE)) { + if (log.isDebugEnabled()) { + log.debug("segment ignored, trace id: {}", segment.getTraceId()); + } return; } + if (log.isDebugEnabled()) { + log.debug("segment listener build, segment id: {}", segment.getSegmentId()); + } + segment.setEndpointId(endpointId); segment.setEndpointName(endpointName);