Skip to content

[dbsp] Test for the InputUpsert operator.#5995

Merged
gz merged 1 commit intomainfrom
input_map_test
Apr 7, 2026
Merged

[dbsp] Test for the InputUpsert operator.#5995
gz merged 1 commit intomainfrom
input_map_test

Conversation

@ryzhyk
Copy link
Copy Markdown
Contributor

@ryzhyk ryzhyk commented Apr 4, 2026

I added this test when hunting down a potential bug. The test didn't find any bugs, but it's a useful test for a subtle piece of logic in InputUpsert, I think it's worth keeping.

I added this test when hunting down a potential bug. The test didn't find any
bugs, but it's a useful test for a subtle piece of logic in InputUpsert, I
think it's worth keeping.

Signed-off-by: Leonid Ryzhyk <ryzhyk@gmail.com>
@ryzhyk ryzhyk requested a review from blp April 4, 2026 17:08
@ryzhyk ryzhyk added DBSP core Related to the core DBSP library QA Testing and quality assurance labels Apr 4, 2026
.unwrap();

let mut state: HashMap<u64, u64> = HashMap::new();
let mut rng = ChaCha8Rng::seed_from_u64(0x_6D61_705F_7374_6167_u64);
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like this number

Copy link
Copy Markdown

@mythical-fred mythical-fred left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ryzhyk ryzhyk added this pull request to the merge queue Apr 7, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Apr 7, 2026
@gz gz added this pull request to the merge queue Apr 7, 2026
Merged via the queue into main with commit 938e649 Apr 7, 2026
1 check passed
@gz gz deleted the input_map_test branch April 7, 2026 05:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DBSP core Related to the core DBSP library QA Testing and quality assurance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants