Skip to content

Fix java first program tutorial setMaximumAttempts value inconsistency#453

Open
illenko wants to merge 2 commits into
temporalio:mainfrom
illenko:fix-java-maximum-attempts-inconsistency
Open

Fix java first program tutorial setMaximumAttempts value inconsistency#453
illenko wants to merge 2 commits into
temporalio:mainfrom
illenko:fix-java-maximum-attempts-inconsistency

Conversation

@illenko

@illenko illenko commented Jan 18, 2026

Copy link
Copy Markdown

What was changed

Value for setMaximumAttempts in RetryOptions config in Run your first Temporal application with the Java SDK tutorial (https://learn.temporal.io/getting_started/java/first_program_in_java/).

Why?

Old value (5000) was not realistic for the scenario and conflicted with the sentence "it'll retry the failed Activity for 3 attempts". Set it to 3 to be more consistent with .NET (https://learn.temporal.io/getting_started/dotnet/first_program_in_dotnet/) and Python tutorials (https://learn.temporal.io/getting_started/python/first_program_in_python/).

@illenko illenko requested a review from a team as a code owner January 18, 2026 10:13
@vercel

vercel Bot commented Jan 18, 2026

Copy link
Copy Markdown

@illenko is attempting to deploy a commit to the Temporal Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant

CLAassistant commented Jan 18, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@illenko

illenko commented Feb 10, 2026

Copy link
Copy Markdown
Author

@angelazhou32 maybe you can help with review and merge?

@illenko

illenko commented Mar 3, 2026

Copy link
Copy Markdown
Author

@angelazhou32 could you please check it?

@angelazhou32 angelazhou32 left a comment

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.

Thank you for your contribution!

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.

3 participants