birschick-bq commented on pull request #2449: URL: https://github.com/apache/calcite/pull/2449#issuecomment-908642395
> @birschick-bq > > > Hmm. Build errors raised by CheckerFramework seem to be a know bug [typetools/checker-framework#979](https://github.com/typetools/checker-framework/issues/979). May need to suppress this false-positive error. > > The errors seem to be related to the nullability of `Class#getClassLoader()` method. According to the javadoc of the respective method it may return `null`. > > > Some implementations may use null to represent the bootstrap class loader > > Is it legal to pass `null` in the `getDefaultCompilerFactory`? @zabetak It is "legal" to pass `null` to getDefaultCompilerFactory - as the parameter is not marked with any explicit attribute. The other methods (i.e., `setParentClassLoader`) have an "explicit" attribute for `@Nullable`. This is an issue of the **CheckerFramework** - not the source code. -- 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]
