[ https://issues.apache.org/jira/browse/APEXMALHAR-2508?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16048744#comment-16048744 ]
ASF GitHub Bot commented on APEXMALHAR-2508: -------------------------------------------- GitHub user ajaygit158 opened a pull request: https://github.com/apache/apex-malhar/pull/634 APEXMALHAR-2508 Add null check while calling cancel() for Timer instance in CacheManager @bhupeshchawda Kindly review You can merge this pull request into a Git repository by running: $ git pull https://github.com/ajaygit158/apex-malhar APEXMALHAR-2508 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/apex-malhar/pull/634.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #634 ---- commit 75210e5f355fe4f4ba720b27b222c7137ff8f6ee Author: ajaygit158 <ajaygit...@gmail.com> Date: 2017-06-14T06:05:06Z APEXMALHAR-2508 Add null check while calling cancel() for Timer instance in CacheManager ---- > CacheManager throws exception when the operator receives a shutdown request > --------------------------------------------------------------------------- > > Key: APEXMALHAR-2508 > URL: https://issues.apache.org/jira/browse/APEXMALHAR-2508 > Project: Apache Apex Malhar > Issue Type: Bug > Reporter: Ajay Gupta > Assignee: Ajay Gupta > Priority: Minor > > CacheManager initializes the Timer (refresher) only if refreshTime is not > null. However, in deactivate, before cancelling the timer, there is no null > check leading to NullPointerException. > Solution : Add a null check before cancelling the timer. -- This message was sent by Atlassian JIRA (v6.4.14#64029)