Skip to content

Re-enable test_jsonbench.py - #6773

Merged
ryzhyk merged 1 commit into
feldera:mainfrom
mihaibudiu:jsonbench
Aug 3, 2026
Merged

Re-enable test_jsonbench.py#6773
ryzhyk merged 1 commit into
feldera:mainfrom
mihaibudiu:jsonbench

Conversation

@mihaibudiu

Copy link
Copy Markdown
Contributor

Fixes #6709

Let's see if the test passes after the recent improvements in CI.

Signed-off-by: Mihai Budiu <mbudiu@feldera.com>
@mihaibudiu
mihaibudiu enabled auto-merge July 30, 2026 18:16
@gz

gz commented Jul 30, 2026

Copy link
Copy Markdown
Contributor

you can control this also editing the github variable, no need to do a commit

@mihaibudiu
mihaibudiu added this pull request to the merge queue Jul 30, 2026

@mythical-fred mythical-fred left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Straightforward rename to re-enable the previously disabled jsonbench test. CI will be the judge — if it times out again (issue #6709 noted two ~1h timeouts), we'll need to actually address the perf/timeout issue rather than just re-enabling. But worth a shot given recent CI improvements. Clean commit message with DCO trailer.

@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 30, 2026
@mihaibudiu
mihaibudiu added this pull request to the merge queue Jul 30, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 30, 2026
@mihaibudiu
mihaibudiu added this pull request to the merge queue Jul 30, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 31, 2026
@blp

blp commented Jul 31, 2026

Copy link
Copy Markdown
Member

@mihaibudiu The test failed

@blp
blp added this pull request to the merge queue Jul 31, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 31, 2026
@ryzhyk
ryzhyk added this pull request to the merge queue Aug 2, 2026
Merged via the queue into feldera:main with commit 8a4b73b Aug 3, 2026
1 check passed
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.

[CI] test_jsonbench.py has been disabled

5 participants