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.

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

handle still in-progress requests in getStoredTaskResultByRequestId #86

Merged

Conversation

WangLiNaruto
Copy link
Contributor

@WangLiNaruto WangLiNaruto force-pushed the feature/handler_in_process branch 2 times, most recently from 2ecb5f7 to 1468f7e Compare November 27, 2024 01:58
@WangLiNaruto WangLiNaruto force-pushed the feature/handler_in_process branch from 1468f7e to 0e83019 Compare November 27, 2024 02:04
Copy link
Contributor

@swaroopar swaroopar left a comment

Choose a reason for hiding this comment

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

LGTM

} catch (IOException e) {
log.error("Failed to retrieve OpenTofuResult for requestId: {}", requestId, e);
throw new ResultAlreadyReturnedOrRequestIdInvalidException(
"Failed to retrieve OpenTofuResult for requestId: " + requestId);
}
}

private boolean isDeployInProcess(String requestId) {
Copy link
Contributor

Choose a reason for hiding this comment

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

please rename this to isDeployStillInProgress()

@swaroopar swaroopar merged commit b2b9d88 into eclipse-xpanse:main Nov 27, 2024
3 checks passed
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.

handle still in-progress requests in getStoredTaskResultByRequestId
2 participants