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

Attila Doroszlai edited comment on HDDS-10317 at 11/10/25 11:40 AM:
--------------------------------------------------------------------

Thanks [~russole].  This requires HDDS-10388 to be implemented first, to make 
{{OmBucketInfo}}'s parent classes immutable.  It may also need to be split into 
several tasks (e.g. one per some set of OM responses) to make the change 
manageable.

I suggest looking for some simpler tasks instead.


was (Author: adoroszlai):
Thanks [~russole].  This requires HDDS-10388 to be implemented first, to make 
{{OmBucketInfo}}'s parent classes immutable.  It may also need to be split into 
several tasks (e.g. one per some set of OM responses) to make the change 
manageable.

> Make OmBucketInfo immutable
> ---------------------------
>
>                 Key: HDDS-10317
>                 URL: https://issues.apache.org/jira/browse/HDDS-10317
>             Project: Apache Ozone
>          Issue Type: Sub-task
>          Components: Ozone Manager
>            Reporter: Attila Doroszlai
>            Priority: Major
>         Attachments: HDDS-10317.txt
>
>
> {code}
> M V EI: 
> org.apache.hadoop.ozone.om.response.bucket.OMBucketCreateResponse.getOmBucketInfo()
>  may expose internal representation by returning 
> OMBucketCreateResponse.omBucketInfo  At OMBucketCreateResponse.java:[line 91]
> M V EI2: new 
> org.apache.hadoop.ozone.om.response.bucket.OMBucketCreateResponse(OzoneManagerProtocolProtos$OMResponse,
>  OmBucketInfo) may expose internal representation by storing an externally 
> mutable object into OMBucketCreateResponse.omBucketInfo  At 
> OMBucketCreateResponse.java:[line 57]
> {code}
> and similar warning for other response classes



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to