Repository: kylin Updated Branches: refs/heads/master edc4397aa -> 3ec601004
KYLIN-3143 the right way to use Preconditions.checkNotNull() Signed-off-by: lidongsjtu <lid...@apache.org> Project: http://git-wip-us.apache.org/repos/asf/kylin/repo Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/3ec60100 Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/3ec60100 Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/3ec60100 Branch: refs/heads/master Commit: 3ec60100454bd2ff841a77581c32bdb3d9b7074e Parents: edc4397 Author: ChaoLong <wayn...@qq.com> Authored: Mon Jan 1 13:04:48 2018 +0000 Committer: lidongsjtu <lid...@apache.org> Committed: Mon Jan 1 21:26:12 2018 +0800 ---------------------------------------------------------------------- .../src/main/java/org/apache/kylin/rest/security/ManagedUser.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/kylin/blob/3ec60100/server-base/src/main/java/org/apache/kylin/rest/security/ManagedUser.java ---------------------------------------------------------------------- diff --git a/server-base/src/main/java/org/apache/kylin/rest/security/ManagedUser.java b/server-base/src/main/java/org/apache/kylin/rest/security/ManagedUser.java index 73a4db0..0fb2791 100644 --- a/server-base/src/main/java/org/apache/kylin/rest/security/ManagedUser.java +++ b/server-base/src/main/java/org/apache/kylin/rest/security/ManagedUser.java @@ -158,7 +158,7 @@ public class ManagedUser extends RootPersistentEntity implements UserDetails { } public void removeAuthoritie(String auth) { - Preconditions.checkNotNull(this.authorities == null); + Preconditions.checkNotNull(this.authorities); authorities.remove(new SimpleGrantedAuthority(auth)); }