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

docs(faq): fix cluster link #557

Merged
merged 5 commits into from
Mar 14, 2017
Merged

docs(faq): fix cluster link #557

merged 5 commits into from
Mar 14, 2017

Conversation

marswong
Copy link
Contributor

Checklist
Affected core subsystem(s)
Description of change

@mention-bot
Copy link

@OneNewLife, thanks for your PR! By analyzing the history of the files in this pull request, we identified @shaoshuai0102 and @atian25 to be potential reviewers.

@codecov
Copy link

codecov bot commented Mar 12, 2017

Codecov Report

Merging #557 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #557   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          26     26           
  Lines         632    632           
=====================================
  Hits          632    632

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0d37e42...82e93d7. Read the comment docs.

@@ -15,7 +15,7 @@ title: 常见问题

1. PM2 模块本身复杂度很高,出了问题很难排查。我们认为框架使用的工具复杂度不应该过高,而 PM2 自身的复杂度超越了大部分应用本身。
2. 没法做非常深的优化。
3. 切实的需求问题,一个进程里跑 leader,其他进程代理到 leader 这种模式([多进程模型](./advanced/cluster.md)),在企业级开发中对于减少远端连接,降低数据通信压力等都是切实的需求。特别当应用规模大到一定程度,这就会是刚需。egg 本身起源于蚂蚁金服和阿里,我们对标的起点就是大规模企业应用的构建,所以要非常全面。这些特性通过 PM2 很难做到。
3. 切实的需求问题,一个进程里跑 leader,其他进程代理到 leader 这种模式([多进程模型](./advanced/cluster-client.md)),在企业级开发中对于减少远端连接,降低数据通信压力等都是切实的需求。特别当应用规模大到一定程度,这就会是刚需。egg 本身起源于蚂蚁金服和阿里,我们对标的起点就是大规模企业应用的构建,所以要非常全面。这些特性通过 PM2 很难做到。
Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

应该是 ./core/cluster-and-ipc.md

@atian25
Copy link
Member

atian25 commented Mar 13, 2017

en/faq.md is also need to fix

@atian25
Copy link
Member

atian25 commented Mar 14, 2017

ping @OneNewLife

`多进程模型` 的链接更改为 `./core/cluster-and-ipc.md`
@marswong
Copy link
Contributor Author

@atian25 sorry,差点搞忘了,btw,英文版的 cluster-and-ipc.md 貌似还没写。

@atian25
Copy link
Member

atian25 commented Mar 14, 2017

反正两个都没有... 链了再说

@marswong
Copy link
Contributor Author

got it

change the `cluster` link address to `./core/cluster-and-ipc.md`
@@ -13,7 +13,7 @@ When you got some trouble, and want to find out what is the final config using a

1. PM2 itself is too complex to issue problems if any.
2. Deep optimization could be difficlut to achieve if choosing PM2.
3. Pattern like one leader process communicating with remote services, along with serveral follower processes delegating request to it (([Cluster](./advanced/cluster.md)), is a rigid demand for reducing connections and data exchange load, espeically when facing applications in very large scale. egg originates from Ant Financial Group and Alibaba Group, we start with applications in that scale at first, so we take these goals into consideration. All of these goals above could be hard to achieve with PM2.
3. Pattern like one leader process communicating with remote services, along with serveral follower processes delegating request to it (([Cluster](./core/cluster-and-ipc.md)), is a rigid demand for reducing connections and data exchange load, espeically when facing applications in very large scale. egg originates from Ant Financial Group and Alibaba Group, we start with applications in that scale at first, so we take these goals into consideration. All of these goals above could be hard to achieve with PM2.
Copy link
Member

Choose a reason for hiding this comment

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

(([))

Copy link
Member

@atian25 atian25 left a comment

Choose a reason for hiding this comment

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

LGTM

@atian25 atian25 merged commit 497b9a9 into eggjs:master Mar 14, 2017
@atian25 atian25 changed the title docs: fix faq typo docs(faq): fix cluster link Mar 14, 2017
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.

5 participants