Closed William-Sang closed 9 years ago
It should come soon, there is a pending pull request to make it compatible #9. But currently we do not support storm 0.9.3.
@ecanzonieri We really appreciate your effort.
Hi,
I've been able to run Pyleus successfully with Storm 0.9.3.2.2.0.0 (Hortonworks HDP 2.2 Build).
Any update on Storm 0.9.3 support?
We have support for heartbeats (see #97), but we still lack a pull request for #10 and manual testing with Storm 0.9.3
Is #10 required for releasing a version (latest -develop) that supports Storm 0.9.3?
I am using Pyleus 0.2.4-develop with Storm 0.9.3 and can confirm heartbeats are working.
As far as I know, it should be required. But I didn't try myself
For the record, I'm also using the latest develop branch and just changed the Storm dependency in pom.xml to "0.9.3", built is custom release of pyleus, and use this to build, deploy and run a topology on a 0.9.3 Storm. Works fine.
@poros Is the lack of a PR for #10 holding up the next release of Pyleus?
I've just tried to compile pyleus-base.jar with storm 0.9.3, now I can build and submit topologies. But they don't work. Here is 0.9.3 storm-supervisors' log: http://pastebin.com/0LvrNizZ
The same code works fine on 0.9.2
Here is 0.9.2 storm-supervisor log: http://pastebin.com/kn0Bwbxt
@kayrus are you sure you're running the latest -develop of Pyleus? I saw similar errors to that when running Pyleus 0.2.4 ... it was heartbeats that was choking things up, since Pyleus 0.2.4 doesn't know how to handle or unpack the heartbeat tuples.
Yes, I use latest develop branch. Topology works, but very, very slow. And it produces error messages. Here is the full log from 0.9.3: http://pastebin.com/ZhW3xDZM
I've tried to reproduce the same behavior with 0.9.2 - it works well.
@jwestboston As I said in another thread, I considered #10 as a blocking issue together with extensive testing with Storm 0.9.3. Storm is a complex system and it is not trivial to ensure that Pyleus does not trigger exotic bugs or degrades performance (as it seems to happen with 0.9.3).
However, I also feel that it is time for a new release. I'll coordinate with @patricklucas
Update: I opened a pull request for #10, but I noticed we also need #107 for properly supporting Storm 0.9.3
We switched directly to Storm 0.9.4. Please upgrade pyleus to version 0.3.0
Today I set up a storm 0.9.3 cluster, and run
Than it occurs: