Fix spelling errors in streaming API documentation by Copilot · Pull Request #715 · microsoft/DiskANN

Copilot · 2026-02-04T03:59:22Z

Fixes three spelling errors in the public documentation of diskann-benchmark-core/src/streaming/api.rs:

Line 16: "maintance" → "maintenance"
Line 71: "exector" → "executor"
Line 74: "aggreate" → "aggregate"

These corrections improve the quality of user-facing documentation.

Changes Made

Fixed spelling error in Stream trait documentation: "maintance" → "maintenance"
Fixed spelling error in Executor trait documentation: "exector" → "executor"
Fixed spelling error in Executor trait documentation: "aggreate" → "aggregate"

Testing

✅ Rust compilation passes (cargo check -p diskann-benchmark-core)
✅ Code review completed with no issues found

The changes are documentation-only and do not affect functionality.

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

hildebrandmw · 2026-02-04T19:09:49Z

@copilot - Please rebase this PR.

Copilot · 2026-02-04T19:11:00Z

@copilot - Please rebase this PR.

I don't have the ability to perform git rebase operations as they require force push capabilities. You'll need to rebase this PR manually or grant me additional permissions to handle branch operations.

hildebrandmw · 2026-02-04T19:14:29Z

@copilot - Can you do a merge commit with main instead?

Copilot · 2026-02-04T19:15:56Z

@copilot - Can you do a merge commit with main instead?

Done. Merged main into this branch with commit e6f2270.

hildebrandmw · 2026-02-04T19:16:54Z

@copilot - This PR now no longer fixes the identified spelling errors. Can you fix this please?

Co-authored-by: hildebrandmw <24898651+hildebrandmw@users.noreply.github.com>

Copilot · 2026-02-04T19:19:20Z

@copilot - This PR now no longer fixes the identified spelling errors. Can you fix this please?

Fixed all three spelling errors in commit 140d7e8:

"maintance" → "maintenance"
"exector" → "executor"
"aggreate" → "aggregate"

Copilot

Pull request overview

This PR corrects three spelling errors in the documentation comments for the streaming API in the DiskANN benchmark core module. The changes address feedback from a previous review requesting issue details for these spelling errors.

Changes:

Fixed spelling error: "maintance" → "maintenance" in the Stream trait documentation
Fixed spelling error: "exector" → "executor" in the Executor trait documentation
Fixed spelling error: "aggreate" → "aggregate" in the Executor trait documentation

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

codecov-commenter · 2026-02-04T19:40:04Z

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.01%. Comparing base (d9d6ce2) to head (140d7e8).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #715   +/-   ##
=======================================
  Coverage   89.01%   89.01%           
=======================================
  Files         428      428           
  Lines       78151    78151           
=======================================
  Hits        69563    69563           
  Misses       8588     8588

Files with missing lines	Coverage Δ
diskann-benchmark-core/src/streaming/api.rs	`98.37% <ø> (ø)`

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hildebrandmw

## What's Changed ### API Breaking Changes * Remove the `experimental_avx512` feature. by @hildebrandmw in #732 * Use VirtualStorageProvider::new_overlay(test_data_root()) in tests by @Copilot in #726 * save and load max_record_size and leaf_page_size for bftrees by @backurs in #724 * [multi-vector] Verify `Standard` won't overflow in its constructor. by @hildebrandmw in #757 * VirtualStorageProvider: Make new() private, add new_physical by @Copilot in #764 * [minmax] Refactor full query by @arkrishn94 in #770 * Bump diskann-quantization to edition 2024. by @hildebrandmw in #772 ### Additions * [multi-vector] Enable cloning of `Mat` and friends. by @hildebrandmw in #759 * adding bftreepaths in mod.rs by @backurs in #775 * [quantization] Add `as_raw_ptr`. by @hildebrandmw in #774 ### Bug Fixes * Fix `diskann` compilation without default-features and add CI tests. by @hildebrandmw in #722 ### Docs and Comments * Updating the benchmark README to use diskann-benchmark by @bryantower in #709 * Fix doc comment: Windows line endings are \r\n not \n\r by @Copilot in #717 * Fix spelling errors in streaming API documentation by @Copilot in #715 * Add performance diagnostic to `diskann-benchmark` by @hildebrandmw in #744 * Add agents.md onboarding guide for coding agents by @Copilot in #765 * [doc] Fix lots of little typos in `diskann-wide` by @hildebrandmw in #771 ### Performance * [diskann-wide] Optimize `load_simd_first` for 8-bit and 16-bit element types. by @hildebrandmw in #747 ### Dependencies * Bump bytes from 1.11.0 to 1.11.1 by @dependabot[bot] in #723 * [diskann] Add note on the selection of `PruneKind` in `graph::config::Builder`. by @hildebrandmw in #734 * [diskann-providers] Remove the LRU dependency and make `vfs` and `serde_json` optional. by @hildebrandmw in #733 ### Infrastructure * Add initial QEMU tests for `diskann-wide`. by @hildebrandmw in #719 * [CI] Skip coverage for Dependabot. by @hildebrandmw in #725 * Add miri test coverage to CI workflow by @Copilot in #729 * [CI] Add minimal ARM checks by @hildebrandmw in #745 * Enable CodeQL security analysis by @Copilot in #754 ## New Contributors * @backurs made their first contribution in #724 * @arkrishn94 made their first contribution in #770 **Full Changelog**: 0.45.0...0.46.0

