[
https://issues.apache.org/jira/browse/HDFS-12425?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16179297#comment-16179297
]
Xiaoyu Yao commented on HDFS-12425:
-----------------------------------
Thanks for the update, [~msingh]. The patch v4 looks good to me overall, just
few comments.
OzoneFileSystem.java
Line 273/290/292, "/" can be replaced with OZONE_URI_DELIMITER
same applies to OzoneBucket.java line 300.
OZoneInputStream.java
1. Line 70/81: suggest using URI to handle this.
bq. Can you please elaborate this comment
{code}
private final URI uri;
...
this.uri = new URI(OZONE_URI_SCHEME, bucket.getQualifiedKeyName(key), null)
{code}
I also suggest change the bucket.getQualifiedKeyName to use URI and return a
URI as well for validation and
easy integration of security in the future.
{code}
private final URI uri;
...
this.uri = bucket.getQualifiedName().resolve(key);
{code}
OzoneBucket.java
getQualifiedKeyName should build a bucket URI at the constructor time and
return it upon request
{code}
public URI getQualifiedName() {
return uri;
}
OzoneBucket() {
...
uri = new URI(OZONE_URI_SCHEME,
getBucketInfo().getVolumeName()).resolve(getBucketName());
}
{code}
bq. Yes, I wanted to use this jira to provide the basic functionality of
read/write/open and getFileInfo. I will replace these functions when the new
put key api's are in.
Sounds good to me.
> Ozone: OzoneFileSystem: OzoneFileystem read/write/create/open/getFileInfo APIs
> ------------------------------------------------------------------------------
>
> Key: HDFS-12425
> URL: https://issues.apache.org/jira/browse/HDFS-12425
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: ozone
> Affects Versions: HDFS-7240
> Reporter: Mukul Kumar Singh
> Assignee: Mukul Kumar Singh
> Labels: ozoneMerge
> Fix For: HDFS-7240
>
> Attachments: HDFS-12425-HDFS-7240.001.patch,
> HDFS-12425-HDFS-7240.002.patch, HDFS-12425-HDFS-7240.003.patch,
> HDFS-12425-HDFS-7240.004.patch
>
>
> This jira will add create/open and read/write APIs for OzoneFileSystem.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
