support mps flow resource #2192
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
andrew-tx
reviewed
Oct 11, 2023
tencentcloud/resource_tc_mps_flow.go
Outdated
| return err | ||
| } | ||
|
|
||
| flowId = *response.Response.Info.FlowId |
There was a problem hiding this comment.
response.Response.Info可能为nil吗
There was a problem hiding this comment.
有可能的。但是这种情况比较少了。前面逻辑会判断err的，如果请求有问题，会通过error报错；如果请求正常，但是这个返回值没有填写，这种情况比较少了。所以一般不去做校验了。
我加一下判断。
lyu571
force-pushed
the
feat/support-mps-flow
branch
from
d4d5262 to
c52fe98
Compare
lyu571
force-pushed
the
feat/support-mps-flow
branch
from
c52fe98 to
3c51fdd
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
tencentcloud_mps_flow
tencentcloud_mps_event
tencentcloud_mps_start_flow_operation（文档暂不暴露）