Skip to content

test: skip expensive gcs benchmarks#5440

Merged
westonpace merged 3 commits intolance-format:mainfrom
westonpace:fix/ci-too-expensive-gcs
Dec 9, 2025
Merged

test: skip expensive gcs benchmarks#5440
westonpace merged 3 commits intolance-format:mainfrom
westonpace:fix/ci-too-expensive-gcs

Conversation

@westonpace
Copy link
Copy Markdown
Member

Skips some of the more expensive basic benchmarks when running in cloud storage. We should get another environment online that tests these against NVMe or some kind of fast local disk.

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits.
Credits must be used to enable repository wide code reviews.

@github-actions github-actions Bot added the python label Dec 8, 2025
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Dec 8, 2025

ACTION NEEDED
Lance follows the Conventional Commits specification for release automation.

The PR title and description are used as the merge commit message. Please update your PR title and description to match the specification.

For details on the error please inspect the "PR Title Check" action.

@westonpace westonpace changed the title Fix/ci too expensive gcs test: skip expensive gcs benchmarks Dec 8, 2025
@github-actions github-actions Bot added the chore label Dec 8, 2025
@westonpace westonpace merged commit a5d9846 into lance-format:main Dec 9, 2025
13 of 14 checks passed
jackye1995 pushed a commit to jackye1995/lance that referenced this pull request Jan 21, 2026
Skips some of the more expensive basic benchmarks when running in cloud
storage. We should get another environment online that tests these
against NVMe or some kind of fast local disk.
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.

2 participants