Skip to content

Commit

Permalink
[MINOR][SQL] Remove Scalac 2.12-specific code in InMemoryFileIndex
Browse files Browse the repository at this point in the history
### What changes were proposed in this pull request?

This PR aims to remove `scalac 2.12`-specific code from InMemoryFileIndex class.

### Why are the changes needed?

Like the comment mentioned, we don't need this because `master` branch is using Scala 2.13 only.
https://github.com/apache/spark/blob/c0d9ca3be14cb0ec8d8f9920d3ecc4aac3cf5adc/sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InMemoryFileIndex.scala#L129

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Pass the CIs.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes apache#43233 from dongjoon-hyun/minor_scalac_2.12.

Authored-by: Dongjoon Hyun <[email protected]>
Signed-off-by: yangjie01 <[email protected]>
  • Loading branch information
dongjoon-hyun committed Jun 20, 2024
1 parent 6297091 commit 9aeb589
Showing 1 changed file with 0 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,6 @@ class InMemoryFileIndex(
case None =>
pathsToFetch += path
}
() // for some reasons scalac 2.12 needs this; return type doesn't matter
}
val filter = FileInputFormat.getInputPathFilter(new JobConf(hadoopConf, this.getClass))
val discovered = InMemoryFileIndex.bulkListLeafFiles(
Expand Down

0 comments on commit 9aeb589

Please sign in to comment.