Skip to content

fix: optimize preview api#1412

Merged
dipratap merged 5 commits intomainfrom
pall
Mar 25, 2026
Merged

fix: optimize preview api#1412
dipratap merged 5 commits intomainfrom
pall

Conversation

@dipratap
Copy link
Copy Markdown
Contributor

@dipratap dipratap commented Mar 8, 2026

Please ensure your pull request adheres to the following guidelines:

  • make sure to link the related issues in this description
  • when merging / squashing, make sure the fixed issue references are visible in the commits, for easy compilation of release notes

Related Issues

Thanks for contributing!

const {
warmupDelayMs = isOptimized ? 750 : 0, // milliseconds
maxRetries = 3,
maxRetries = 5,
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.

Please retry at a delay of 2 sec with a max retry count of 8

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I am a bit concerned that adding a retry delay of 2 sec might result in more api timeouts (16s time breach) since we wait for API call also for 5s max. I have increased it to 1.5s for now, let me know your thoughts.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 9, 2026

This PR will trigger a patch release when merged.

@dipratap dipratap requested a review from nit23uec March 9, 2026 04:43
@dipratap dipratap merged commit 3efb9de into main Mar 25, 2026
7 checks passed
@dipratap dipratap deleted the pall branch March 25, 2026 06:02
solaris007 pushed a commit that referenced this pull request Mar 25, 2026
@solaris007
Copy link
Copy Markdown
Member

🎉 This PR is included in version @adobe/spacecat-shared-tokowaka-client-v1.12.2 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants