You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by "ivanzlenko (via GitHub)" <gi...@apache.org> on 2025/08/10 13:14:56 UTC

[PR] HDDS-13132. Convert redundant fields to local var in production code [ozone]

ivanzlenko opened a new pull request, #8921:
URL: https://github.com/apache/ozone/pull/8921

   ## What changes were proposed in this pull request?
   Cleared production code base from fields which could be converted to local variables.
   
   Please describe your PR in detail:
   The production codebase has been updated to comply with the PMD SingularField rule. Instances where compliance was not feasible have been suppressed. In more complex cases, TODO comments were added with corresponding Jira tickets to refactor those sections when converting the field to a local variable is not straightforward.
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/HDDS-13132
   
   ## How was this patch tested?
   
   Patch is tested using existing automation. New tests are not required. 
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org


Re: [PR] HDDS-13132. Convert redundant fields to local var in production code [ozone]

Posted by "myskov (via GitHub)" <gi...@apache.org>.
myskov merged PR #8921:
URL: https://github.com/apache/ozone/pull/8921


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org


Re: [PR] HDDS-13132. Convert redundant fields to local var in production code [ozone]

Posted by "ivanzlenko (via GitHub)" <gi...@apache.org>.
ivanzlenko commented on PR #8921:
URL: https://github.com/apache/ozone/pull/8921#issuecomment-3182028263

   @adoroszlai can you have a look? 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org