catch (...) {
} or {{catch (...) { logger.error() }}} merely to satisfy Java's compile-time exception checking. Always propagate the exception up or throw RuntimeException (or, if it "can't happen," AssertionError). This makes the exceptions visible to automated tests.logger.warn()
is often a cop-out: is this an error or not? If it is don't hide it behind a warn; if it isn't, no need for the warning.SSTableWriter writer = new SSTableWriter(cfs.getTempSSTablePath(), columnFamilies.size(), StorageService.getPartitioner()); |
var = bar == null ? doFoo() : doBar(); |
Please observe the following order for your imports:
\[blank line\] |
\[blank line\] |
https://c.statcounter.com/9397521/0/fe557aad/1/|stats