Github user danny0405 commented on the issue:

    https://github.com/apache/storm/pull/2647
  
    @revans2 
    I approve with you promotion totally.
    
    The only concern are all kinds of cache we use here, now storm has many 
caches not just for scheduling. I just think if we can make a 
disk-storage-backend for all of such caches master needs. Disk cache has better 
fault-tolerance and is much cheeper than memory, but this is another promotion 
direction, and has nothing to do with this patch.
    
    BTW: thx for your nice work.



---

Reply via email to