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/07/26 21:48:28 UTC

[GitHub] [ozone] matthieuschulz opened a new pull request, #3624: topology fix

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

   ## What changes were proposed in this pull request?
   
   Minor Doc fixing of the  XML code under the Dynamic List section in hadoop-hdds/docs/content/feature/Topology.md 
   
   ## What is the link to the Apache JIRA
   
   https://gr-oss.atlassian.net/browse/OZ-45
   
   ## How was this patch tested?
   
   ran the following:  checkstyle.sh, findbugs.sh and rat.sh. 
   


-- 
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 #3624: HDDS-7052. Topology Dynamic List document issue

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

   Thanks @adoroszlai  for looking into the jira id.  cc @swagle 


-- 
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] matthieuschulz commented on pull request #3624: HDDS-7052. Topology Dynamic List document issue

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

   Hi @adoroszlai , I have made the change, thanks for the catch!


-- 
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] adoroszlai commented on pull request #3624: HDDS-7052. Topology Dynamic List document issue

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

   Thanks @matthieuschulz for the fix, @neils-dev and @xBis7 for the review.


-- 
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] adoroszlai commented on pull request #3624: HDDS-7052. Topology Dynamic List document issue

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

   @matthieuschulz please register in Apache Jira and make a comment on the issue or let me know your Jira ID in any other way.


-- 
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 #3624: HDDS-7052. Topology Dynamic List document issue

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

   Thanks @matthieuschulz for preparing this PR and its submission!  Thanks @xBis7 for your help.  Thanks @adoroszlai for the review and comments.
   Looks great.  LGTM.
     


-- 
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] xBis7 commented on pull request #3624: HDDS-7052. Topology Dynamic List document issue

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

   LGTM. Thanks @adoroszlai and @neils-dev for the review.


-- 
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 #3624: topology fix

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

   Hi @matthieuschulz .  Thanks for this PR!  Please update the Jira linked issue to HDDS-7054.  Please also assign yourself to this jira.


-- 
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] xBis7 commented on pull request #3624: HDDS-7052. Topology Dynamic List document issue

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

   @adoroszlai  Matthieu's Jira ID is `matthieuschulz`. Thanks for looking into this.


-- 
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] adoroszlai merged pull request #3624: HDDS-7052. Topology Dynamic List document issue

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


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