You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2022/05/26 09:19:31 UTC

[GitHub] [ozone] JacksonYao287 opened a new pull request, #3459: HDDS-6804. use common backgroundScmService for backgroundPipelieScruber

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

   ## What changes were proposed in this pull request?
   
   since HDDS-6771 is merged , backgroundPipelieScruber should use the common backgroundScmService to simplify the code
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/HDDS-4656
   
   ## How was this patch tested?
   exit UT
   


-- 
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


[GitHub] [ozone] neils-dev commented on pull request #3459: HDDS-6804. use common backgroundScmService for backgroundPipelieScruber

Posted by GitBox <gi...@apache.org>.
neils-dev commented on PR #3459:
URL: https://github.com/apache/ozone/pull/3459#issuecomment-1138860160

   @JacksonYao287, thanks!  The link to the Apache Jira in this PR should be fixed to point to the HDDS-6804 issue **_instead_** of the current HDDS-4656.


-- 
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


[GitHub] [ozone] neils-dev commented on pull request #3459: HDDS-6804. use common backgroundScmService for backgroundPipelieScruber

Posted by GitBox <gi...@apache.org>.
neils-dev commented on PR #3459:
URL: https://github.com/apache/ozone/pull/3459#issuecomment-1141287652

   > @neils-dev , sorry, Neil, i do not get your point here. this jira has nothing to do with [HDDS-4656](https://issues.apache.org/jira/browse/HDDS-4656). do i miss something?
   
   @JacksonYao287 - thanks.  Looks like @adoroszlai corrected the mistake.  You are not missing anything with it.  It was simply that the "What is the link to the Apache Jira" section of the PR was pointing to the 4656 jira.  It was corrected and now is pointing to HDDS-6894 as intended.


-- 
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


[GitHub] [ozone] JacksonYao287 commented on pull request #3459: HDDS-6804. use common backgroundScmService for backgroundPipelieScruber

Posted by GitBox <gi...@apache.org>.
JacksonYao287 commented on PR #3459:
URL: https://github.com/apache/ozone/pull/3459#issuecomment-1142086837

   thanks @sodonnel @neils-dev and @adoroszlai for the review! i have committed the patch


-- 
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


[GitHub] [ozone] JacksonYao287 commented on pull request #3459: HDDS-6804. use common backgroundScmService for backgroundPipelieScruber

Posted by GitBox <gi...@apache.org>.
JacksonYao287 commented on PR #3459:
URL: https://github.com/apache/ozone/pull/3459#issuecomment-1139318544

   @neils-dev , sorry, Neil, i do not get your point here.  this jira has nothing to do with [HDDS-4656](https://issues.apache.org/jira/browse/HDDS-4656). do i miss something?


-- 
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


[GitHub] [ozone] JacksonYao287 merged pull request #3459: HDDS-6804. use common backgroundScmService for backgroundPipelieScruber

Posted by GitBox <gi...@apache.org>.
JacksonYao287 merged PR #3459:
URL: https://github.com/apache/ozone/pull/3459


-- 
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