Skip to content

feat(python): add preprocessed_data and range_id for range-based BTree index#5941

Open
touch-of-grey wants to merge 1 commit intolance-format:mainfrom
touch-of-grey:feat/pylance-preprocessed-data-range-id
Open

feat(python): add preprocessed_data and range_id for range-based BTree index#5941
touch-of-grey wants to merge 1 commit intolance-format:mainfrom
touch-of-grey:feat/pylance-preprocessed-data-range-id

Conversation

@touch-of-grey
Copy link
Contributor

Summary

  • Expose preprocessed_data and range_id parameters in pylance's create_scalar_index binding
  • Extract range_id from kwargs and serialize into ScalarIndexParams.params JSON for BTREE
  • Extract preprocessed_data from kwargs via ArrowArrayStreamReader and pass to index builder
  • Use execute_uncommitted when preprocessed_data is provided (matching Java binding's should_skip_commit logic)
  • Forward both params through Python create_scalar_index

This is a prerequisite for lance-ray's range-based sorted BTree index support (lance-format/lance-ray#92).

Test plan

  • cargo clippy passes
  • cargo fmt passes
  • Tested end-to-end via lance-ray sorted BTree tests (13/13 passing)

cc @jackye1995 @chenghao-guo

🤖 Generated with Claude Code

…sed BTree index

Expose the preprocessed_data and range_id parameters in pylance's
create_scalar_index binding, enabling range-based sorted BTree index
builds from external orchestrators like lance-ray.

- Extract range_id from kwargs and serialize into ScalarIndexParams for BTREE
- Extract preprocessed_data from kwargs via ArrowArrayStreamReader
- Use execute_uncommitted when preprocessed_data is provided (skip commit)
- Forward both params through Python create_scalar_index

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request python

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments