Skip to content

HDDS-1479. Update S3.md documentation#791

Closed
vivekratnavel wants to merge 1 commit into
apache:trunkfrom
vivekratnavel:HDDS-1479
Closed

HDDS-1479. Update S3.md documentation#791
vivekratnavel wants to merge 1 commit into
apache:trunkfrom
vivekratnavel:HDDS-1479

Conversation

@vivekratnavel
Copy link
Copy Markdown
Contributor

HDDS-791 implemented range get operation.

But, S3.md documentation still has the below line:

GET Object | implemented | Range headers are not supported

This PR deletes the "Range headers are not supported" from the doc.

@vivekratnavel
Copy link
Copy Markdown
Contributor Author

@bharatviswa504 Please review when you find time

@hadoop-yetus
Copy link
Copy Markdown

🎊 +1 overall

Vote Subsystem Runtime Comment
0 reexec 23 Docker mode activated.
_ Prechecks _
+1 @author 0 The patch does not contain any @author tags.
_ trunk Compile Tests _
+1 mvninstall 1017 trunk passed
+1 mvnsite 20 trunk passed
+1 shadedclient 1686 branch has no errors when building and testing our client artifacts.
_ Patch Compile Tests _
+1 mvninstall 21 the patch passed
+1 mvnsite 16 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 706 patch has no errors when building and testing our client artifacts.
_ Other Tests _
+1 asflicense 26 The patch does not generate ASF License warnings.
2598
Subsystem Report/Notes
Docker Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-791/1/artifact/out/Dockerfile
GITHUB PR #791
Optional Tests dupname asflicense mvnsite
uname Linux f43850e657c2 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 tool maven
Personality personality/hadoop.sh
git revision trunk / f682a17
maven version: Apache Maven 3.3.9
Max. process+thread count 411 (vs. ulimit of 5500)
modules C: hadoop-hdds/docs U: hadoop-hdds/docs
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-791/1/console
Powered by Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

@vivekratnavel
Copy link
Copy Markdown
Contributor Author

/label ozone

@elek elek added the ozone label May 2, 2019
Copy link
Copy Markdown
Member

@elek elek left a comment

Choose a reason for hiding this comment

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

+1 Thanks the fix.

You are right thanks to @bharatviswa504 we have range support since https://issues.apache.org/jira/browse/HDDS-791

@elek elek closed this in 3cb1d09 May 2, 2019
shanthoosh added a commit to shanthoosh/hadoop that referenced this pull request Oct 15, 2019
This PR consists of the following changes:
* Switching all the API methods from using byte[] array as key type to string.
* Fixed `CoordinatorMetadataStore`, `ZkMetadataStore` tests due to the type change of key.

Shortly in a followup PR,  namespace unification for different metadata stored in standalone and YARN model will be done.

Author: Shanthoosh Venkataraman <spvenkat@usc.edu>

Reviewers: Prateek <prateekm@linkedin.com>

Closes apache#791 from shanthoosh/metadata_store_api_cleanup
Sign up for free to 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