You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@kylin.apache.org by "Ted Yu (JIRA)" <ji...@apache.org> on 2016/11/15 02:10:58 UTC

[jira] [Updated] (KYLIN-2172) Potential NPE in ExecutableManager#updateJobOutput

     [ https://issues.apache.org/jira/browse/KYLIN-2172?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ted Yu updated KYLIN-2172:
--------------------------
    Description: 
{code}
           if (newStatus != null && oldStatus != newStatus) {
...
        } catch (PersistentException e) {
            logger.error("error change job:" + jobId + " to " + newStatus.toString());
{code}

If newStatus is null and the execution got into catch block, newStatus.toString() would produce NPE.

  was:
{code}
           if (newStatus != null && oldStatus != newStatus) {
...
        } catch (PersistentException e) {
            logger.error("error change job:" + jobId + " to " + newStatus.toString());
{code}
If newStatus is null and the execution got into catch block, newStatus.toString() would produce NPE.


> Potential NPE in ExecutableManager#updateJobOutput
> --------------------------------------------------
>
>                 Key: KYLIN-2172
>                 URL: https://issues.apache.org/jira/browse/KYLIN-2172
>             Project: Kylin
>          Issue Type: Bug
>            Reporter: Ted Yu
>            Priority: Minor
>
> {code}
>            if (newStatus != null && oldStatus != newStatus) {
> ...
>         } catch (PersistentException e) {
>             logger.error("error change job:" + jobId + " to " + newStatus.toString());
> {code}
> If newStatus is null and the execution got into catch block, newStatus.toString() would produce NPE.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)