Skip to content
This repository has been archived by the owner on Oct 22, 2021. It is now read-only.

docs: Add RFC example #12

Merged
merged 4 commits into from
Jul 17, 2021
Merged

docs: Add RFC example #12

merged 4 commits into from
Jul 17, 2021

Conversation

bokket
Copy link
Member

@bokket bokket commented Jul 17, 2021

Signed-off-by: bokket 3100563328@qq.com

Signed-off-by: bokket <3100563328@qq.com>
@bokket bokket changed the title RFC-11:Add Tcp Pair RFC-12:Add Tcp Pair Jul 17, 2021
Signed-off-by: bokket <3100563328@qq.com>
@Xuanwo
Copy link
Contributor

Xuanwo commented Jul 17, 2021

I think this RFC needs to be divided into two parts.

  1. A RFC to go-endpoint to add TCP support
  2. A RFC to hdfs to describe how will hdfs use endpoint (Maybe it's too clear that don't need a RFC)

@bokket
Copy link
Member Author

bokket commented Jul 17, 2021

I think this RFC needs to be divided into two parts.

  1. A RFC to go-endpoint to add TCP support
  2. A RFC to hdfs to describe how will hdfs use endpoint (Maybe it's too clear that don't need a RFC)

It can be transferred to beyondstorage/go-endpoint/pull?
If not , may be merge 0-example and I go beyondstorage/go-endpoint again

@Xuanwo
Copy link
Contributor

Xuanwo commented Jul 17, 2021

It can be transferred to beyondstorage/go-endpoint/pull?

No, we cannot.

If not , may be merge 0-example and I go beyondstorage/go-endpoint again

LGTM, let's remove the rfc-11 and just keep the example.

@Xuanwo Xuanwo changed the title RFC-12:Add Tcp Pair docs: Add RFC example Jul 17, 2021
Signed-off-by: bokket <3100563328@qq.com>
docs/rfcs/0-example.md Outdated Show resolved Hide resolved
docs/rfcs/0-example.md Outdated Show resolved Hide resolved
docs/rfcs/0-example.md Outdated Show resolved Hide resolved
docs/rfcs/0-example.md Outdated Show resolved Hide resolved
Signed-off-by: bokket <3100563328@qq.com>
@Xuanwo Xuanwo merged commit 109c679 into beyondstorage:master Jul 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants