You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/11/05 15:04:15 UTC

[GitHub] [pulsar] eolivelli opened a new pull request #12643: [Functions] Prevent NPE while stopping a non started Pulsar LogAppender

eolivelli opened a new pull request #12643:
URL: https://github.com/apache/pulsar/pull/12643


   ### Motivation
   
   Exception in thread "FUNCTIONAME-0" java.lang.NullPointerException
   at org.apache.pulsar.functions.instance.LogAppender.stop(LogAppender.java:111)
   at org.apache.pulsar.functions.instance.JavaInstanceRunnable.close(JavaInstanceRunnable.java:472)
   at org.apache.pulsar.functions.instance.JavaInstanceRunnable.run(JavaInstanceRunnable.java:288)
   
   ### Modifications
   
   Add a Null Check
   
   ### Verifying this change
   
   This change is a trivial rework / code cleanup without any test coverage.
   
   ### Documentation
   
   - [x] `no-need-doc` 
   
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] merlimat merged pull request #12643: [Functions] Prevent NPE while stopping a non started Pulsar LogAppender

Posted by GitBox <gi...@apache.org>.
merlimat merged pull request #12643:
URL: https://github.com/apache/pulsar/pull/12643


   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] merlimat merged pull request #12643: [Functions] Prevent NPE while stopping a non started Pulsar LogAppender

Posted by GitBox <gi...@apache.org>.
merlimat merged pull request #12643:
URL: https://github.com/apache/pulsar/pull/12643






-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] devinbost commented on pull request #12643: [Functions] Prevent NPE while stopping a non started Pulsar LogAppender

Posted by GitBox <gi...@apache.org>.
devinbost commented on pull request #12643:
URL: https://github.com/apache/pulsar/pull/12643#issuecomment-962000578






-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] devinbost commented on pull request #12643: [Functions] Prevent NPE while stopping a non started Pulsar LogAppender

Posted by GitBox <gi...@apache.org>.
devinbost commented on pull request #12643:
URL: https://github.com/apache/pulsar/pull/12643#issuecomment-962000578


   Good catch. LGTM


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org