mesos / storm

Storm on Mesos!
Apache License 2.0
139 stars 66 forks source link

Update protobuf version to fix Issue #221 #233

Closed JessicaLHartog closed 6 years ago

JessicaLHartog commented 6 years ago

For more details, see comments on #221

erikdw commented 6 years ago

@JessicaLHartog : just wanna state explicitly something I'm sure you are tracking: you'll need to rebase this change after merging #232, and then force-push in order to have this PR only have the 2 commits to the pom.xml & .travis.yml.

JessicaLHartog commented 6 years ago

@erikdw Yep, I knew I'd have to rebase, I just wanted Travis not to yell at me and make it so that this doesn't have merge conflicts while both this PR and #232 were open for review.