Skip to content

HDDS-1191. Replace Ozone Rest client with S3 client in smoketests and docs - #529

Closed
elek wants to merge 1 commit into
apache:trunkfrom
elek:HDDS-1191
Closed

HDDS-1191. Replace Ozone Rest client with S3 client in smoketests and docs#529
elek wants to merge 1 commit into
apache:trunkfrom
elek:HDDS-1191

Conversation

@elek

@elekelek commented Feb 28, 2019

Copy link
Copy Markdown
Member

As it's discussed in the the parent jira the rest support for Ozone Client protocol can be removed to use S3 Rest API instead of that.

Some of the unit tests are already disabled, so it seems to be better to remove it from the documentation (and from the smoketests).

See: https://issues.apache.org/jira/browse/HDDS-1191

@elekelek added the ozone label Feb 28, 2019
to using the REST protocol instead of RPC.
If no protocol is specified, the Ozone shell defaults to the native RPC
protocol.
1. Scheme - This should be `o3` which is the native RPC protocol to access

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

whitespace:end of line

Ozone ships with its own client library that supports both RPC (Remote
Procedure Call) and REST (Representational State Transfer). This library is
the primary user interface to ozone.
Ozone ships with its own client library that supports RPC. For generic use cases the S3

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

whitespace:end of line

Procedure Call) and REST (Representational State Transfer). This library is
the primary user interface to ozone.
Ozone ships with its own client library that supports RPC. For generic use cases the S3
compatible REST interface also can be used instead of the Ozone client.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

whitespace:end of line

@hadoop-yetus

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeComment
0reexec36Docker mode activated.
_ Prechecks _
+1@author0The patch does not contain any @author tags.
-1test4tests0The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.
_ trunk Compile Tests _
0mvndep58Maven dependency ordering for branch
+1mvninstall1088trunk passed
+1compile928trunk passed
-1mvnsite113dist in trunk failed.
+1shadedclient2921branch has no errors when building and testing our client artifacts.
+1javadoc46trunk passed
_ Patch Compile Tests _
0mvndep24Maven dependency ordering for patch
-1mvninstall17dist in the patch failed.
+1compile902the patch passed
+1javac902the patch passed
-1mvnsite28dist in the patch failed.
-1whitespace0The patch has 3 line(s) that end in whitespace. Use git apply --whitespace=fix <<patch_file>>. Refer https://git-scm.com/docs/git-apply
+1shadedclient599patch has no errors when building and testing our client artifacts.
+1javadoc43the patch passed
_ Other Tests _
+1unit26docs in the patch passed.
-1unit28dist in the patch failed.
+1asflicense38The patch does not generate ASF License warnings.
4924
SubsystemReport/Notes
DockerClient=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-529/1/artifact/out/Dockerfile
GITHUB PR#529
Optional Testsdupname asflicense mvnsite compile javac javadoc mvninstall unit shadedclient
unameLinux 07ed38007244 4.4.0-138-generic #164-Ubuntu SMP Tue Oct 2 17:16:02 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitypersonality/hadoop.sh
git revisiontrunk / 538bb48
mavenversion: Apache Maven 3.3.9
Default Java1.8.0_191
mvnsitehttps://builds.apache.org/job/hadoop-multibranch/job/PR-529/1/artifact/out/branch-mvnsite-hadoop-ozone_dist.txt
mvninstallhttps://builds.apache.org/job/hadoop-multibranch/job/PR-529/1/artifact/out/patch-mvninstall-hadoop-ozone_dist.txt
mvnsitehttps://builds.apache.org/job/hadoop-multibranch/job/PR-529/1/artifact/out/patch-mvnsite-hadoop-ozone_dist.txt
whitespacehttps://builds.apache.org/job/hadoop-multibranch/job/PR-529/1/artifact/out/whitespace-eol.txt
unithttps://builds.apache.org/job/hadoop-multibranch/job/PR-529/1/artifact/out/patch-unit-hadoop-ozone_dist.txt
Test Resultshttps://builds.apache.org/job/hadoop-multibranch/job/PR-529/1/testReport/
Max. process+thread count411 (vs. ulimit of 5500)
modulesC: hadoop-hdds/docs hadoop-ozone/dist U: .
Console outputhttps://builds.apache.org/job/hadoop-multibranch/job/PR-529/1/console
Powered byApache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@anuengineer

Copy link
Copy Markdown
Contributor

👍 I will commit this soon.

@elekelek closed this Mar 1, 2019
shanthoosh pushed a commit to shanthoosh/hadoop that referenced this pull request Oct 15, 2019
Jira: SAMZA-1722: https://issues.apache.org/jira/browse/SAMZA-1722
Author: sanil15 <sanil.jain15@gmail.com>
Reviewers: Prateek Maheshwari <pmaheshwari@apache.org>
Closesapache#529 from Sanil15/SAMZA-1722
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@elek@hadoop-yetus@anuengineer