Address PR comment

Signed-off-by: Chen Dai <[email protected]>
opensearch-project · Mar 6, 2024 · ce21393 · ce21393
1 parent c22c7d1
commit ce21393
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/.../main/scala/org/opensearch/flint/spark/skipping/bloomfilter/BloomFilterMightContain.scala b/.../main/scala/org/opensearch/flint/spark/skipping/bloomfilter/BloomFilterMightContain.scala
@@ -72,17 +72,16 @@ case class BloomFilterMightContain(bloomFilterExpression: Expression, valueExpre
   }
 
   /**
-   * Generate expression code for Spark codegen execution. Sample result code (assume right value
-   * expression is not null):
+   * Generate expression code for Spark codegen execution. Sample result code:
    * ```
    *   boolean filter_isNull_0 = true;
    *   boolean filter_value_0 = false;
-   *   if (!(false)) {
+   *   if (!right_isNull) {
    *     filter_isNull_0 = false;
    *     filter_value_0 =
    *       org.opensearch.flint.core.field.bloomfilter.classic.ClassicBloomFilter.readFrom(
-   *         new java.io.ByteArrayInputStream(inputadapter_value_1)
-   *       ).mightContain(5193736701160936137L);
+   *         new java.io.ByteArrayInputStream(left_value)
+   *       ).mightContain(right_value);
    *   }
    * ```
    */