[GH-2321] Remove unnecessary commons-lang dependency (#2322)

Author: jiayuasu

pom.xml

@@ -328,11 +328,6 @@
                 <version>${log4j.version}</version>
                 <scope>test</scope>
             </dependency>
-            <dependency>
-                <groupId>commons-lang</groupId>
-                <artifactId>commons-lang</artifactId>
-                <version>2.6</version>
-            </dependency>
             <dependency>
                 <groupId>org.apache.commons</groupId>
                 <artifactId>commons-math3</artifactId>

spark-shaded/pom.xml

@@ -236,7 +236,6 @@
                                     <exclude>org.scala-lang:scala-library</exclude>
                                     <exclude>org.apache.commons:commons-*</exclude>
                                     <exclude>commons-pool:commons-pool</exclude>
-                                    <exclude>commons-lang:commons-lang</exclude>
                                     <exclude>commons-io:commons-io</exclude>
                                     <exclude>commons-logging:commons-logging</exclude>
                                 </excludes>

spark/common/pom.xml

@@ -130,10 +130,6 @@
             <groupId>edu.ucar</groupId>
             <artifactId>cdm-core</artifactId>
         </dependency>
-        <dependency>
-            <groupId>commons-lang</groupId>
-            <artifactId>commons-lang</artifactId>
-        </dependency>
         <dependency>
             <groupId>org.apache.logging.log4j</groupId>
             <artifactId>log4j-1.2-api</artifactId>

spark/common/src/main/java/org/apache/sedona/core/spatialRDD/SpatialRDD.java

@@ -25,7 +25,6 @@
 import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
-import org.apache.commons.lang.NullArgumentException;
 import org.apache.log4j.Logger;
 import org.apache.sedona.common.FunctionsGeoTools;
 import org.apache.sedona.common.jts2geojson.GeoJSONWriter;
@@ -542,7 +541,7 @@ public boolean analyze(Envelope datasetBoundary, Integer approximateTotalCount)
    */
   public void saveAsWKB(String outputLocation) {
     if (this.rawSpatialRDD == null) {
-      throw new NullArgumentException("save as WKB cannot operate on null RDD");
+      throw new IllegalArgumentException("save as WKB cannot operate on null RDD");
     }
     this.rawSpatialRDD
         .mapPartitions(
@@ -571,7 +570,7 @@ public Iterator<String> call(Iterator<T> iterator) throws Exception {
   /** Save as WKT */
   public void saveAsWKT(String outputLocation) {
     if (this.rawSpatialRDD == null) {
-      throw new NullArgumentException("save as WKT cannot operate on null RDD");
+      throw new IllegalArgumentException("save as WKT cannot operate on null RDD");
     }
     this.rawSpatialRDD
         .mapPartitions(

spark/common/src/test/java/org/apache/sedona/core/spatialRDD/SpatialRDDWriterTest.java

@@ -25,7 +25,6 @@
 import java.io.IOException;
 import java.util.List;
 import org.apache.commons.io.FileUtils;
-import org.apache.commons.lang.NullArgumentException;
 import org.apache.sedona.common.enums.FileDataSplitter;
 import org.apache.sedona.common.utils.GeomUtils;
 import org.junit.AfterClass;
@@ -137,8 +136,8 @@ public void testSaveAsWKT() throws IOException {
     verifyResult(resultWKT.rawSpatialRDD.takeOrdered(5), spatialRDD.rawSpatialRDD.takeOrdered(5));
   }
 
-  /** Test throws NullArgumentException when Spatial RDD is null. */
-  @Test(expected = NullArgumentException.class)
+  /** Test throws IllegalArgumentException when Spatial RDD is null. */
+  @Test(expected = IllegalArgumentException.class)
   public void testSaveAsEmptyWKB() {
 
     PointRDD emptySpatialRDD = new PointRDD();