elasticsearch/elasticsearch — comment on commit 0c384af9bf7305b79002f7866e9fa038075269b2
File: plugins/transport/thrift/src/main/java/org/elasticsearch/thrift/ThriftRestImpl.java line 74
By nkvoll:

If that implementation is sure to never be changed, then I suppose it might be ok, its just that I generally think that "catch and ignore" is an anti-pattern where at least a logging message should be used. It wouldn't affect performance, and if it is changed down the line, it would at least show up in the logs.