AMashenkov commented on a change in pull request #424:
URL: https://github.com/apache/ignite-3/pull/424#discussion_r747427125
##########
File path:
modules/schema/src/main/java/org/apache/ignite/internal/schema/NativeTypes.java
##########
@@ -195,6 +196,7 @@ public static NativeType timestamp() {
* @param val Object to map to native type.
* @return {@code null} for {@code null} value. Otherwise returns
NativeType according to the value's type.
*/
+ @Contract("null -> null")
Review comment:
We already use jetbrains annotations in the project (see `@Nullable` and
`@NotNull)`.
The contract is clearly described in javadoc and the annotation just helps
to suppress IDEA warning and affect nothing.
Proof:
`Note that this annotation just describes how the code works and doesn't add
any functionality by means of code generation.`
--
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]
