-
-
Notifications
You must be signed in to change notification settings - Fork 60
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
Can this work without getClassLoader
?
#85
Comments
Hi there, sorry - not familiar with Google App Engine so can't make a recommendation offhand. |
It looks like nippy is using LZ4Factory to compress, which uses `getClassLoader`. Is there a way to use nippy which doesn't compress, or compresses using something which doesn't use `getClassLoader`?
|
It is possible to use different compressors and/or no compression, please Not sure if this'll help in your case though assuming it's a compile-time On Aug 30, 2016 00:05, "jeaye" notifications@github.com wrote:
|
Not sure if this'll help in your case though assuming it's a compile-time
dependency error?
No, this is a run-time error which happens when I try to freeze data using nippy's default configuration. Google App Engine prevents the usage of `getClassLoader`, among other things, at run-time. Disabling compression would likely do the trick.
|
Great, then disabling compression should do the trick. On Aug 30, 2016 10:33, "jeaye" notifications@github.com wrote:
|
Bah, after further inspection this happens on load, at run-time, because of this: https://github.com/ptaoussanis/nippy/blob/master/src/taoensso/nippy/compression.clj#L101 Even if I don't enable encryption or compression, this def is evaluated on load. Ideally, there's a compile-time flag for this or the LZ4Factory isn't created until it's needed. Can you see a way in which you'd be willing to support this? |
Happy to consider a PR, otherwise will need a few weeks to be able to look First step will be confirming that this is the only thing stopping use on Cheers On Aug 30, 2016 11:37 AM, "jeaye" notifications@github.com wrote:
|
Just wrapping this global in a c.c/delay should fix that, no? |
Should do, just have my hands full with other priorities atm- PR welcome with confirmation that that's the only sticking issue for GAE. |
On Tue, Aug 30, 2016 at 03:05:06AM -0700, Max Penet wrote:
delay does the trick, for sure; that, combined with some workarounds for tufte and I'm running on GAE now. I'll follow up with a PR. |
Fix merged, thanks. |
Can we get a new version of this on clojars so I can stop using my fork? Thanks very much. 🙇 |
Sure, just pushed |
Thanks!
|
No problem :-) |
getClassLoader
is apparently not allowed on Google App Engine, where I'd love to run nippy to interface with my datastore. Unfortunately, I get run-time permission errors along the lines of:Thanks for the awesome Clojure projects.
The text was updated successfully, but these errors were encountered: