Skip to content

ci: extend the macOS Java publication build timeout - #10968

Merged
BubbleCal merged 1 commit into
mainfrom
yang/t032-java-publish-timeout
Nov 12, 2026
Merged

BubbleCal merged 1 commit into
mainfrom
yang/t032-java-publish-timeout

Conversation

@BubbleCal

@BubbleCal BubbleCal commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor

The beta.18 macOS JNI publication build exceeded its 90-minute job limit while compiling lance-jni, so Maven publication was skipped despite both Linux builds succeeding. GitHub explicitly reported the timeout; the log contains no compiler error.

Raise only the macOS job limit to 150 minutes. Release optimization, runner, architecture, source checkout, artifact handling and publication behavior remain unchanged.

Validation: applicable pre-commit checks, YAML parsing and git diff --check passed. All three native builds and Maven packaging dry run passed on this PR head. No Rust or Java source changed. Actual beta.18 publication remains a separate recovery using the merged workflow and the explicit release tag.

@github-actions github-actions Bot added A-ci CI / build workflows ci Github Action or Test issues labels Nov 11, 2026
@BubbleCal
BubbleCal marked this pull request as ready for review November 12, 2026 02:18

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

@lance-gatekeeper lance-gatekeeper Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Gate recommendation: approve with a non-blocking risk.

Raising the deadline is a reasonable, reversible mitigation for the reported macOS JNI timeout. The remaining risk is that 150 minutes may still be insufficient for a cold release build, leaving Maven publication waiting on a successful native build. If it times out again, use build-stage timings to guide the next change.

@lance-gatekeeper lance-gatekeeper Bot added K-approved Latest Gatekeeper recommendation permits acceptance. K-risk Latest Gatekeeper recommendation includes a non-blocking risk. labels Nov 12, 2026
@BubbleCal
BubbleCal merged commit 99ce601 into main Nov 12, 2026
18 of 19 checks passed
@BubbleCal
BubbleCal deleted the yang/t032-java-publish-timeout branch November 12, 2026 03:08
BubbleCal added a commit to lancedb/lancedb that referenced this pull request Nov 12, 2026
Upgrade the Rust Lance dependency, Java lance-core, and Python test
dependency from beta.17 to the officially published
[v15.0.0-beta.18](https://github.com/lance-format/lance/releases/tag/v15.0.0-beta.18)
(`079fd77683a7acf056b0912f6ecda938e39ee120`). This provides a compatible
SDK for the newer Lance source already adopted downstream. Rust registry
versions and checksums are unchanged; the Python lock uses the six
published beta.18 wheels.

### Compatibility

The adopted release includes [Lance
#10863](lance-format/lance#10863):
`FilteredReadOptionsProto.has_fragment_ids` distinguishes an explicit
empty fragment selection. Older executors ignore this field, so upgrade
executors before relying on the corrected empty-selection behavior. Rust
callers constructing this generated struct need the new field. [Lance
#10112](lance-format/lance#10112) also replaces
the internal `hamming_distance_batch` API with
`hamming_distance_batch_into`.

The Linux Python matrix timeout is raised from 30 to 60 minutes: the
Python 3.13 cold native build and both test passes completed, but the
job hit its limit during cleanup. Build profiles, test commands, and
coverage are unchanged.

### Validation

- Three native FRI/ANN regressions passed: deferred-compaction refined
RQ, deferred index remapping, and approximate-query builder.
- `cargo clippy --quiet --locked --features remote --tests --examples --
-D warnings` passed with pinned Rust 1.97.0.
- Locked metadata, formatting, Python Makefile-scope Ruff checks, and
`uv lock --check --offline` passed.
- YAML, whitespace, end-of-file, and typo checks passed for the
timeout-only workflow change.

### Publication and CI

Java beta.18 is now published. After [Lance
#10968](lance-format/lance#10968) merged, the
[normal publication
workflow](https://github.com/lance-format/lance/actions/runs/43146352505)
built all three JNI targets and published successfully; the Maven jar is
available. The upstream Python publication run was cancelled after its
Windows ARM job hit the one-hour timeout; six other wheels were
published and are the actual artifacts recorded in this lock.
Cross-platform SDK CI is still required. No performance claim is made by
this dependency update.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-ci CI / build workflows ci Github Action or Test issues K-approved Latest Gatekeeper recommendation permits acceptance. K-risk Latest Gatekeeper recommendation includes a non-blocking risk.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants