From 1cddb60a0002156152a0420222d2196a065a7151 Mon Sep 17 00:00:00 2001 From: "Xin,Zhang" Date: Thu, 1 Oct 2020 17:28:18 +0800 Subject: [PATCH] add @SuppressWarnings(NonAtomicVolatileUpdate) (#5606) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Co-authored-by: 吴晟 Wu Sheng --- .../commons/datacarrier/consumer/MultipleChannelsConsumer.java | 1 + .../commons/datacarrier/partition/SimpleRollingPartitioner.java | 1 + 2 files changed, 2 insertions(+) diff --git a/apm-commons/apm-datacarrier/src/main/java/org/apache/skywalking/apm/commons/datacarrier/consumer/MultipleChannelsConsumer.java b/apm-commons/apm-datacarrier/src/main/java/org/apache/skywalking/apm/commons/datacarrier/consumer/MultipleChannelsConsumer.java index d9d243681..c65b8e81b 100644 --- a/apm-commons/apm-datacarrier/src/main/java/org/apache/skywalking/apm/commons/datacarrier/consumer/MultipleChannelsConsumer.java +++ b/apm-commons/apm-datacarrier/src/main/java/org/apache/skywalking/apm/commons/datacarrier/consumer/MultipleChannelsConsumer.java @@ -30,6 +30,7 @@ import org.apache.skywalking.apm.commons.datacarrier.buffer.QueueBuffer; public class MultipleChannelsConsumer extends Thread { private volatile boolean running; private volatile ArrayList consumeTargets; + @SuppressWarnings("NonAtomicVolatileUpdate") private volatile long size; private final long consumeCycle; diff --git a/apm-commons/apm-datacarrier/src/main/java/org/apache/skywalking/apm/commons/datacarrier/partition/SimpleRollingPartitioner.java b/apm-commons/apm-datacarrier/src/main/java/org/apache/skywalking/apm/commons/datacarrier/partition/SimpleRollingPartitioner.java index ff3bdf5d3..379c7252d 100644 --- a/apm-commons/apm-datacarrier/src/main/java/org/apache/skywalking/apm/commons/datacarrier/partition/SimpleRollingPartitioner.java +++ b/apm-commons/apm-datacarrier/src/main/java/org/apache/skywalking/apm/commons/datacarrier/partition/SimpleRollingPartitioner.java @@ -22,6 +22,7 @@ package org.apache.skywalking.apm.commons.datacarrier.partition; * use normal int to rolling. */ public class SimpleRollingPartitioner implements IDataPartitioner { + @SuppressWarnings("NonAtomicVolatileUpdate") private volatile int i = 0; @Override