Skip to content

chore: Make timeoutCompletionStage accept java Duration.#2063

Merged
He-Pin merged 1 commit intomainfrom
timeoutCompletionStage
Aug 23, 2025
Merged

chore: Make timeoutCompletionStage accept java Duration.#2063
He-Pin merged 1 commit intomainfrom
timeoutCompletionStage

Conversation

@He-Pin
Copy link
Copy Markdown
Member

@He-Pin He-Pin commented Aug 23, 2025

Motivation:

This method should accept Java's duration

Should be backported to 1.2.0 too

@He-Pin He-Pin requested a review from pjfanning August 23, 2025 10:15
@He-Pin He-Pin added this to the 2.0.x milestone Aug 23, 2025
@He-Pin He-Pin force-pushed the timeoutCompletionStage branch 2 times, most recently from fc2b594 to 6733daa Compare August 23, 2025 10:26
@He-Pin He-Pin modified the milestones: 2.0.x, 1.2.0 Aug 23, 2025
@He-Pin He-Pin force-pushed the timeoutCompletionStage branch from 6733daa to 76e1550 Compare August 23, 2025 11:56
@pjfanning pjfanning modified the milestones: 1.2.0, 2.0.x Aug 23, 2025
@He-Pin He-Pin force-pushed the timeoutCompletionStage branch from 76e1550 to 9c7f5e5 Compare August 23, 2025 14:15
@He-Pin He-Pin requested a review from pjfanning August 23, 2025 15:28
Copy link
Copy Markdown
Member

@pjfanning pjfanning left a comment

Choose a reason for hiding this comment

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

lgtm

@He-Pin He-Pin merged commit 442ecd7 into main Aug 23, 2025
9 of 10 checks passed
@He-Pin He-Pin deleted the timeoutCompletionStage branch August 23, 2025 18:09
He-Pin added a commit to He-Pin/incubator-pekko that referenced this pull request Aug 23, 2025
He-Pin added a commit that referenced this pull request Aug 23, 2025
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