Skip to content

Downgrade checkpoint fallback log from error to warn#205

Merged
tonyhb merged 1 commit intoinngest:mainfrom
aluzzardi:fix/downgrade-checkpointing-error
Mar 11, 2026
Merged

Downgrade checkpoint fallback log from error to warn#205
tonyhb merged 1 commit intoinngest:mainfrom
aluzzardi:fix/downgrade-checkpointing-error

Conversation

@aluzzardi
Copy link
Contributor

@aluzzardi aluzzardi commented Mar 11, 2026

Downgrade checkpoint fallback log from error to warn


Note

Downgrades the log level for checkpoint fallback from Error to Warn in AppendOp, reflecting that falling back to async response is a handled/expected degradation path rather than an unrecoverable error.

Written by Mendral for commit a4602ee.

Signed-off-by: Andrea Luzzardi <andrea@luzzardi.com>
@changeset-bot
Copy link

changeset-bot bot commented Mar 11, 2026

⚠️ No Changeset found

Latest commit: a4602ee

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

@mendral-app mendral-app bot left a comment

Choose a reason for hiding this comment

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

LGTM

Change is correct — falling back to async response is a recoverable path, so Warn is the appropriate severity. No security, correctness, or data loss concerns.

Tag @mendral-app with feedback or questions. View session

@tonyhb tonyhb merged commit fce4403 into inngest:main Mar 11, 2026
5 checks passed
@aluzzardi aluzzardi deleted the fix/downgrade-checkpointing-error branch March 11, 2026 23:45
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