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
feat: add s3 api. #49
Conversation
合并到develop分支吧 |
@@ -55,7 +55,7 @@ For a Maven project, add the following to your `pom.xml` file: | |||
<dependency> | |||
<groupId>group.rxcloud</groupId> | |||
<artifactId>cloud-runtimes-api</artifactId> | |||
<version>1.15.RELEASE</version> | |||
<version>1.16-SNAPSHOT</version> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1.16.RELEASE
@@ -69,6 +69,6 @@ For a Gradle project, add the following to your `build.gradle` file: | |||
dependencies { | |||
// ... | |||
// https://mvnrepository.com/artifact/group.rxcloud/cloud-runtimes-api | |||
implementation group: 'group.rxcloud', name: 'cloud-runtimes-api', version: '1.15.RELEASE' | |||
implementation group: 'group.rxcloud', name: 'cloud-runtimes-api', version: '1.16-SNAPSHOT' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
1.16.RELEASE
* @param putObjectInput | ||
* @return | ||
*/ | ||
PutObjectOutput putObject(PutObjectInput putObjectInput); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
不需要考虑异步情况吗?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
如果考虑未来有异步调用,建议和其他api保持一致,使用mono/flux。
这样既可以有异步,又可以有同步。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
我觉得还是考虑异步吧,s3不像redis,redis确保响应很快所以就做成同步的了,异步意义不大。
但s3会涉及到长时间的大数据传输,如果不支持异步,未来可能会遇到瓶颈。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
赞同你的建议,我改一下
Description
add s3 api.
Issue reference
#48
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list: