You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by "AnonHxy (via GitHub)" <gi...@apache.org> on 2023/01/23 13:43:03 UTC

[GitHub] [bookkeeper] AnonHxy opened a new pull request, #3753: Remove the duplicated JournalQueueStats

AnonHxy opened a new pull request, #3753:
URL: https://github.com/apache/bookkeeper/pull/3753

   Descriptions of the changes in this PR:
   
   ### Motivation
   
   It seems that  L1131-1132 is duplicated of L1146-1147, so we need remove the first one.
   
   https://github.com/apache/bookkeeper/blob/7b5b6b240ca7551da266903078f2dd2ba2906b96/bookkeeper-server/src/main/java/org/apache/bookkeeper/bookie/Journal.java#L1126-L1148
   
   ### Changes
   
   Remove the duplicated JournalQueueStats
   
   


-- 
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: commits-unsubscribe@bookkeeper.apache.org

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


[GitHub] [bookkeeper] AnonHxy commented on pull request #3753: Fix the duplicated JournalQueueStats

Posted by "AnonHxy (via GitHub)" <gi...@apache.org>.
AnonHxy commented on PR #3753:
URL: https://github.com/apache/bookkeeper/pull/3753#issuecomment-1407421666

   @StevenLuMT @eolivelli  PTAL


-- 
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: commits-unsubscribe@bookkeeper.apache.org

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


[GitHub] [bookkeeper] hangc0276 commented on pull request #3753: Fix the duplicated JournalQueueStats

Posted by "hangc0276 (via GitHub)" <gi...@apache.org>.
hangc0276 commented on PR #3753:
URL: https://github.com/apache/bookkeeper/pull/3753#issuecomment-1409895171

   @horizonzy Please help take a look at this Pr, 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: commits-unsubscribe@bookkeeper.apache.org

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


[GitHub] [bookkeeper] StevenLuMT merged pull request #3753: Fix the duplicated JournalQueueStats

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


-- 
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: commits-unsubscribe@bookkeeper.apache.org

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