address comments

sql/core/src/main/java/org/apache/spark/sql/sources/v2/DataSourceOptions.java

@@ -163,21 +163,14 @@ public double getDouble(String key, double defaultValue) {
    */
   public static final String DATABASE_KEY = "database";
 
-  /**
-   * Returns the value of the singular path option.
-   */
-  public Optional<String> path() {
-    return get(PATH_KEY);
-  }
-
   /**
    * Returns all the paths specified by both the singular path option and the multiple
    * paths option.
    */
   public String[] paths() {
-    String[] singularPath = path().map(s -> new String[]{s}).orElseGet(() -> new String[0]);
+    String[] singularPath =
+      get(PATH_KEY).map(s -> new String[]{s}).orElseGet(() -> new String[0]);
     Optional<String> pathsStr = get(PATHS_KEY);
-    System.out.println(pathsStr);
     if (pathsStr.isPresent()) {
       ObjectMapper objectMapper = new ObjectMapper();
       try {

sql/core/src/test/scala/org/apache/spark/sql/sources/v2/DataSourceOptionsSuite.scala

@@ -85,7 +85,7 @@ class DataSourceOptionsSuite extends SparkFunSuite {
       DataSourceOptions.PATH_KEY -> "abc",
       DataSourceOptions.TABLE_KEY -> "tbl").asJava)
 
-    assert(options.path.get() == "abc")
+    assert(options.paths().toSeq == Seq("abc"))
     assert(options.tableName().get() == "tbl")
     assert(!options.databaseName().isPresent)
   }
@@ -95,15 +95,13 @@ class DataSourceOptionsSuite extends SparkFunSuite {
       DataSourceOptions.PATH_KEY -> "abc",
       DataSourceOptions.PATHS_KEY -> """["c", "d"]""").asJava)
 
-    assert(options.path.get() == "abc")
     assert(options.paths().toSeq == Seq("abc", "c", "d"))
   }
 
   test("standard options with only multi-paths") {
     val options = new DataSourceOptions(Map(
       DataSourceOptions.PATHS_KEY -> """["c", "d\"e"]""").asJava)
 
-    assert(!options.path.isPresent)
     assert(options.paths().toSeq == Seq("c", "d\"e"))
   }
 }