Skip to content

Fix MDBX transaction overlaps in CategoryIndex & integrate single transaction#236

Open
MithunEndee wants to merge 2 commits intomasterfrom
fix-single-transaction
Open

Fix MDBX transaction overlaps in CategoryIndex & integrate single transaction#236
MithunEndee wants to merge 2 commits intomasterfrom
fix-single-transaction

Conversation

@MithunEndee
Copy link
Copy Markdown

Pull Request

Summary

Integrates the single transaction and resolves MDBX_TXN_OVERLAPPING crashes in CategoryIndex. Updated the indexing methods (add, remove, add_batch) to reuse existing write transactions instead of accidentally opening conflicting nested read transactions.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Refactor / cleanup

Related Issue

Closes #

Checklist

  • Code compiles and tests pass
  • New tests added where applicable
  • Documentation updated if needed
  • No unintended breaking changes

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 7, 2026

VectorDB Benchmark - Ready To Run

CI Passed ([lint + unit tests] (https://github.com/endee-io/endee/actions/runs/25476601767)) - benchmark options unlocked.

Post one of the command below. Only members with write access can trigger runs.


Available Modes

Mode Command What runs
Dense /correctness_benchmarking dense HNSW insert throughput · query P50/P95/P99 · recall@10 · concurrent QPS
Hybrid /correctness_benchmarking hybrid Dense + sparse BM25 fusion · same suite + fusion latency overhead

Infrastructure

Server Role Instance
Endee Server Endee VectorDB — code from this branch t2.large
Benchmark Server Benchmark runner t3a.large

Both servers start on demand and are always terminated after the run — pass or fail.


How Correctness Benchmarking Works

1. Post /correctness_benchmarking <mode>
2. Endee Server Create  →  this branch's code deployed  →  Endee starts in chosen mode
3. Benchmark Server Create  →  benchmark suite transferred
4. Benchmark Server runs correctness benchmarking against Endee Server
5. Results posted back here  →  pass/fail + full metrics table
6. Both servers terminated   →  always, even on failure

After a new push, CI must pass again before this menu reappears.

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.

2 participants