Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/449
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabl
Github user d2r commented on the pull request:
https://github.com/apache/storm/pull/449#issuecomment-146644018
@duke-bartholomew , Seems STORM-694 was closed because #492 was merged in.
Should this pull request be closed?
---
If your project is set up for it, you can reply to this e
Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/492#issuecomment-91033826
Thanks @knusbaum merged it into master.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project d
Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/492
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabl
Github user harshach commented on the pull request:
https://github.com/apache/storm/pull/492#issuecomment-90666963
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user ptgoetz commented on the pull request:
https://github.com/apache/storm/pull/492#issuecomment-90661810
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and w
Github user nathanmarz commented on the pull request:
https://github.com/apache/storm/pull/492#issuecomment-87942556
+1
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled an
Github user duke-bartholomew commented on the pull request:
https://github.com/apache/storm/pull/449#issuecomment-87809406
@knusbaum Thanks for the input :+1:
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your proje
Github user knusbaum commented on the pull request:
https://github.com/apache/storm/pull/492#issuecomment-87761632
#449
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled a
GitHub user knusbaum opened a pull request:
https://github.com/apache/storm/pull/492
STORM-694: java.lang.ClassNotFoundException:
backtype.storm.daemon.common.SupervisorInfo
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/knusba
Github user knusbaum commented on the pull request:
https://github.com/apache/storm/pull/449#issuecomment-87743965
You're including `common` twice, once on line 29, and once on line 18.
The real solution is to remove the import of `[backtype.storm.daemon.common
SupervisorInfo]
Github user duke-bartholomew commented on the pull request:
https://github.com/apache/storm/pull/449#issuecomment-85884589
@caofangkun
indeed, you are right. If I start from a new clone, it seems to work.
Thanks for your reply!
It still confuses me though why this is
Github user caofangkun commented on the pull request:
https://github.com/apache/storm/pull/449#issuecomment-85783849
@duke-bartholomew
It works very well when I try to reproduce this issue:
```
git clone https://github.com/apache/storm.git
cd storm
mvn
Github user duke-bartholomew commented on the pull request:
https://github.com/apache/storm/pull/449#issuecomment-85680330
@caofangkun
I just tried again on the latest master branch, and sadly enough, yes the
issue is still there.
When executing 'mvn package' or 'mvn test' I s
Github user caofangkun commented on the pull request:
https://github.com/apache/storm/pull/449#issuecomment-85329513
@duke-bartholomew
Does this still a issue?
See SupervisorInfo has already be imported at :
https://github.com/apache/storm/blob/master/storm-core/src/clj
15 matches
Mail list logo
