Skip to content

Update java requirement: we now need Java 21#2507

Merged
gueniai merged 2 commits into
mainfrom
require-java-21
Jun 15, 2026
Merged

Update java requirement: we now need Java 21#2507
gueniai merged 2 commits into
mainfrom
require-java-21

Conversation

@asnare

@asnare asnare commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Changes

This PR updates the Java requirements for the CLI from 11 to 21. The most recent builds of Morpheus switched from Java 11 to 21, so this is now a requirement.

Linked issues

Context: databrickslabs/morpheus#952 (private)

Functionality

  • updated relevant user documentation
  • modified existing command: databricks labs lakebridge install-transpile

Tests

  • updated integration tests

@asnare asnare self-assigned this Jun 13, 2026
@asnare asnare added the bug Something isn't working label Jun 13, 2026
@asnare asnare requested a review from a team as a code owner June 13, 2026 11:03
@asnare asnare added documentation Improvements or additions to documentation feat/cli actions that are visible to the user labels Jun 13, 2026
@CLAassistant

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@codecov

codecov Bot commented Jun 13, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 67.16%. Comparing base (e969dff) to head (2b75cd7).

Files with missing lines Patch % Lines
...atabricks/labs/lakebridge/transpiler/installers.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2507   +/-   ##
=======================================
  Coverage   67.16%   67.16%           
=======================================
  Files         107      107           
  Lines       10027    10027           
  Branches     1083     1083           
=======================================
  Hits         6735     6735           
  Misses       3092     3092           
  Partials      200      200           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@asnare asnare enabled auto-merge June 13, 2026 11:34
@github-actions

github-actions Bot commented Jun 13, 2026

Copy link
Copy Markdown

✅ 183/183 passed, 8 flaky, 2 skipped, 56m37s total

Flaky tests:

  • 🤪 test_installs_and_runs_local_bladebridge (10.999s)
  • 🤪 test_installs_and_runs_pypi_bladebridge (23.236s)
  • 🤪 test_transpiles_informatica_to_sparksql_non_interactive[False] (25.524s)
  • 🤪 test_transpiles_informatica_to_sparksql_non_interactive[True] (26.192s)
  • 🤪 test_transpiles_informatica_to_sparksql (26.542s)
  • 🤪 test_transpile_teradata_sql (26.646s)
  • 🤪 test_transpile_teradata_sql_non_interactive[False] (25.768s)
  • 🤪 test_transpile_teradata_sql_non_interactive[True] (26.398s)

Running from acceptance #4830

@gueniai gueniai disabled auto-merge June 15, 2026 04:31
@gueniai gueniai merged commit bbc2333 into main Jun 15, 2026
15 checks passed
@gueniai gueniai deleted the require-java-21 branch June 15, 2026 04:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working documentation Improvements or additions to documentation feat/cli actions that are visible to the user

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants