[GitHub] storm issue #2871: [STORM-3252] Bug fix for blobstore sync

2018-10-12 Thread kishorvpatil
Github user kishorvpatil commented on the issue: https://github.com/apache/storm/pull/2871 > @kishorvpatil I am not sure if we want to swallow IOException, since it may be due to some serious problem which we want to propagate and nimbus cannot continue in that case. Agreed. We

[GitHub] storm issue #2871: [STORM-3252] Bug fix for blobstore sync

2018-10-11 Thread arunmahadevan
Github user arunmahadevan commented on the issue: https://github.com/apache/storm/pull/2871 @revans2 , updated the log message, please check if it makes sense. @kishorvpatil I am not sure if we want to swallow IOException, since it may be due to some serious problem which we

[GitHub] storm issue #2871: [STORM-3252] Bug fix for blobstore sync

2018-10-11 Thread arunmahadevan
Github user arunmahadevan commented on the issue: https://github.com/apache/storm/pull/2871 Cherry picked commits from https://github.com/apache/storm/pull/2773 and linked with a JIRA. ---