You had one job Netty logging guard

In pre-release versions of Elasticsearch 5.0.0, users were subject to
log messages of the form "your platform does not.*reliably.*potential
system instability". This is because we disable Netty from being unsafe,
and Netty throws up this scary info-level message when unsafe is
unavailable, even if it was unavailable because the user requested that
it be unavailabe. Users were rightly confused, and concerned. So, we
contributed a guard to Netty to prevent this log message from showing up
when unsafe was explicitly disabled. This guard shipped with all
versions of Netty that shipped starting with Elasticsearch
5.0.0. Unfortunately, this guard was lost in an unrelated refactoring
and now with the 4.1.10.Final upgrade, users will again see this
message. This commit is a hack around this until we can get a fix
upstream again.

Relates #24469
This commit is contained in:
Jason Tedor 2017-05-03 18:49:08 -04:00 committed by GitHub
parent cacba6bc46
commit 06364cf6f0
1 changed files with 5 additions and 1 deletions

View File

@ -101,7 +101,11 @@ class Netty4InternalESLogger extends AbstractInternalLogger {
@Override
public void info(String msg) {
logger.info(msg);
if (!("Your platform does not provide complete low-level API for accessing direct buffers reliably. " +
"Unless explicitly requested, heap buffer will always be preferred to avoid potential system " +
"instability.").equals(msg)) {
logger.info(msg);
}
}
@Override