lance-format / lance-format/lance

Add a min_batch_size property to the scanner

Open
#2,021 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Rust
Stars
7.1k
Forks
852
Avg merge
3d 18h
Merged PRs (30d)
272

Description

The current batch size is a max_batch_size only. It will slice large batches but will not merge smaller batches. There are cases where it is important for users to have exact sized batches (e.g. some training software will fill in small batches with dummy data / padding and users want to avoid that). We can be clear in the documentation that setting the property could have a performance impact.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Locate the scanner implementation and its existing max_batch_size handling, then read how batches are split and exposed to users. Done means a min_batch_size property can merge smaller batches to the requested size, with documentation warning about possible performance impact and coverage for the new behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
data-engineering
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.