Github user revans2 commented on the issue:

    https://github.com/apache/storm/pull/2442
  
    @jerrypeng I reverted the one rename you asked about and rebased to fix a 
merge conflict.  I also squashed because I thought we were really close on this.
    
    I don't understand your comment about using heap space instead of stack 
space with the -Xss setting.  We don't have Xss anywhere in the code base.  If 
you could point out the code that has issues I am happy to look at it.
    
    As for Configs.java we already support that.  We use a service loader for 
org.apache.storm.validation.Validated to enable config validation. That is how 
we made org.apache.storm.DaemonConfig and org.apache.storm.hdfs.spout.Configs 
work.  If you really want me to try and clean things up for 2.x I can move some 
of the configs into separate classes, but that might be better as a separate 
jira.


---

Reply via email to