You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by li...@apache.org on 2018/01/01 14:39:21 UTC

kylin git commit: KYLIN-3143 the right way to use Preconditions.checkNotNull()

Repository: kylin
Updated Branches:
  refs/heads/master edc4397aa -> 3ec601004


KYLIN-3143 the right way to use Preconditions.checkNotNull()

Signed-off-by: lidongsjtu <li...@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 <wa...@qq.com>
Authored: Mon Jan 1 13:04:48 2018 +0000
Committer: lidongsjtu <li...@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));
     }