I disagree. Intellij idea IDE has a static code analysis, which uses that annotation too. IDE highlights possible problems. It helps to make our code more stable and bugless.
пт, 27 мар. 2020 г. в 12:06, Pavel Tupitsyn <ptupit...@apache.org>: > I disagree, it would be a step back. > > > What's the reason for using it? > Null was a billion dollar mistake [1]. > NullPointerExceptions happen quite a lot in Ignite, and annotations provide > some clues to avoid those. > > [1] https://en.wikipedia.org/wiki/Tony_Hoare#Apologies_and_retractions > > On Fri, Mar 27, 2020 at 10:58 AM Anton Vinogradov <a...@apache.org> wrote: > > > 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 > > > -- BR, Sergey Antonov
