GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Flaky: dynamodb_conditional_put_skips_existing_row (1 row instead of 2) · Issue #7908 · feldera/feldera

Flaky: dynamodb_conditional_put_skips_existing_row (1 row instead of 2) #7908

New issue
Copy link
New issue
Copy link
Open
Open
Flaky: dynamodb_conditional_put_skips_existing_row (1 row instead of 2)#7908
Copy link
Assignees
bestouff
Labels
connectorsIssues related to the adapters/connectors crateIssues related to the adapters/connectors cratefred-observeTest failure diagnosed by the fred-observe botTest failure diagnosed by the fred-observe bot

Description

@mythical-fred-oss
mythical-fred-oss
bot
opened on Nov 10, 2026
Issue body actions

Verdict

Flaky test, not caused by PR #7905. dynamodb_conditional_put_skips_existing_row failed in the Adapter Tests job of a merge-queue run for #7905, a PR that only touches checkpoint/GC code (crates/connectors/src/controller.rs, crates/dbsp/src/circuit/checkpointer.rs, crates/dbsp/src/circuit/dbsp_handle.rs, crates/connectors/src/engine.rs, crates/static-connectors/src/engine.rs). git diff against main shows zero changes under crates/connectors/src/integrated/dynamodb/.

Evidence

  • Job: Adapter Tests / Test Adapters (amd64)
  • Failure (log line ~2700):
    thread 'main' panicked at crates/connectors/src/integrated/dynamodb/test.rs:1477:5:
    assertion `left == right` failed
      left: 1
     right: 2
    
    Line 1477 is assert_eq!(rows.len(), 2); in dynamodb_conditional_put_skips_existing_row: after a conditional-put transaction that re-emits an existing key (expected to be skipped) alongside one new key, the table ends up with only 1 row instead of 2 — the new row didn't land, or the whole transaction was dropped.
  • The test runs against DynamoDbLocal (an emulator container); this shape of failure (fewer rows than expected after a single batch) is consistent with a transaction/container timing race rather than a logic bug, and no other test in the same suite run failed in this area.

Next step

  • Re-running the merge queue is likely to pass; this looks like an isolated flake.
  • Worth checking whether DynamoDbLocal's conditional-transaction handling has any known timing sensitivity, and whether the test should retry the scan or add a settle delay.

Activity

  1. mythical-fred-oss
    assigned
    bestouff
    on Nov 10, 2026
  2. mythical-fred-oss
    added
    connectorsIssues related to the adapters/connectors crateIssues related to the adapters/connectors crate
    fred-observeTest failure diagnosed by the fred-observe botTest failure diagnosed by the fred-observe bot
    on Nov 10, 2026
  3. yermakoffivan
    added a commit that references this issue on Nov 11, 2026

    [connectors] Make DynamoDB test failures show their cause

    c6ede75
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

  • @bestouff
    bestouff

Labels

connectorsIssues related to the adapters/connectors crateIssues related to the adapters/connectors cratefred-observeTest failure diagnosed by the fred-observe botTest failure diagnosed by the fred-observe bot

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

    You are visitor 01402503 — GeezerVison does not store the pages you look up.
    Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.