-
Notifications
You must be signed in to change notification settings - Fork 65
DCell tries to setup regardless, so long as its required #51
Comments
Oh also I tried it in non jruby as well, did same thing with 2.0.0 |
It looks like you're trying to use DCell's facilities before it's initialized. I don't know what else to say but you can't do that? You need to move any code that uses DCell itself to run after DCell has been initialized. |
That error is occuring as soon as I require DCell. I commented out everything except require dcell, and thats enough to cause the error. |
Strange, that shouldn't happen. I'll have to take a more in-depth look |
More info, created new rails app, added DCell, bundle, couldnt reproduce error. I noticed dcell was locked at older version, so I did bundle update, installed a bunch of dependencies, and updated celluloid to 15.2 -- tried it again, error popped back up. So it happened somewhere on version update, it installed a bunch of new dependencies as well, reel, http_parser, celluloid-io, which i didnt see before. guessing it could be somewhere in there. ill play around with it more when i get home, here is an example of the bug in a brand new rails app, in this instance Im not actually even requiring dcell anywhere. https://github.com/jasonayre/dcell_bug_example
Bug should appear |
It seems to be an issue with reel, i pulled out the explorer stuff entirely and its fixed. I feel strongly that stuff should exist on its own in a separate codebase or gem anyways. Ill put up a pulll request. |
There is a known issue which I have not fixed where reel needs |
@halorgium I just got bitten by Reel requiring |
@halorgium while I assure you Reel has received quite a bit of love recently, that's a different metric than the overall maturity of the project. Perhaps we could better set expectations by explicitly noting the maturity of each project in the README |
@tarcieri yer, I meant perhaps my attention to the project with regard to the larger changes which |
@jasonayre do you still have original issue? |
Hi,
So, firstly, I just released a SOA framework gem that is powered by dcell
https://github.com/jasonayre/cylons
If I could get a link in the readme that would be cool!
But anyways, Im having an issue was wondering if you could shed some light on, I believe it to be a bug but..
I noticed when running my apps that are using my gem, when DCell shouldn't be connecting to anything or doing anything, it still acts like it tried to connect, or is somehow launching threads, Im not sure. I thought it was a bug in my own code, because I was really sloppy with my requires in the main file,
https://github.com/jasonayre/cylons/blob/master/lib/cylons.rb
-- The problem is, if I include my gem in a rails app, and I run a rake task, or spec, or do anything where rails is loaded, even though the railtie is being skipped, when I exit the console or task finishes, it throws this error:
SOMETIMES, it will throw that error I should say, but everytime without fail it says
Terminating 4 actors
As if it somehow trying to load dead supervision groups that don't even exist in the context of this application run? Im horribly confused as to what is happening, as it's not even connecting to ZK or anything.
Checkout https://github.com/jasonayre/cylons_demo
The admin app in particular, thats the one Ive been testing with. I commented out every require and all the code in my main module in the main cylons.rb file (and pointed to the local gem), and yet, it still does the same thing. Heres the thing, the railtie right now is responsible for creating the rpc service clients dynamically, from what is present in the zookeeper registry. So, theres no way for the app to know that any actors exist anywhere without there being a connection, which there cannot possibly be when all the code is commented out. I pushed a branch to demonstrate.
https://github.com/jasonayre/cylons/tree/dcell_bug_example
So ya if you pull down the cylons_demo + point the admin app to that branch, and load a console, you should see when you exit, either the big error, or the
Terminating 4 actors
It seems kind of random.
But ya, if you pull down gem and uncomment require 'dcell', youll see it won't happen.
Anyways, any idea what Im missing, or what could be happening?
P.S. great gem, hope you're planning on continuing development on it! :)
The text was updated successfully, but these errors were encountered: