Skip to content

refactor: centralize Java dependency versions#157

Open
hfutatzhanghb wants to merge 3 commits into
lance-format:mainfrom
hfutatzhanghb:agent/centralize-java-dependency-versions
Open

refactor: centralize Java dependency versions#157
hfutatzhanghb wants to merge 3 commits into
lance-format:mainfrom
hfutatzhanghb:agent/centralize-java-dependency-versions

Conversation

@hfutatzhanghb

@hfutatzhanghb hfutatzhanghb commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • centralize repeated Java dependency and plugin versions in the root Maven properties
  • replace repeated Hive, Hadoop, Derby, Guava, JUnit, Mockito, AssertJ, and plugin version literals with property references
  • preserve module-specific one-off versions and Glue's explicit SLF4J override
  • update Unity table column type_json generation to include the column name required by the current Unity Catalog service

Validation

  • rtk git diff --check
  • rtk xmllint --noout java/pom.xml java/lance-namespace-hive2/pom.xml java/lance-namespace-hive3/pom.xml java/lance-namespace-iceberg/pom.xml java/lance-namespace-unity/pom.xml java/lance-namespace-glue/pom.xml
  • rtk rg -n 'junit-version|2\.3\.9|2\.8\.5|3\.1\.3|10\.14\.2\.0|33\.4\.0-jre|5\.18\.0|3\.26\.3|3\.3\.0' java -g 'pom.xml'
  • GitHub Actions on PR head 571c07dda4a59b550f5b92ffb6990de80cca9200

Local Maven build/test/lint was intentionally skipped per the Lance GitHub workflow policy; GitHub Actions is the validation gate.

@hfutatzhanghb
hfutatzhanghb force-pushed the agent/centralize-java-dependency-versions branch from 26410cd to 729246b Compare July 13, 2026 06:14
@hfutatzhanghb hfutatzhanghb changed the title build: centralize Java dependency versions refactor: centralize Java dependency versions Jul 13, 2026
@hfutatzhanghb

Copy link
Copy Markdown
Contributor Author

Hi @jackye1995 @eddyxu . Could you please help review this PR when have free time? Thanks a lot.

@hfutatzhanghb
hfutatzhanghb force-pushed the agent/centralize-java-dependency-versions branch from d11881e to 571c07d Compare July 13, 2026 06:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant