[HUDI-3239] Convert BaseHoodieTableFileIndex to Java

[alexeykudinkin]

Tips

• Thank you very much for contributing to Apache Hudi.
• Please review https://hudi.apache.org/contribute/how-to-contribute before opening a pull request.

What is the purpose of the pull request

Converting BaseHoodieTableFileIndex to Java, removing Scala as a dependency from "hudi-common"

Brief change log

See above

Verify this pull request

This pull request is already covered by existing tests, such as (please describe tests).

Committer checklist

• Has a corresponding JIRA in PR title & commit

• Commit message is descriptive of the change

• CI is green

• Necessary doc changes done or have another open PR

• For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

[yanghua]

@alexeykudinkin Thanks for your contribution! Can you figure out why CI failed?

[alexeykudinkin]

@yanghua there's a long stack of PRs, and there were issues w/ Metadata Table that were failing tests in the PR at the very bottom of it.

These issues have been addressed here in #4716, and as soon as it lands i'll rebase onto it.

[yanghua]

@yanghua there's a long stack of PRs, and there were issues w/ Metadata Table that were failing tests in the PR at the very bottom of it.

These issues have been addressed here in #4716, and as soon as it lands i'll rebase onto it.

Sounds good.

[yihua]

Good job on converting BaseHoodieTableFileIndex from Scala to Java

[yihua]

Should the following dep be removed in this pom as well? Similar for the other bundle poms, hudi-hadop-mr/pom.xml,hudi-common/pom.xml where scala dep is added in #4531.

    <!-- Scala -->
    <!-- TODO(HUDI-3239) remove this dep -->
    <dependency>
      <groupId>org.scala-lang</groupId>
      <artifactId>scala-library</artifactId>
      <version>${scala.version}</version>
    </dependency>

[alexeykudinkin]

Oh, great catch!
Did mindless search and delete somehow bundles didn't bubble up in it. Sorry about that.

[yihua]

LGTM. I put up two more nits regarding the hudi-common pom.

[yihua]

Should the <pluginManagement> be removed altogether? I see maven-compiler-plugin is added before mainly for compiling both Java and Scala code.

[alexeykudinkin]

Fair enough

[yihua]

same here for this plugin.

[alexeykudinkin]

@hudi-bot run azure

[hudi-bot]

CI report:

• dd2daec Azure: FAILURE Azure: SUCCESS

Bot commands

@hudi-bot supports the following commands:

• @hudi-bot run azure re-run the last Azure build