Add more javadoc.

Weixiang Sun · Weixiang Sun · commit 91a0e396d67d · 2022-05-12T15:27:28.000-07:00
diff --git a/pinot-core/src/main/java/org/apache/pinot/core/query/reduce/CountGapfillProcessor.java b/pinot-core/src/main/java/org/apache/pinot/core/query/reduce/CountGapfillProcessor.java
@@ -30,6 +30,13 @@
 
 /**
  * Helper class to reduce and set gap fill results into the BrokerResponseNative
+ *
+ * {@link CountGapfillProcessor} is only applying the count aggregation function on the gapfilled result for each time
+ * bucket.
+ *
+ * {@link CountGapfillProcessor} is different from {@link GapfillProcessor} that {@link GapfillProcessor} will create
+ * the gapfilled entries, but {@link CountGapfillProcessor} does not generate the gapfilled entries. It just updated
+ * the aggregated count as necessary.
  */
 class CountGapfillProcessor extends BaseGapfillProcessor {
   protected final Set<Key> _filteredSet;
diff --git a/pinot-core/src/main/java/org/apache/pinot/core/query/reduce/SumAvgGapfillProcessor.java b/pinot-core/src/main/java/org/apache/pinot/core/query/reduce/SumAvgGapfillProcessor.java
@@ -30,6 +30,13 @@
 
 /**
  * Helper class to reduce and set gap fill results into the BrokerResponseNative
+ *
+ * {@link SumAvgGapfillProcessor} is only applying the sum and/or avg aggregation function on the gapfilled result for
+ * each time bucket.
+ *
+ * {@link SumAvgGapfillProcessor} is different from {@link GapfillProcessor} that {@link GapfillProcessor} will create
+ * the gapfilled entries, but {@link SumAvgGapfillProcessor} does not generate the gapfilled entries. It just updated
+ * the aggregated sum and/or avg counters as necessary.
  */
 class SumAvgGapfillProcessor extends BaseGapfillProcessor {
   private final double [] _sumes;
