John Gilmore
John Gilmore
It's not necessary, no. It was an effort to combine code that (to my mind at least) belongs together. If I had not created the multilang package, something like the...
I've now moved the existing code back to where it was and updated the dependancies. With the new code in the new package.
Will do. Referring to what @francis-liberty mentioned, that there might be other message types, already internal to Storm, which implement the fields of Immission and Emission? Could @xumingming comment on...
An emission is something that is received from somewhere, while an immission is something that is sent to somewhere. Admittedly, immission is not a word that is regularly used, although...
That is how I understood the purpose of tuple. @francis-liberty you'll note that both Emission and Immission have tuple fields. Although they are lists of objects, that represent the idea...
I could perhaps call an Immission a BoltMsg. Then emissions are messages sent from both spout and bolt non-JVM components to shell components, A SpoutMsg is a message sent from...
So I've changed these names now to make them more understandable. But just for interest's sake: you don't have to play with "e-"s and "i-"s. They're just words for what...
My editor removes trailing whitespace. I thought this was required. The editor removed trailing whitespace in Config.java (of which there was a lot). I couldn't really find shell component tests....
I've had a look at the Storm tests. Specifically the shell component tests for ruby and python (multilang_test.clj). These tests also test the changes I made to Storm. They start...
So I've reset the pure whitespace changes to project.clj and bootstrap.clj. Config.java was not a pure whitespace change; I added the STORM_MULTILANG_SERIALIZER item. Just say if want me to re-add...