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

Refine docs #52

Merged
merged 9 commits into from
Jan 2, 2024
Merged

Refine docs #52

merged 9 commits into from
Jan 2, 2024

Conversation

rainj-me
Copy link
Contributor

@rainj-me rainj-me commented Dec 7, 2023

What type of PR is this?

Check the PR title.

  • This PR title match the format: <type>(optional scope): <description>
  • The description of this PR title is user-oriented and clear enough for others to understand.
  • Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo

(Optional) Translate the PR title into Chinese.

(Optional) More detailed description for this PR(en: English/zh: Chinese).

en:
zh(optional):

(Optional) Which issue(s) this PR fixes:

(optional) The PR that updates user documentation:

@rainj-me rainj-me requested a review from ihciah December 7, 2023 23:09
LICENSE-MIT Outdated
@@ -0,0 +1,25 @@
Copyright (c) 2023 ihciah, rainj-me and other Monolake Contributors
Copy link
Member

Choose a reason for hiding this comment

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

Copyright belongs to a ByteDance Group not Individuals. You may use CloudWeGo Contributors or Monolake contributors.

README.md Outdated
Monoio is a subproject of [CloudWeGo](https://www.cloudwego.io).

Due to the limit resource, any help to make the monolake more mature, reporting issues or requesting features are welcome. Refer the [Contributing](./CONTRIBUTING.md) documents for the guidelines.

Choose a reason for hiding this comment

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

"Due to limited resources"

Copy link
Contributor

Choose a reason for hiding this comment

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

Getting started page has the same content as the README. We could get rid of that page

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Try to keep it for the cloudwego wiki page.

Copy link
Contributor

Choose a reason for hiding this comment

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

Suggest we add X and Y axis labels to make the graphs more meaningful to the community

docs/monolake.md Outdated
@@ -0,0 +1,109 @@
# Monolake, a proxy framework base on Rust and Iouring
Copy link
Contributor

@har23k har23k Dec 13, 2023

Choose a reason for hiding this comment

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

nit: Change "Iouring" to follow official spelling of "io_uring" through the docs

docs/monolake.md Outdated
@@ -0,0 +1,109 @@
# Monolake, a proxy framework base on Rust and Iouring

Earlier 2023, Cloudflare released a blog to introduce their Oxy, a Rust-based modern proxy framework. We have the similar requirements at Volcano Engine (a public cloud from Bytedance Inc.), so we start Monolake project, a layer 4/7 proxy framework base on Rust and Iouring.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggestion: We avoid mentioning Oxy, another project, in the first line of Monolake docs. If we do decide to mention it, we be more specific about what exactly our requirements are or what specifically in Oxy inspired us

docs/monolake.md Outdated

## Architecture of Monolake

There are 3 major categories in monolake, the runtime & transport, the tls and the http. Monolake currently supoprt Iouring and epoll runtime which are benefit from monoio(a thread-per-core rust runtime). The layer 4 proxy is implemented in monolake. The tls category currently support both rustls and native-tls, user can switch between these two solutions in case there is critical security defect in one of them. For the http category, monolake support http/1.1 and h2, we are currently working on the thrift protocol support, the grpc and h3 protocol support is planned.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggestion: It's not very obvious what "Transport" means at first glance here. If my understanding is correct, its the features that monolake proxy framework provides like dynamic handler chain update, metrics e.t.c.

We could have 4 components: Runtime, Proxy Infra, TLS and HTTP

@rainj-me rainj-me merged commit 5a770d9 into main Jan 2, 2024
6 of 8 checks passed
@har23k har23k deleted the refine-docs branch November 14, 2024 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants