-
Notifications
You must be signed in to change notification settings - Fork 140
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
[테크니컬 라이팅] 망쵸(김주환) 미션 제출합니다. #572
Open
3Juhwan
wants to merge
6
commits into
woowacourse:3juhwan
Choose a base branch
from
3Juhwan:level4
base: 3juhwan
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
+122
−0
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
This reverts commit 130ca6a.
kunsanglee
suggested changes
Nov 5, 2024
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.
안녕하세요 망쵸 👍
잘 작성해주셔서 문장이 약간 어색한 부분에 대한 코멘트만 남겼습니다.
고생 많으셨습니다 😇
Comment on lines
+62
to
+66
이 방식은 영수증 업로드를 구현하던 초기에 논의되었다. 구현하기 위해서는 서버에서 이미지를 직접 받아서 업로드해야 한다. 서버가 직접 이미지를 업로드하는 방법에는 SpringBoot를 이용한 Multipart 업로드와 Stream을 이용하는 방법이 있는데, 각 방법은 명확한 단점이 있다. Multipart 방법은 서버의 힙 메모리와 디스크를 많이 사용하고, Stream 방법은 네트워크에 영향을 많이 받는다. 때문에 우리 팀은 AWS S3의 pre-signed URL 업로드 방식을 선택했다. 따라서 클라이언트는 이미지를 S3에 직접 업로드한다. | ||
|
||
더 큰 문제는 별도의 서버를 구성하는 데에 발생하는 비용이다. 서버를 직접 구현해야 할 뿐더러, 장애가 발생했을 때의 재시도와 롤백을 고민해야 한다. 서버를 지속적으로 유지보수해야 하는 것도 큰 비용이다. | ||
|
||
우리 팀은 pre-signed URL 업로드 방법을 사용할 수 있고 비용이 적게 드는 방식을 택하고 싶어서, 다른 방식을 고민했다. |
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.
Suggested change
이 방식은 영수증 업로드를 구현하던 초기에 논의되었다. 구현하기 위해서는 서버에서 이미지를 직접 받아서 업로드해야 한다. 서버가 직접 이미지를 업로드하는 방법에는 SpringBoot를 이용한 Multipart 업로드와 Stream을 이용하는 방법이 있는데, 각 방법은 명확한 단점이 있다. Multipart 방법은 서버의 힙 메모리와 디스크를 많이 사용하고, Stream 방법은 네트워크에 영향을 많이 받는다. 때문에 우리 팀은 AWS S3의 pre-signed URL 업로드 방식을 선택했다. 따라서 클라이언트는 이미지를 S3에 직접 업로드한다. | |
더 큰 문제는 별도의 서버를 구성하는 데에 발생하는 비용이다. 서버를 직접 구현해야 할 뿐더러, 장애가 발생했을 때의 재시도와 롤백을 고민해야 한다. 서버를 지속적으로 유지보수해야 하는 것도 큰 비용이다. | |
우리 팀은 pre-signed URL 업로드 방법을 사용할 수 있고 비용이 적게 드는 방식을 택하고 싶어서, 다른 방식을 고민했다. | |
이 방식은 영수증 업로드를 구현하던 시점에 논의했다. 구현하기 위해서는 서버에서 이미지를 직접 받아서 업로드해야 한다. 서버가 직접 이미지를 업로드하는 방법에는 SpringBoot를 이용한 Multipart 업로드와 Stream을 이용하는 방법이 있는데, 각 방법은 명확한 단점이 있다. Multipart 방법은 서버의 힙 메모리와 디스크를 많이 사용하고, Stream 방법은 네트워크에 영향을 많이 받는다. 때문에 우리 팀은 | |
앞선 방법들의 단점을 피하기 위해 AWS S3의 pre-signed URL 업로드 방식을 선택했고, 클라이언트는 이미지를 S3에 직접 업로드한다. | |
더 큰 문제는 별도의 서버를 구성하는 데에 발생하는 비용이다. 서버를 직접 구현해야 할 뿐더러, 장애가 발생했을 때의 재시도와 롤백을 고민해야 한다. 서버를 지속적으로 유지보수해야 하는 것도 큰 비용이다. | |
우리 팀은 pre-signed URL 업로드 방법을 사용할 수 있었지만, 비용이 적게 드는 방식을 택하고 싶어서, 다른 방식을 고민했다. |
말을 조금 다듬어 봤습니다.
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.
fork한 리포지토리의 브랜치 명 변경으로 이전 PR이 닫혔습니다. 다시 제출합니다.