[ 
https://issues.apache.org/jira/browse/HDDS-805?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16704374#comment-16704374
 ] 

Yiqun Lin commented on HDDS-805:
--------------------------------

Some minor comments from me:

* Can we remove class {{OzoneBlockTokenSelector}}? As I see we can get token 
from XceiverClient.
* Can we separate the inner class {{ChunkOutputStreamEntry}} from 
{{ChunkGroupOutputStream}}? Since we have added builder constructor functions, 
this class become large enough.

BTW, which JIRA tracking the work of ozone block token validation? I didn't 
find this logic and also maybe I missed something.

 

> Block token: Client api changes for block token
> -----------------------------------------------
>
>                 Key: HDDS-805
>                 URL: https://issues.apache.org/jira/browse/HDDS-805
>             Project: Hadoop Distributed Data Store
>          Issue Type: Sub-task
>          Components: Security
>            Reporter: Ajay Kumar
>            Assignee: Ajay Kumar
>            Priority: Major
>         Attachments: HDDS-805-HDDS-4.00.patch
>
>




--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to