You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kylin.apache.org by pengjianhua <pe...@zte.com.cn> on 2017/11/24 13:20:09 UTC
Review Request 64063: KYLIN-3060 The logical processing of creating or
updating streaming table has a bug in server,
which will cause a NullPointerException.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64063/
-----------------------------------------------------------
Review request for kylin, Dayue Gao, Dong Li, hongbin ma, and Wang Xiaoyu.
Bugs: KYLIN-3060
https://issues.apache.org/jira/browse/KYLIN-3060
Repository: kylin
Description
-------
When you set the invalid parameter for creating or updating a streaming table, the client will show an error message dialog.
I analyze the Kylin server log, there are some null pointer exceptions, then I analyze the StreamingController.java, whose logical processing of streaming table has a bug, which will cause a NullPointerException.
So I have repaired it, then the client will tell you the detail error message, the server no longer throws an null point exception.
you can check the patch, thanks!
Kylin server log
2017-11-24 18:42:21,531 ERROR [http-bio-7070-exec-10] controller.StreamingController:202 : Failed to deal with the request:null
java.lang.NullPointerException
at org.apache.kylin.source.kafka.KafkaConfigManager.updateKafkaConfig(KafkaConfigManager.java:156)
at org.apache.kylin.rest.service.KafkaConfigService.updateKafkaConfig(KafkaConfigService.java:84)
at org.apache.kylin.rest.controller.StreamingController.updateStreamingConfig(StreamingController.java:198)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:606)
Diffs
-----
server-base/src/main/java/org/apache/kylin/rest/controller/StreamingController.java 593abea
Diff: https://reviews.apache.org/r/64063/diff/1/
Testing
-------
Thanks,
pengjianhua