Skip to content

Implement VSETATTR#1476

Open
tiagonapoli wants to merge 315 commits intomainfrom
tiagonapoli/add-vsetattr
Open

Implement VSETATTR#1476
tiagonapoli wants to merge 315 commits intomainfrom
tiagonapoli/add-vsetattr

Conversation

@tiagonapoli
Copy link
Collaborator

No description provided.

…ets read/write/delete callbacks working _during_ index creation; this will simplify restoration and cleanup some stuff on the DiskANN side
tiagonapoli and others added 5 commits January 23, 2026 10:37
* Implement VGETATTR

* Fix test

* update comments

* Minor refactor on test

* ADdress comments

* Fix format

* Fix build

* Handle unknown GarnetStatus

---------

Co-authored-by: tiagonapoli <tiagonapoli@microsoft.com>
* DiskANN grid E2E tests

* nit

* Address comments

---------

Co-authored-by: tiagonapoli <tiagonapoli@microsoft.com>
@tiagonapoli tiagonapoli force-pushed the tiagonapoli/add-vsetattr branch from 9fba39e to 030486d Compare January 27, 2026 15:43
@tiagonapoli tiagonapoli force-pushed the tiagonapoli/add-vsetattr branch from 030486d to ba2ad23 Compare January 29, 2026 14:15
@tiagonapoli tiagonapoli marked this pull request as ready for review January 30, 2026 17:52
@hailangx
Copy link
Member

hailangx commented Feb 2, 2026

@copilot does the VSETATTR and vadd converge on the attributes write ?

Copy link
Contributor

Copilot AI commented Feb 2, 2026

@hailangx I've opened a new pull request, #1531, to work on those changes. Once the pull request is ready, I'll request review from you.

Base automatically changed from vectorApiPoC to main February 6, 2026 21:50
@kevin-montrose kevin-montrose changed the base branch from main to vectorApi-storev1 February 9, 2026 15:30
Base automatically changed from vectorApi-storev1 to main February 25, 2026 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants