You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2020/03/12 09:35:36 UTC
[GitHub] [bookkeeper] eolivelli opened a new pull request #2286: Do not log
WARNING about PlacementPolicy for tests and simple setups
eolivelli opened a new pull request #2286: Do not log WARNING about PlacementPolicy for tests and simple setups
URL: https://github.com/apache/bookkeeper/pull/2286
Descriptions of the changes in this PR:
Do not log 'New ensemble: {} is not adhering to Placement Policy. quarantinedBookies' in case of ensembleSize = 1
### Motivation
In tests cases and in simple demos it is very annoying to see that log line at "WARNING" level.
If we are requiring only 1 bookie it is not a big deal to WARN that the placement policy is not "adhering"
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [bookkeeper] eolivelli commented on issue #2286: Do not log WARNING
about PlacementPolicy for tests and simple setups
Posted by GitBox <gi...@apache.org>.
eolivelli commented on issue #2286: Do not log WARNING about PlacementPolicy for tests and simple setups
URL: https://github.com/apache/bookkeeper/pull/2286#issuecomment-598092255
@atris @mino181295 @diegosalvi 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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [bookkeeper] sijie merged pull request #2286: Do not log WARNING
about PlacementPolicy for tests and simple setups
Posted by GitBox <gi...@apache.org>.
sijie merged pull request #2286: Do not log WARNING about PlacementPolicy for tests and simple setups
URL: https://github.com/apache/bookkeeper/pull/2286
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [bookkeeper] mino181295 commented on issue #2286: Do not log
WARNING about PlacementPolicy for tests and simple setups
Posted by GitBox <gi...@apache.org>.
mino181295 commented on issue #2286: Do not log WARNING about PlacementPolicy for tests and simple setups
URL: https://github.com/apache/bookkeeper/pull/2286#issuecomment-598162992
+1, 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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [bookkeeper] atris commented on issue #2286: Do not log WARNING
about PlacementPolicy for tests and simple setups
Posted by GitBox <gi...@apache.org>.
atris commented on issue #2286: Do not log WARNING about PlacementPolicy for tests and simple setups
URL: https://github.com/apache/bookkeeper/pull/2286#issuecomment-598164297
+1, please go ahead
--
Regards,
Atri
*l'apprenant*
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services