You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by GitBox <gi...@apache.org> on 2020/05/29 23:23:23 UTC
[GitHub] [druid] clintropolis opened a new pull request #9952: only close exec if it exists
clintropolis opened a new pull request #9952:
URL: https://github.com/apache/druid/pull/9952
Fix NPE of the form
```
java.lang.NullPointerException: null
at org.apache.druid.indexing.kinesis.KinesisRecordSupplier.close(KinesisRecordSupplier.java:563) ~[?:?]
...```
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
[GitHub] [druid] jihoonson commented on pull request #9952: Fix NPE in KinesisRecordSupplier when closing on supervisor
Posted by GitBox <gi...@apache.org>.
jihoonson commented on pull request #9952:
URL: https://github.com/apache/druid/pull/9952#issuecomment-636267151
The CI failed due to insufficient test coverage, but I don't think that applies to this change. I'm merging this PR.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org
[GitHub] [druid] jihoonson merged pull request #9952: Fix NPE in KinesisRecordSupplier when closing on supervisor
Posted by GitBox <gi...@apache.org>.
jihoonson merged pull request #9952:
URL: https://github.com/apache/druid/pull/9952
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org