You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ignite.apache.org by Ilya Kasnacheev <il...@gmail.com> on 2020/04/03 13:01:47 UTC

Re: Get rid of @Nullable and @NotNull

Hello!

I think that we should stick with existing coding guidelines, you may use
Nullable to specify that something may be null and it's fine.

Then, everything else is assumed not null and problem solved, NotNull not
needed.

Regards,
-- 
Ilya Kasnacheev


пт, 27 мар. 2020 г. в 10:58, Anton Vinogradov <av...@apache.org>:

> Folks,
>
> Found we still use @Nullable annotation.
>
> What's the reason for using it?
> Everything is Object and Nullable :)
>
> How about get rid of @Nullable usages and restrict its usage by checkstyle
> plugin?
>
> BTW, We already "do not use @NotNull annotation" (с) Coding Guidelines [1]
> which may have some cense in contrast to @Nullable.
> But I see a lot of usages at code.
>
> How about to get rid of @NotNull too and to add such check to checkstyle
> plugin too?
>
> [1]
>
> https://cwiki.apache.org/confluence/display/IGNITE/Coding+Guidelines#CodingGuidelines-@Annotations
>