You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2020/06/27 10:05:56 UTC
[GitHub] [hadoop-ozone] captainzmc opened a new pull request #1135: HDDS-3883. Update doc and remove useless conf of o3fs.
captainzmc opened a new pull request #1135:
URL: https://github.com/apache/hadoop-ozone/pull/1135
## What changes were proposed in this pull request?
The latest usage of O3FS has been updated in HDDS-3704. Based on HDDS-3704, I verified that there were other useless configurations not removed. So delete them here.
## What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-3883
Please replace this section with the link to the Apache JIRA)
## How was this patch tested?
update doc no need test.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org
[GitHub] [hadoop-ozone] adoroszlai commented on pull request #1135: HDDS-3883. Update doc and remove useless conf of o3fs.
Posted by GitBox <gi...@apache.org>.
adoroszlai commented on pull request #1135:
URL: https://github.com/apache/hadoop-ozone/pull/1135#issuecomment-650770114
@captainzmc acceptance test failed with:
```
UnsupportedFileSystemException: fs.AbstractFileSystem.o3fs.impl=null: No AbstractFileSystem configured for scheme: o3fs
```
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org
[GitHub] [hadoop-ozone] codecov-commenter commented on pull request #1135: HDDS-3883. Update doc and remove useless conf of o3fs.
Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on pull request #1135:
URL: https://github.com/apache/hadoop-ozone/pull/1135#issuecomment-650549390
# [Codecov](https://codecov.io/gh/apache/hadoop-ozone/pull/1135?src=pr&el=h1) Report
> Merging [#1135](https://codecov.io/gh/apache/hadoop-ozone/pull/1135?src=pr&el=desc) into [master](https://codecov.io/gh/apache/hadoop-ozone/commit/f8fcc4760b227308c4a72f06f2e35864e69a1f22&el=desc) will **increase** coverage by `2.84%`.
> The diff coverage is `74.46%`.
[![Impacted file tree graph](https://codecov.io/gh/apache/hadoop-ozone/pull/1135/graphs/tree.svg?width=650&height=150&src=pr&token=5YeeptJMby)](https://codecov.io/gh/apache/hadoop-ozone/pull/1135?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #1135 +/- ##
============================================
+ Coverage 70.56% 73.40% +2.84%
- Complexity 9427 9964 +537
============================================
Files 965 969 +4
Lines 49063 49413 +350
Branches 4803 4839 +36
============================================
+ Hits 34620 36271 +1651
+ Misses 12137 10836 -1301
Partials 2306 2306
```
| [Impacted Files](https://codecov.io/gh/apache/hadoop-ozone/pull/1135?src=pr&el=tree) | Coverage Δ | Complexity Δ | |
|---|---|---|---|
| [.../java/org/apache/hadoop/ozone/OzoneConfigKeys.java](https://codecov.io/gh/apache/hadoop-ozone/pull/1135/diff?src=pr&el=tree#diff-aGFkb29wLWhkZHMvY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9oYWRvb3Avb3pvbmUvT3pvbmVDb25maWdLZXlzLmphdmE=) | `100.00% <ø> (ø)` | `1.00 <0.00> (ø)` | |
| [...main/java/org/apache/hadoop/ozone/OzoneConsts.java](https://codecov.io/gh/apache/hadoop-ozone/pull/1135/diff?src=pr&el=tree#diff-aGFkb29wLWhkZHMvY29tbW9uL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9oYWRvb3Avb3pvbmUvT3pvbmVDb25zdHMuamF2YQ==) | `84.21% <ø> (ø)` | `1.00 <0.00> (ø)` | |
| [...nt/algorithms/ContainerPlacementPolicyFactory.java](https://codecov.io/gh/apache/hadoop-ozone/pull/1135/diff?src=pr&el=tree#diff-aGFkb29wLWhkZHMvc2VydmVyLXNjbS9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvaGFkb29wL2hkZHMvc2NtL2NvbnRhaW5lci9wbGFjZW1lbnQvYWxnb3JpdGhtcy9Db250YWluZXJQbGFjZW1lbnRQb2xpY3lGYWN0b3J5LmphdmE=) | `83.33% <ø> (ø)` | `2.00 <0.00> (ø)` | |
| [...src/main/java/org/apache/hadoop/ozone/OmUtils.java](https://codecov.io/gh/apache/hadoop-ozone/pull/1135/diff?src=pr&el=tree#diff-aGFkb29wLW96b25lL2NvbW1vbi9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvaGFkb29wL296b25lL09tVXRpbHMuamF2YQ==) | `80.16% <ø> (+4.95%)` | `41.00 <0.00> (+3.00)` | |
| [...hadoop/ozone/om/protocol/OzoneManagerProtocol.java](https://codecov.io/gh/apache/hadoop-ozone/pull/1135/diff?src=pr&el=tree#diff-aGFkb29wLW96b25lL2NvbW1vbi9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvaGFkb29wL296b25lL29tL3Byb3RvY29sL096b25lTWFuYWdlclByb3RvY29sLmphdmE=) | `0.00% <ø> (ø)` | `0.00 <0.00> (ø)` | |
| [...g/apache/hadoop/ozone/om/codec/OMDBDefinition.java](https://codecov.io/gh/apache/hadoop-ozone/pull/1135/diff?src=pr&el=tree#diff-aGFkb29wLW96b25lL296b25lLW1hbmFnZXIvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2hhZG9vcC9vem9uZS9vbS9jb2RlYy9PTURCRGVmaW5pdGlvbi5qYXZh) | `0.00% <0.00%> (ø)` | `0.00 <0.00> (?)` | |
| [...one/om/response/bucket/OMBucketCreateResponse.java](https://codecov.io/gh/apache/hadoop-ozone/pull/1135/diff?src=pr&el=tree#diff-aGFkb29wLW96b25lL296b25lLW1hbmFnZXIvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2hhZG9vcC9vem9uZS9vbS9yZXNwb25zZS9idWNrZXQvT01CdWNrZXRDcmVhdGVSZXNwb25zZS5qYXZh) | `100.00% <ø> (ø)` | `4.00 <0.00> (ø)` | |
| [...one/om/response/bucket/OMBucketDeleteResponse.java](https://codecov.io/gh/apache/hadoop-ozone/pull/1135/diff?src=pr&el=tree#diff-aGFkb29wLW96b25lL296b25lLW1hbmFnZXIvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2hhZG9vcC9vem9uZS9vbS9yZXNwb25zZS9idWNrZXQvT01CdWNrZXREZWxldGVSZXNwb25zZS5qYXZh) | `100.00% <ø> (ø)` | `5.00 <0.00> (ø)` | |
| [...m/response/bucket/OMBucketSetPropertyResponse.java](https://codecov.io/gh/apache/hadoop-ozone/pull/1135/diff?src=pr&el=tree#diff-aGFkb29wLW96b25lL296b25lLW1hbmFnZXIvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2hhZG9vcC9vem9uZS9vbS9yZXNwb25zZS9idWNrZXQvT01CdWNrZXRTZXRQcm9wZXJ0eVJlc3BvbnNlLmphdmE=) | `100.00% <ø> (ø)` | `3.00 <0.00> (ø)` | |
| [...ne/om/response/bucket/acl/OMBucketAclResponse.java](https://codecov.io/gh/apache/hadoop-ozone/pull/1135/diff?src=pr&el=tree#diff-aGFkb29wLW96b25lL296b25lLW1hbmFnZXIvc3JjL21haW4vamF2YS9vcmcvYXBhY2hlL2hhZG9vcC9vem9uZS9vbS9yZXNwb25zZS9idWNrZXQvYWNsL09NQnVja2V0QWNsUmVzcG9uc2UuamF2YQ==) | `69.23% <ø> (ø)` | `3.00 <0.00> (ø)` | |
| ... and [293 more](https://codecov.io/gh/apache/hadoop-ozone/pull/1135/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/hadoop-ozone/pull/1135?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/hadoop-ozone/pull/1135?src=pr&el=footer). Last update [0f2a118...1be6ba2](https://codecov.io/gh/apache/hadoop-ozone/pull/1135?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org
[GitHub] [hadoop-ozone] captainzmc commented on pull request #1135: HDDS-3883. Update doc and remove useless conf of o3fs.
Posted by GitBox <gi...@apache.org>.
captainzmc commented on pull request #1135:
URL: https://github.com/apache/hadoop-ozone/pull/1135#issuecomment-650776524
> @captainzmc acceptance test failed with:
>
> ```
> UnsupportedFileSystemException: fs.AbstractFileSystem.o3fs.impl=null: No AbstractFileSystem configured for scheme: o3fs
> ```
Thanks for @adoroszlai 's hint. Before, I just manually tested the shell of hadoop2 and hadoop3. Maybe there's something I missed. I'm going to close this PR first.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org
[GitHub] [hadoop-ozone] captainzmc closed pull request #1135: HDDS-3883. Update doc and remove useless conf of o3fs.
Posted by GitBox <gi...@apache.org>.
captainzmc closed pull request #1135:
URL: https://github.com/apache/hadoop-ozone/pull/1135
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org