remove if-else block and print statement

apache · Dec 8, 2023 · c9dd1ce · c9dd1ce
1 parent c4f5761
commit c9dd1ce
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 6 deletions.
diff --git a/...rc/main/scala/org/apache/spark/sql/sedona_sql/expressions/raster/AggregateFunctions.scala b/...rc/main/scala/org/apache/spark/sql/sedona_sql/expressions/raster/AggregateFunctions.scala
@@ -63,10 +63,7 @@ class RS_Union_Aggr extends Aggregator[(GridCoverage2D, Int), ArrayBuffer[BandDa
       val widthNewRaster = RasterAccessors.getWidth(raster)
       val heightNewRaster = RasterAccessors.getHeight(raster)
 
-      if (width == widthNewRaster && height == heightNewRaster)
-        true
-      else
-        false
+      width == widthNewRaster && height == heightNewRaster
     }
   }
 

diff --git a/spark/common/src/test/scala/org/apache/sedona/sql/rasteralgebraTest.scala b/spark/common/src/test/scala/org/apache/sedona/sql/rasteralgebraTest.scala
@@ -890,8 +890,6 @@ class rasteralgebraTest extends TestBaseScala with BeforeAndAfter with GivenWhen
 
       df = df.selectExpr("RS_Union_aggr(raster, index) as rasters")
 
-      println(df.show(10, 50))
-
       val actualBands = df.selectExpr("RS_NumBands(rasters)").first().get(0)
       val expectedBands = 4
       assertEquals(expectedBands, actualBands)