Github user revans2 commented on the pull request:

    https://github.com/apache/storm/pull/644#issuecomment-128368560
  
    I am fine with those changes in principal.  Although I do want to spend 
some time reading through the code to convince myself that there are no corner 
cases that we are missing.
    
    Also I would love to see some unit tests to show we can recover.  I believe 
that hsync works on the local file system so you would not need to bring up an 
HDFS mini cluster, just create a file system using ```file:///...``` as the URL 
and write there. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

Reply via email to