yahoo / storm-yarn

Storm-yarn enables Storm clusters to be deployed into machines managed by Hadoop YARN.
Other
417 stars 161 forks source link

when supervisor completed not request new yarn containers #75

Closed tedxia closed 9 years ago

tedxia commented 9 years ago

When supervisor completed, MasterServer not request new containers because when call startAllSupervisors, numSupervisors always be 0.

revans2 commented 9 years ago

The change looks reasonable. If you want to contribute you need to file out https://github.com/yahoo/storm-yarn/blob/master/CLA.pdf and send it to storm-yarn-cla at yahoo-inc dot com.

I assume that you have tested this?

tedxia commented 9 years ago

I have test this on our test cluster based on yarn 2.4, and I have send the CLA file, please checkout it, thanks.

revans2 commented 9 years ago

@tedxia Thanks for the fix.