Skip to content

Add LroOptions for Long Running Operation#519

Merged
parthban-db merged 3 commits into
mainfrom
parthban-db/stack/add-lro-options
Oct 9, 2025
Merged

Add LroOptions for Long Running Operation#519
parthban-db merged 3 commits into
mainfrom
parthban-db/stack/add-lro-options

Conversation

@parthban-db

@parthban-db parthban-db commented Oct 9, 2025

Copy link
Copy Markdown
Contributor

🥞 Stacked PR

Use this link to review incremental changes.


What changes are proposed in this pull request?

This PR adds LroOptions for Long-Running Operations. These options are similar to those in the Go SDK. Currently, this only has a timeout, but this can be extended in the future.

How is this tested?

N/A

NO_CHANGELOG=true

@hectorcast-db hectorcast-db 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.

Small nit, otherwise LGTM


@Override
public String toString() {
return "LroOptions{" + "timeout=" + timeout + '}';

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.

nit: different quotes?

@github-actions

github-actions Bot commented Oct 9, 2025

Copy link
Copy Markdown
Contributor

If integration tests don't run automatically, an authorized user can run them manually by following the instructions below:

Trigger:
go/deco-tests-run/sdk-java

Inputs:

  • PR number: 519
  • Commit SHA: d8def6b5b1c2bcd52cc66823fdf1affb9958c735

Checks will be approved automatically on success.

@parthban-db parthban-db added this pull request to the merge queue Oct 9, 2025
Merged via the queue into main with commit eee4632 Oct 9, 2025
16 checks passed
@parthban-db parthban-db deleted the parthban-db/stack/add-lro-options branch October 9, 2025 12:42
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