-
Notifications
You must be signed in to change notification settings - Fork 84
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
Exception: namespace 'flambo.function' not found #42
Comments
See the section on AOT in the README https://github.com/yieldbot/flambo#aot |
The requiring namespaces are aot'd as per the readme. |
since |
https://gist.github.com/alilee/565d0097b4c36edcbd26 I'm starting to have issues when compiling/re-constructing serialised functions with closures (referencing let or param bindings inside the f/fn). Could this be related? Thanks very much for your help. |
I think this could be related to some subtleties in how serialised-fn works - there isn't much guidance in the docs. This is what I'm having trouble with:
If you take
|
parse/parse-int must also be a serializable-fn, otherwise it cannot be On Tuesday, December 16, 2014, Alister Lee notifications@github.com wrote:
|
I get the same issue with this:
Is that what you were suggesting? |
Hmmm. And same with this:
|
try moving your |
Is this the issue? (from serializable-fn): |
No, that is an old TODO. Deep recursion works. I'll try your example when On Tuesday, December 16, 2014, Alister Lee notifications@github.com wrote:
|
Thanks a million. I think that is the issue though. Is your version ahead of yieldbot's?
|
So-- it seems like there is a bug in the deep recursion where elements of a map are not recursed into for serialization properly. it is the |
Fantastic - thanks very much again for all your help. I will test! |
Ok, so it's not really a bug per se. https://github.com/sritchie/carbonite/blob/master/src/clj/carbonite/serializer.clj#L62-L69
Unfortunately there isn't a great fix, other than not using functions as keys or values in a map if you wish it to be serializable :/ |
Or just try the stuff from PR #27. I did take it into my fork (https://github.com/chrisbetz/flambo) and it works. I could check for you later today or you give it a try yourself. Cheers,
|
That works, but not in the repl. You'll have to AOT all your code, losing the ability to do ad-hoc or interactive spark stuff which may or may not matter to you. @chrisbetz I would be quite curious to see if that works in your fork or not. |
Hi, took me some time. You might want to check commit 99c998c in my fork for the test, which is "green". Happy holidays, Chris |
Getting this error message requiring flambo.api:
It seems to be resolved if I aot it:
in the
project.clj
Any thoughts?
The text was updated successfully, but these errors were encountered: