Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Expose LongPollApiMaxWaitTime per request #218

Open
longquanzheng opened this issue Feb 21, 2024 · 1 comment
Open

Expose LongPollApiMaxWaitTime per request #218

longquanzheng opened this issue Feb 21, 2024 · 1 comment
Labels
enhancement New feature or request

Comments

@longquanzheng
Copy link
Contributor

iwf-java-sdk/src/main/java/io/iworkflow/core/UnregisteredClient.java

Line 369 in 5d52f3d

request.waitTimeSeconds(clientOptions.getLongPollApiMaxWaitTimeSeconds().get());

It may be hard to get the timeout, or maybe expose the value via API
@longquanzheng longquanzheng added the bug Something isn't working label Feb 21, 2024
@longquanzheng longquanzheng changed the title Fix waitTime by replacing with the workflow timeout Expose LongPollApiMaxWaitTime instead of hardcoded value Jul 1, 2024
@longquanzheng longquanzheng changed the title Expose LongPollApiMaxWaitTime instead of hardcoded value Expose LongPollApiMaxWaitTime per request Jul 8, 2024
@longquanzheng
Copy link
Contributor Author

May not need this anymore because not sure people want to set it per request

@longquanzheng longquanzheng added enhancement New feature or request and removed bug Something isn't working labels Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@longquanzheng