[GitHub] [ignite-3] AMashenkov commented on a change in pull request #424: IGNITE-15783 Mapper API and simple implementation.

[GitBox <gi...@apache.org>]

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.




-- 
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: notifications-unsubscribe@ignite.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org