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

[ko] Translate 'Distributed Apps' in Korean #1285

Merged
merged 3 commits into from
Sep 8, 2022

Conversation

bconfiden2
Copy link
Contributor

Translate 'Distributed Apps' into Korean

Relevant with #1176

  • [ x ] This PR does not contain plagiarism
    • don’t copy other people’s work unless you are quoting and contributing it to them.
  • [ x ] I have signed off on all commits
    • signing off (ex: git commit -s) is to affirm that commits comply DCO.

@netlify
Copy link

netlify bot commented Sep 2, 2022

Deploy Preview for cncfglossary ready!

Name Link
🔨 Latest commit d5e3277
🔍 Latest deploy log https://app.netlify.com/sites/cncfglossary/deploys/6319a14296dc63000891e520
😎 Deploy Preview https://deploy-preview-1285--cncfglossary.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@bconfiden2
Copy link
Contributor Author

영문 문서의 컨텍스트를 최대한 유지하려고 하긴 했지만 어느 정도 의역한 부분들이 몇 군데 있습니다... 리뷰 부탁드리겠습니다!! 🙇‍♂️ 🙇‍♂️

@seokho-son
Copy link
Collaborator

@bconfiden2
dev-ko 브랜치 관리 작업 중에, upstream/dev-ko에 특정 커밋의 hash가 변경되는 상황이 발생하여

해당 PR에
hash가 변경되기 이전 커밋이 추가되었습니다. (자동으로)
https://github.com/cncf/glossary/pull/1285/commits

git fetch upstream
git rebase upstream/dev-ko
하고 origin remote에 다시 푸시하시면 해당 커밋을 PR에서 삭제할 수 있습니다.
혹시 처리 가능하실까요? :)

@bconfiden2 bconfiden2 force-pushed the 0822-distributed-apps branch from 7f2f4f5 to 7595d50 Compare September 6, 2022 05:47
@bconfiden2
Copy link
Contributor Author

처리하여 푸시하였습니다~! 🙇

Copy link
Collaborator

@yunkon-kim yunkon-kim left a comment

Choose a reason for hiding this comment

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

@bconfiden2 의역을 필요로 하는 원문이었던 것 같습니다 ^^

어렵지 않게 읽고 이해한다는 측면을 최우선으로 생각하였고요. 국문화 하기 난감한 부분들이 있다는 생각이 들어 제안해 주신 국문과 원물을 여러번 비교해서 의견 드렸습니다.

content/ko/distributed-apps.md Outdated Show resolved Hide resolved
content/ko/distributed-apps.md Outdated Show resolved Hide resolved
content/ko/distributed-apps.md Outdated Show resolved Hide resolved

단일 컴퓨터에서 실행 중인 애플리케이션은 곧 단일 장애 지점(Single Point Of Failure)을 나타낸다. 이는 해당 컴퓨터에 장애가 발생할 경우 애플리케이션 역시 사용할 수 없게 된다는 의미이다.
분산 애플리케이션은 종종 [모놀리식 애플리케이션](/ko/monolithic-apps/)과 대조되는데,
모놀리식 애플리케이션은 다양한 컴포넌트들의 규모를 독립적으로 조절할 수 없기 때문에 유연하게 확장(scale)하기가 어렵다.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
모놀리식 애플리케이션은 다양한 컴포넌트들의 규모를 독립적으로 조절할 수 없기 때문에 유연하게 확장(scale)하기가 어렵다.
모놀리식 애플리케이션은 다양한 구성 요소 각각의 규모를 독립적으로 조절할 수 없기 때문에 유연하게 규모를 조절하는 것이 매우 어려운 일이다.

약간 의역을 추가했습니다.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

"컴포넌트" 라는 표현을 문서의 나머지 부분들과 통일성을 위해 우선은 그대로 가져갔는데, 다른 곳에서도 전체적으로 "구성 요소"라고 변경하는 것은 어떨까요 ?!

Copy link
Collaborator

Choose a reason for hiding this comment

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

@bconfiden2

먼저 말씀드리면 논의를 많이할 필요가 있는 부분은 아니라 생각합니다 ^^ 컴포넌트와 구성 요소의 의미는 크게 다르지 않기도 하고요. 그래서 말씀하신 것처럼 일관성을 유지하면 좋다는데 동의하는 바 입니다.

하지만, 지극히 개인적인 해석 입니다만.. 컴포넌트가 주로 개별성 또는 독립성을 나타내는 의미로 쓰였습니다 (개별 마이크로서비스 컴포넌트, 수많은 애플리케이션 컴포넌트). 그런데 모놀리식 내부의 작은 부분(컴포넌트)들은 개별성 독립성이 아닌 개별적으로 떼어내기 애매한 것으로 설명되어야 하기에 여기만 구성 요소로 살짝 틀어본 것 입니다.

@bconfiden2 께서 결정하시면 될 것 같습니다 ! (다음 리뷰까지 받아보시고 결론 내리셔도 되고요 ^^)

content/ko/distributed-apps.md Outdated Show resolved Hide resolved
content/ko/distributed-apps.md Show resolved Hide resolved
Signed-off-by: bconfiden2 <bconfiden2@naver.com>
@bconfiden2 bconfiden2 force-pushed the 0822-distributed-apps branch from 0511e01 to c40f1e3 Compare September 7, 2022 04:30
Copy link
Collaborator

@yunkon-kim yunkon-kim left a comment

Choose a reason for hiding this comment

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

LGTM!! @bconfiden2

Copy link
Collaborator

@jihoon-seo jihoon-seo left a comment

Choose a reason for hiding this comment

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

@bconfiden2 감사합니다!
사소한 comment 하나를 달았습니다.
확인 부탁드립니다. 😊

content/ko/distributed-apps.md Outdated Show resolved Hide resolved
seokho-son and others added 2 commits September 8, 2022 17:00
Co-authored-by: Jihoon Seo <46767780+jihoon-seo@users.noreply.github.com>
Signed-off-by: Seokho Son <shsongist@gmail.com>
Co-authored-by: Jihoon Seo <46767780+jihoon-seo@users.noreply.github.com>
Signed-off-by: Seokho Son <shsongist@gmail.com>
@seokho-son
Copy link
Collaborator

@jihoon-seo 리뷰 수정 사항이 명확하여, 제가 반영하였습니다. :)

Copy link
Collaborator

@seokho-son seokho-son left a comment

Choose a reason for hiding this comment

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

LGTM !! Thanks all.

@seokho-son seokho-son merged commit f3bf0eb into cncf:dev-ko Sep 8, 2022
@bconfiden2 bconfiden2 deleted the 0822-distributed-apps branch March 21, 2023 01:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lang/ko for Korean
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants