Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support Storm 1.0.0+ #177

Merged
merged 5 commits into from Nov 5, 2016
Merged

Support Storm 1.0.0+ #177

merged 5 commits into from Nov 5, 2016

Commits on Nov 1, 2016

  1. changes for getting storm-1.0.0 to build and pass tests

    (Also includes changes to resolve conflicts from merging erikdw/storm-mesos:1.0.0-WIP into master.)
    
    I got the tests to work for `storm-1.0.0` based on DarinJ/storm:1.0.0-WIP with a number of changes:
    
    Main changes for fixing tests:
    * change "config maps" from `Map<String, TopologyDetails>` type (which is just *wrong*) to just `Map`.
    * add in a couple of keys that are required by TopologyDetails via asserts now:
     * `topologyConf.put("topology.worker.max.heap.size.mb", 768.0);`
     * `topologyConf.put("topology.priority", 0);`
    * change some constructor arguments:
     * removed final null param in `SupervisorDetails` case
     * added extra param in `Cluster` case (this one is also storm-1.0+ specific)
    
    Other main changes:
    * added storm-shim-1x module and corresponding profile to top-level pom.xml
    * `import org.apache.storm.thrift.TBase;` instead of `import org.apache.thrift7.TBase;`
    * updated `bin/build-release.sh` to support setting profile for 1x
    * ran this sed to fixup stuff appropriately (note `gsed` from brew since I'm on Mac)
     * `find . -type f -not -path '*/\.*' -not -name '*.class' -not -path '*storm-shim-9x*' -not -path '*storm-shim-10x*' -print0 | xargs -0 gsed -i 's/backtype/org.apache/g'`
    erikdw committed Nov 1, 2016
    Copy the full SHA
    5a3710e View commit details
    Browse the repository at this point in the history
  2. enable storm1.0 profile by default instead of storm9, since we are ch…

    …anging the master branch to only support storm-1.0+
    erikdw committed Nov 1, 2016
    Copy the full SHA
    c629c60 View commit details
    Browse the repository at this point in the history
  3. remove redundant org.mockito:mockito-all dependency which was causing…

    … WARNINGs to be emitted during mvn builds
    erikdw committed Nov 1, 2016
    Copy the full SHA
    7611d94 View commit details
    Browse the repository at this point in the history
  4. update Travis CI config to remove references to the pre-1.0 versions …

    …of storm which are no longer supported in master, and also update the latest Mesos release to 1.0.1
    erikdw committed Nov 1, 2016
    Copy the full SHA
    5c111f8 View commit details
    Browse the repository at this point in the history
  5. remove shims for storm-0.x since master will not support those versio…

    …ns, and update default storm version from 1.0.0 to 1.0.2
    erikdw committed Nov 1, 2016
    Copy the full SHA
    b09cbc6 View commit details
    Browse the repository at this point in the history