You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by "jianghuazhu (via GitHub)" <gi...@apache.org> on 2023/12/11 10:43:11 UTC

[PR] HDDS-9890. Fix wrong characters in OmMetadataManagerImpl [ozone]

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

   ## What changes were proposed in this pull request?
   In OmMetadataManagerImpl, there are some wrong symbols, the purpose of this PR is to fix them.
   Details: HDDS-9890
   
   ## What is the link to the Apache JIRA
   https://issues.apache.org/jira/browse/HDDS-9890
   
   ## How was this patch tested?
   This is a change related to documentation and does not require too much testing. Just keep the unit tests passing.
   


-- 
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-9890. Fix wrong characters in OmMetadataManagerImpl [ozone]

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

   Thanks @kerneltime .


-- 
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-9890. Fix wrong characters in OmMetadataManagerImpl [ozone]

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

   @jianghuazhu can you please rebase your fork branch.


-- 
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-9890. Fix comment typo in OmMetadataManagerImpl [ozone]

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

   Thanks @adoroszlai .


-- 
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-9890. Fix wrong characters in OmMetadataManagerImpl [ozone]

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

   Thanks @adoroszlai  for the comment and review.
   The problem described by this jira must be fixed, so I created a separate jira and tried to solve it.
   Now I need to wait for more similar questions, right?


-- 
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-9890. Fix comment typo in OmMetadataManagerImpl [ozone]

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

   Thanks @hemantk-12 for the comment and review.
   I'll update soon.


-- 
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-9890. Fix comment typo in OmMetadataManagerImpl [ozone]

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

   Thanks @jianghuazhu for the patch.  Since it's a trivial comment change, I think we can merge it without CI.


-- 
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-9890. Fix wrong characters in OmMetadataManagerImpl [ozone]

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

   Thanks @jianghuazhu for spotting the typo.  Please consider https://github.com/apache/ozone/pull/5557#issuecomment-1799061921


-- 
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-9890. Fix wrong characters in OmMetadataManagerImpl [ozone]

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

   https://github.com/jianghuazhu/ozone/actions/runs/7164919925
   Here is the ci/cd of the fork branch, it looks normal.
   Can you help review this PR, @adoroszlai  @hemantk-12 .
   Thanks.
   


-- 
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-9890. Fix comment typo in OmMetadataManagerImpl [ozone]

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


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