[jira] [Commented] (KYLIN-3292) The setting config dialog will cause NPE in Kylin server

[ASF GitHub Bot (JIRA)]

[ 
https://issues.apache.org/jira/browse/KYLIN-3292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16410690#comment-16410690
 ] 

ASF GitHub Bot commented on KYLIN-3292:
---

chenzhx closed pull request #116: KYLIN-3292
URL: https://github.com/apache/kylin/pull/116
 
 
   

This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:

As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):

diff --git a/webapp/app/partials/admin/admin.html 
b/webapp/app/partials/admin/admin.html
index b57ce0eb0e..5c66942a7e 100644
--- a/webapp/app/partials/admin/admin.html
+++ b/webapp/app/partials/admin/admin.html
@@ -123,7 +123,7 @@ Update Config
   
 
 
-  Update
+  Update
   Cancel
 
   


 


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> The setting config dialog will cause NPE in Kylin server
> 
>
> Key: KYLIN-3292
> URL: https://issues.apache.org/jira/browse/KYLIN-3292
> Project: Kylin
>  Issue Type: Bug
>  Components: Web 
>Affects Versions: v2.4.0
>Reporter: Peng Xing
>Assignee: Peng Xing
>Priority: Minor
> Attachments: Set_config_null.png
>
>
> Open the 'Set Config' dialog, when the default key and value are null, the 
> update button should be disabled, please refer to the attach picture.
> Otherwise it will cause NPE in Kylin server, as follows.
> {code:java}
> java.lang.NullPointerException
> at 
> java.util.concurrent.ConcurrentHashMap.hash(ConcurrentHashMap.java:333)
> at 
> java.util.concurrent.ConcurrentHashMap.get(ConcurrentHashMap.java:988)
> at 
> org.apache.kylin.common.BackwardCompatibilityConfig.check(BackwardCompatibilityConfig.java:91)
> at 
> org.apache.kylin.common.KylinConfigBase.setProperty(KylinConfigBase.java:197)
> at 
> org.apache.kylin.rest.service.AdminService.updateConfig(AdminService.java:87)
> at 
> org.apache.kylin.rest.service.AdminService$$FastClassBySpringCGLIB$$e043a6d8.invoke()
> at 
> org.springframework.cglib.proxy.MethodProxy.invoke(MethodProxy.java:204)
> at 
> org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.invokeJoinpoint(CglibAopProxy.java:738)
> at 
> org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:157)
> at 
> org.springframework.security.access.intercept.aopalliance.MethodSecurityInterceptor.invoke(MethodSecurityInterceptor.java:69)
> at 
> org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:179)
> at 
> org.springframework.aop.framework.CglibAopProxy$DynamicAdvisedInterceptor.intercept(CglibAopProxy.java:673)
> at 
> org.apache.kylin.rest.service.AdminService$$EnhancerBySpringCGLIB$$f1977d91.updateConfig()
> at 
> org.apache.kylin.rest.controller.AdminController.updateKylinConfig(AdminController.java:106)
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

[jira] [Commented] (KYLIN-3292) The setting config dialog will cause NPE in Kylin server

[ASF subversion and git services (JIRA)]

[ 
https://issues.apache.org/jira/browse/KYLIN-3292?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16410691#comment-16410691
 ] 

ASF subversion and git services commented on KYLIN-3292:


Commit cfb21d7c97d72c9c4dfc1079b7d147ef8363b2f4 in kylin's branch 
refs/heads/master from [~xingpeng1]
[ https://gitbox.apache.org/repos/asf?p=kylin.git;h=cfb21d7 ]

KYLIN-3292 The update button should be disabled with none key and value in 
setting config dialog (#116)



> The setting config dialog will cause NPE in Kylin server
> 
>
> Key: KYLIN-3292
> URL: https://issues.apache.org/jira/browse/KYLIN-3292
> Project: Kylin
>  Issue Type: Bug
>  Components: Web 
>Affects Versions: v2.4.0
>Reporter: Peng Xing
>Assignee: Peng Xing
>Priority: Minor
> Attachments: Set_config_null.png
>
>
> Open the 'Set Config' dialog, when the default key and value are null, the 
> update button should be disabled, please refer to the attach picture.
> Otherwise it will cause NPE in Kylin server, as follows.
> {code:java}
> java.lang.NullPointerException
> at 
> java.util.concurrent.ConcurrentHashMap.hash(ConcurrentHashMap.java:333)
> at 
> java.util.concurrent.ConcurrentHashMap.get(ConcurrentHashMap.java:988)
> at 
> org.apache.kylin.common.BackwardCompatibilityConfig.check(BackwardCompatibilityConfig.java:91)
> at 
> org.apache.kylin.common.KylinConfigBase.setProperty(KylinConfigBase.java:197)
> at 
> org.apache.kylin.rest.service.AdminService.updateConfig(AdminService.java:87)
> at 
> org.apache.kylin.rest.service.AdminService$$FastClassBySpringCGLIB$$e043a6d8.invoke()
> at 
> org.springframework.cglib.proxy.MethodProxy.invoke(MethodProxy.java:204)
> at 
> org.springframework.aop.framework.CglibAopProxy$CglibMethodInvocation.invokeJoinpoint(CglibAopProxy.java:738)
> at 
> org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:157)
> at 
> org.springframework.security.access.intercept.aopalliance.MethodSecurityInterceptor.invoke(MethodSecurityInterceptor.java:69)
> at 
> org.springframework.aop.framework.ReflectiveMethodInvocation.proceed(ReflectiveMethodInvocation.java:179)
> at 
> org.springframework.aop.framework.CglibAopProxy$DynamicAdvisedInterceptor.intercept(CglibAopProxy.java:673)
> at 
> org.apache.kylin.rest.service.AdminService$$EnhancerBySpringCGLIB$$f1977d91.updateConfig()
> at 
> org.apache.kylin.rest.controller.AdminController.updateKylinConfig(AdminController.java:106)
> {code}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)