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

HDDS-2793. concept/Datanodes.md translation #520

Merged
merged 4 commits into from Apr 4, 2020

Conversation

iamabug
Copy link
Contributor

@iamabug iamabug commented Feb 3, 2020

What changes were proposed in this pull request?

Datanodes.zh.md added

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-2793

How was this patch tested?

hugo server -D

This extra indirection helps tremendously with scaling Ozone. SCM has far
less block data to process and the name node is a different service are
critical to scaling Ozone.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

the name node is a different service are critical to scaling Ozone

I think it does not make sense in grammar.

Copy link
Contributor

Choose a reason for hiding this comment

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

Good catch. Can you change EN document from "name node is a" to " namespace service (Ozone Manager) as a "? That will make sense.

@adoroszlai adoroszlai added the documentation Improvements or additions to documentation label Feb 3, 2020
date: "2017-09-14"
weight: 4
summary: TODO translated summary
summary: Ozone 支持 Amazon 的简易存储服务(S3)协议,你可以原封不动地在 Ozone 上使用基于 S3 客户端和 S3 SDK 的应用。
Copy link
Contributor

Choose a reason for hiding this comment

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

Amazon 的简易存储服务(S3)=> Amazon S3

![FunctionalOzone](ContainerMetadata.png)

Ozone 的存储容器是一个自包含的超级块,容器中包含一系列的 Ozone 块,以及存储实际数据的磁盘文件,这是默认的存储容器格式。对于 Ozone 来说,容器只是提供了一个协议说明,实际的存储方式并不重要,换句话说,扩展或引入新的容器设计也是很容易的。因此,上述格式应当被看作是 Ozone 存储容器的参考实现。
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
Contributor

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.

扩展或引入新的容器设计也是很容易的=》我们可以很容易扩展或引入新的容器实现格式。

### 容器的位置发现

SCM 如何获得容器的位置?方法和 HDFS 中的方法十分相似。数据节点会定期发送类似于块报告的容器报告,但比块报告的内容简洁的多,比如,对于一个存储容量为 196 TB 的单数据节点组成的集群,Ozone 大概会拥有四万个容器,相比于 HDFS 的一百五十万个块,块报告数量缩减为四十分之一。
Copy link
Contributor

Choose a reason for hiding this comment

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

方法和 HDFS 中的方法十分相似=》这一点和现有的HDFS 十分相似

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
Contributor

Choose a reason for hiding this comment

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

196 TB 的单数据节点组成的集群-》196 TB 的集群

Copy link
Contributor

@xiaoyuyao xiaoyuyao left a comment

Choose a reason for hiding this comment

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

LGTM, a few comments added inline

@iamabug
Copy link
Contributor Author

iamabug commented Mar 7, 2020

retest

Copy link
Contributor

@xiaoyuyao xiaoyuyao left a comment

Choose a reason for hiding this comment

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

LGTM, +1.

@xiaoyuyao
Copy link
Contributor

I check the acceptance test logs which points to the om-ha tests with permission issue. This does not seem to relate to the document change here. cc: @hanishakoneru and @bharatviswa504

2019-11-16T11:59:00.0117168Z ==============================================================================
2019-11-16T11:59:00.0120741Z ozone-om-ha-testOMHA :: Smoketest ozone cluster startup
2019-11-16T11:59:00.0121273Z ==============================================================================
2019-11-16T11:59:12.8359463Z Stop Leader OM and Verify Failover | FAIL |
2019-11-16T11:59:12.8360383Z OSError: [Errno 13] Permission denied: '/opt/hadoop/tmpsCDIs5'
2019-11-16T11:59:12.8360708Z ------------------------------------------------------------------------------
2019-11-16T12:01:07.8768425Z Restart OM and Verify Ratis Logs | PASS |
2019-11-16T12:01:07.8772261Z ------------------------------------------------------------------------------
2019-11-16T12:01:24.9714368Z Test Multiple Failovers | FAIL |
2019-11-16T12:01:24.9715403Z OSError: [Errno 13] Permission denied: '/opt/hadoop/tmpLn9PKJ'
2019-11-16T12:01:24.9715697Z ------------------------------------------------------------------------------
2019-11-16T12:01:24.9742716Z ozone-om-ha-testOMHA :: Smoketest ozone cluster startup | FAIL |
2019-11-16T12:01:24.9749529Z 3 critical tests, 1 passed, 2 failed
2019-11-16T12:01:24.9749662Z 3 tests total, 1 passed, 2 failed

@hanishakoneru
Copy link
Contributor

@xiaoyuyao, OM-HA tests are disabled and should not be running.
In the following CI run, the acceptance test suite does not run OM-HA tests:
https://github.com/apache/hadoop-ozone/pull/520/checks?check_run_id=485882139

Is there any other run you are referring to?

@xiaoyuyao
Copy link
Contributor

If you download the acceptance test results from the link and
https://github.com/apache/hadoop-ozone/suites/485213917/artifacts/2531797

search for this file 7_Run .hadoop-ozonedev-supportchecksacceptance.sh.txt
You can find the error above.

@xiaoyuyao
Copy link
Contributor

==============================================================================
2019-11-16T11:59:00.0120726Z ozone-om-ha-testOMHA :: Smoketest ozone cluster startup
2019-11-16T11:59:00.0121265Z ==============================================================================
2019-11-16T11:59:12.8359426Z Stop Leader OM and Verify Failover | FAIL |
2019-11-16T11:59:12.8360374Z OSError: [Errno 13] Permission denied: '/opt/hadoop/tmpsCDIs5'
2019-11-16T11:59:12.8360702Z ------------------------------------------------------------------------------
2019-11-16T12:01:07.8768389Z Restart OM and Verify Ratis Logs | PASS |
2019-11-16T12:01:07.8772219Z ------------------------------------------------------------------------------
2019-11-16T12:01:24.9714328Z Test Multiple Failovers | FAIL |
2019-11-16T12:01:24.9715394Z OSError: [Errno 13] Permission denied: '/opt/hadoop/tmpLn9PKJ'
2019-11-16T12:01:24.9715691Z

@hanishakoneru
Copy link
Contributor

This error is from an old run. Confirmed with @xiaoyuyao that the new runs do not have the OM HA tests.

@adoroszlai
Copy link
Contributor

Thanks @iamabug for the translation and @xiaoyuyao for reviewing it.

@adoroszlai adoroszlai merged commit 0fbacbc into apache:master Apr 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
5 participants