[
https://issues.apache.org/jira/browse/HDDS-517?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16644430#comment-16644430
]
LiXin Ge commented on HDDS-517:
-------------------------------
Thanks [~bharatviswa] for reviewing this. fixed version patch 002 is uploaded.
> 1. This patch needs to be rebased on top of trunk.
Done.
> 2. The setting of x-amz-request-id is not required
Done.
> 3. Why do we need this check .header("Content-Length", body == null ? 0 :
> length), and also why do we need OutputStream for HEADObject?
Actually it's from [~elek] advice: {{3. I think Content-Length should be 0 in
case of missing body.}}. I'm both OK with keep the OutputStream or not, patch
002 removed the OutputStream, but that's better to hear from [~elek] if [~elek]
has some other consideration.
> 4. Few observations is content type and content length returned for us is zero
Done.
> 5. And also I think no need to add x-amz-version-id by default
Done
> Implement HeadObject REST endpoint
> ----------------------------------
>
> Key: HDDS-517
> URL: https://issues.apache.org/jira/browse/HDDS-517
> Project: Hadoop Distributed Data Store
> Issue Type: Sub-task
> Reporter: Elek, Marton
> Assignee: LiXin Ge
> Priority: Major
> Labels: newbie
> Attachments: HDDS-517.000.patch, HDDS-517.001.patch,
> HDDS-517.002.patch
>
>
> The HEAD operation retrieves metadata from an object without returning the
> object itself. This operation is useful if you are interested only in an
> object's metadata. To use HEAD, you must have READ access to the object.
> Steps:
> 1. Look up the volume
> 2. Read the key and return to the user.
> The AWS reference is here:
> https://docs.aws.amazon.com/AmazonS3/latest/API/RESTObjectHEAD.html
> We have a simple version of this call in HDDS-444 but without Range support.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