hildebrandmw and others added 3 commits February 3, 2026 19:45

Roll-up of internal changes.

3154e97

Bump Rust version in CI.

bb765fe

Initial plan

5c456c8

Copilot AI assigned Copilot and hildebrandmw Feb 4, 2026

Copilot AI mentioned this pull request Feb 4, 2026

[Sync] Roll-up of internal changes for development change-over. #714

Merged

Copilot started work on behalf of hildebrandmw February 4, 2026 03:59 View session

Copilot AI changed the title ~~[WIP] Create issue to address feedback on roll-up PR~~ Provide issue details for spelling errors in streaming API docs Feb 4, 2026

Copilot AI requested a review from hildebrandmw February 4, 2026 04:00

Copilot finished work on behalf of hildebrandmw February 4, 2026 04:00

Copilot AI mentioned this pull request Feb 4, 2026

Fix spelling errors in streaming API documentation #716

Closed

Base automatically changed from mhildebr/sync-squashed to main February 4, 2026 17:26

Copilot started work on behalf of hildebrandmw February 4, 2026 19:10 View session

Copilot finished work on behalf of hildebrandmw February 4, 2026 19:11

Copilot started work on behalf of hildebrandmw February 4, 2026 19:14 View session

Merge remote-tracking branch 'origin/main' into copilot/sub-pr-714

e6f2270

Copilot finished work on behalf of hildebrandmw February 4, 2026 19:16

Copilot started work on behalf of hildebrandmw February 4, 2026 19:17 View session

Fix spelling errors in streaming API documentation

140d7e8

Co-authored-by: hildebrandmw <24898651+hildebrandmw@users.noreply.github.com>

hildebrandmw approved these changes Feb 4, 2026

View reviewed changes

hildebrandmw marked this pull request as ready for review February 4, 2026 19:20

hildebrandmw requested review from a team and Copilot February 4, 2026 19:20

Copilot started reviewing on behalf of hildebrandmw February 4, 2026 19:20 View session

Copilot AI reviewed Feb 4, 2026

View reviewed changes

Copilot AI changed the title ~~Provide issue details for spelling errors in streaming API docs~~ Fix spelling errors in streaming API documentation Feb 4, 2026

Copilot AI requested a review from hildebrandmw February 4, 2026 19:23

Copilot finished work on behalf of hildebrandmw February 4, 2026 19:23

hildebrandmw approved these changes Feb 4, 2026

View reviewed changes

harsha-simhadri approved these changes Feb 5, 2026

View reviewed changes

suhasjs approved these changes Feb 5, 2026

View reviewed changes

hildebrandmw merged commit 5c6cdba into main Feb 5, 2026
15 checks passed

hildebrandmw deleted the copilot/sub-pr-714 branch February 5, 2026 23:27

hildebrandmw mentioned this pull request Feb 13, 2026

Bump version to 0.46.0 #776

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

Fix spelling errors in streaming API documentation#715

Fix spelling errors in streaming API documentation#715
hildebrandmw merged 5 commits intomainfrom
copilot/sub-pr-714

Copilot AI commented Feb 4, 2026 •

edited

Loading

Uh oh!

hildebrandmw commented Feb 4, 2026

Uh oh!

Copilot AI commented Feb 4, 2026

Uh oh!

hildebrandmw commented Feb 4, 2026

Uh oh!

Copilot AI commented Feb 4, 2026

Uh oh!

hildebrandmw commented Feb 4, 2026

Uh oh!

Copilot AI commented Feb 4, 2026

Uh oh!

Copilot AI left a comment

Uh oh!

codecov-commenter commented Feb 4, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Comments

Conversation

Copilot AI commented Feb 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes Made

Testing

Uh oh!

hildebrandmw commented Feb 4, 2026

Uh oh!

Copilot AI commented Feb 4, 2026

Uh oh!

hildebrandmw commented Feb 4, 2026

Uh oh!

Copilot AI commented Feb 4, 2026

Uh oh!

hildebrandmw commented Feb 4, 2026

Uh oh!

Copilot AI commented Feb 4, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

codecov-commenter commented Feb 4, 2026

Codecov Report

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Copilot AI commented Feb 4, 2026 •

edited

Loading