From 0f0554f8fae79d3b26c8202eb42fcf6dfc5e5e26 Mon Sep 17 00:00:00 2001 From: Gian Merlino Date: Fri, 31 Jan 2020 16:21:08 -0800 Subject: [PATCH] LimitedSequence: Improve suppression comment. (#9298) --- .../apache/druid/java/util/common/guava/LimitedSequence.java | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/core/src/main/java/org/apache/druid/java/util/common/guava/LimitedSequence.java b/core/src/main/java/org/apache/druid/java/util/common/guava/LimitedSequence.java index 2045c33b6ad..3fcc834b1e6 100644 --- a/core/src/main/java/org/apache/druid/java/util/common/guava/LimitedSequence.java +++ b/core/src/main/java/org/apache/druid/java/util/common/guava/LimitedSequence.java @@ -52,7 +52,9 @@ final class LimitedSequence extends YieldingSequenceBase return new LimitedYielder<>(subYielder, limitedAccumulator); } - @SuppressWarnings("InnerClassMayBeStatic") // False alarm: flagged by IDEA inspections, but can't actually be static. + // False alarm: This is flagged as InnerClassMayBeStatic by IDEA inspections, but can't actually be static, because + // it depends on the non-static inner class LimitedYieldingAccumulator. + @SuppressWarnings("InnerClassMayBeStatic") private class LimitedYielder implements Yielder { private final Yielder subYielder;