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

add: 数据面MOSN概念介绍 #203

Merged
merged 5 commits into from
Apr 22, 2020
Merged

Conversation

wangfakang
Copy link
Contributor

Issue-ID: #67
Signed-off-by: Fakang Wang fakangwang@gmail.com

提交 PR 后请先自查以下内容:

  • PR 的标题格式为:在空白文档中新增内容 add: 文档标题,修改已有内容的文档 update:文档标题
  • 在该 PR 已指明解决的 issue 链接:
  • 已阅读并保证所提交 PR 符合写作规范
  • 文章内容格式请参考 PR add: practice/jwt-authorization #186
  • 保留下文中的声明。

声明

  • 我认可本书的版权信息:本书著作权归属于 ServiceMesher 社区,我将保留对所修改内容的署名权。

  • 我已阅读参与贡献说明,并加入了 istio-handbook-wg GitHub team。

@netlify
Copy link

netlify bot commented Apr 19, 2020

Deploy preview for istio-handbook ready!

Built with commit 39e9492

https://deploy-preview-203--istio-handbook.netlify.app

@mesher-bot mesher-bot requested review from 3ks and malphi April 19, 2020 09:10
Copy link
Contributor

@ikingye ikingye left a comment

Choose a reason for hiding this comment

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

建议再检查一下文章格式,还有挺多问题。

concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
Copy link
Contributor

@ikingye ikingye left a comment

Choose a reason for hiding this comment

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

还有一些问题需要修改。

concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
MOSN 整体框架采用分而治之的架构思想,其中每一层通过工厂设计模式向外暴露其接口,方便用户灵活的注册自身的需求,采用协程池的方式使得用户以同步的编码风格达到异步的异曲同工之妙。通过协程的类型 MOSN 中分为 read 和 proxy worker 两大类协程,read 协程主要是处理网络的读取及协议解析,proxy worker 协程用来完成读取后数据的加工、路由、转发等,其架构如下图所示:

![concepts-mosn-goroutine](../images/concepts-mosn-goroutine.png)
Copy link
Contributor

Choose a reason for hiding this comment

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

图不清晰,不过这个可以后续改。

concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
concepts/mosn.md Outdated Show resolved Hide resolved
@ikingye
Copy link
Contributor

ikingye commented Apr 21, 2020

/LGTM

Copy link
Member

@rootsongjc rootsongjc left a comment

Choose a reason for hiding this comment

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

/LGTM

@mesher-bot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: rootsongjc, wangfakang

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [rootsongjc,wangfakang]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mesher-bot mesher-bot removed the lgtm label Apr 22, 2020
@mesher-bot
Copy link

New changes are detected. LGTM label has been removed.

@mergify mergify bot merged commit eb1922f into servicemesher:master Apr 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants