Visitar URL original
Graph analytical view counts an edge twice after a background compaction · Issue #11192 · ArcadeData/arcadedb · GitHub
Skip to content

Graph analytical view counts an edge twice after a background compaction #11192

Closed

Description

@lvca

A graph analytical view could count an edge twice after a background compaction, and kept the wrong count until the next compaction or rebuild.

A transaction's records become readable as soon as it commits, and the view hears about the change a moment later. If a compaction's scan read the new edge in between, and the change notice arrived after the compaction swapped in its result, the edge was added again on top of a base that already held it. Full builds were protected against this since #8378, but compactions were not. Path counts, degrees and algorithms on the view then answer with the extra edge.

Found through Issue10189AsyncViewServesWhileUpdatingTest, which fails intermittently in CI. Issue10208ViewVertexSubtypesTest also fails intermittently, but there the view is right: the test raced the rebuild its own assertion starts.

Activity

  1. added this to the 26.11.1 milestone on Nov 3, 2026
  2. self-assigned this
    on Nov 3, 2026
  3. added
    concurrencyThreading / concurrency / MVCC
    engineCore engine: document/schema API
    severity:majorA feature is broken or returns wrong results, with no practical workaround
    on Nov 3, 2026
  4. added a commit that references this issue on Nov 3, 2026
  5. lvca commented on Nov 3, 2026

    @lvca
    MemberAuthor

    Fixed in #11193.

    A committed edge is readable right away, but the graph analytical view hears about it a moment later. If a background compaction read the edge and the notice arrived after the compaction had swapped in its result, the view added the edge a second time. Degrees, path counts and algorithms on the view then saw the extra edge until the next compaction.

    Compactions now record what their scan read and check every late change against it, the same protection full builds already had. The new regression test reproduced the double count every time before the fix.


    ❤️ Thanks for helping make ArcadeDB better! ArcadeDB is free and open source, sustained by its community. If it's useful to you or your company, please consider becoming a sponsor to keep development going.

  6. removed
    in progressThe team is actively working on this issue
    on Nov 3, 2026
  7. added a commit that references this issue on Nov 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugconcurrencyThreading / concurrency / MVCCengineCore engine: document/schema APIseverity:majorA feature is broken or returns wrong results, with no practical workaround

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions