pulsar-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [pulsar] cdbartholomew commented on issue #5079: Use "info" as the default root logger level
Date Sat, 14 Sep 2019 16:06:51 GMT
cdbartholomew commented on issue #5079: Use "info" as the default root logger level
URL: https://github.com/apache/pulsar/pull/5079#issuecomment-531491624
 
 
   I believe that this new default was also triggering a bug [3113](https://github.com/eclipse/jetty.project/issues/3133)
in jetty: "Logging of `key.readyOps()` can throw unchecked `CancelledKeyException`"
   
   This bug is only exposed if debug logging is enabled:
   
   `if (LOG.isDebugEnabled())
                           LOG.debug("selected {} {} {} ",key.readyOps(),key,attachment);
   `
   
   I was seeing this in the broker after sending lots of REST requests. After hitting this
bug, the broker no longer responds to REST requests, so it's pretty ugly.
   
   Once I changed root level logging to info, I no longer see the jetty bug.
   
   I think this is an important change to get in the next milestone so people don't trip over
this. It's also something to note can happen with debug logging enabled the broker even when
this default is changed, so it might be worth considering upgrading the jetty version. The
jetty bug is fixed in 9.4.15 (current Pulsar version is 9.4.12).
   

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

Mime
View raw message