julianhyde opened a new pull request, #3939: URL: https://github.com/apache/calcite/pull/3939
In [CALCITE-4199](https://issues.apache.org/jira/browse/CALCITE-4199) we enabled CheckerFramework (via annotations and CI jobs) in the 'core' and 'linq4j' modules; this change further enables CheckerFramework in the 'server' module, and fixes all violations. There are also a large number of 'cosmetic' modifications to improve code quality without changing behavior, including: * Replace `this.x = x; assert x != null;` with `this.x = requireNonNull(x);` in constructors * Replace `assert` in other code locations where it implements an invariant. We don't use `requireNonNull` because it throws `NullPointerException`; we would prefer to throw `AssertionError` or `IllegalStateException` * Replace `x.equals("")` and `x.size() == 0` with `x.isEmpty()` * Make fields `final` where possible * Make private methods and inner classes `static` where possible * In class `Pair<K, V>` make the type variables `K` and `V` no longer nullable by default (you can make each of them nullable if you need) * Always import `Objects.requireNonNull`, `Integer.parseInt`, `Float.parseFloat`, `Double.parseDouble`, `Byte.parseByte`, `Short.parseShort`, `Boolean.parseBoolean`, `Long.parseLong` via static import This PR supersedes #3937 -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
