-
-
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
serializable-whitelist from a JVM property? #131
Comments
@jarohen Hi James! Yes, this is a nice idea - would totally be up for that. Cheers |
@jarohen Quick update: have sketched out some ideas for this, but would like to sleep on it before making any final choices. How time sensitive are you? Could this wait another day or two? |
Another day or two should be fine - thanks :) |
@jarohen Is there a specific reason that you're using ":" as a separator at https://github.com/juxt/crux/pull/1044/files#diff-a7947e61ae94e39666030e564721083dR23 as opposed to ","? |
Not particularly - I've been working with classpaths a lot recently so maybe it was fresh in my mind. Comma'd be fine :) |
Release info, relevant API docs. Please let me know if this addresses your issue? Cheers! |
That's great, cheers @ptaoussanis, and thanks for the fast turnaround 😄 I'll upgrade and run our tests against it - will let you know |
Looking good here 😄 Thanks, will close. |
Hi, I am trying to upgrade crux-core and crux-jdbc from 20.06-1.9.1-beta to 20.09-1.11.0-beta. Then I got an error at compile time
I figured out that this issue is related to the error, but I still can not find a way to add java.time.OffsetDateTime to serializable-whitelist. Anybody knows how to do that? (I am using [com.taoensso/nippy "2.15.3"]) |
Hi @qwwwpp!
Have you tried checking the docstring of You have a few options here, including:
(alter-var-root #'taoensso.nippy/*serializable-whitelist*
(fn [old-whitelist]
(conj old-whitelist "java.time.OffsetDateTime")))
All of these should do the trick, feel free to use whatever's most convenient. Apologies for the inconvenience, hope that helps! Cheers :-) |
Thank you for your help. I was using yogthos/config to control all my configurations, so I tried to change |
Great, thanks for confirming! |
Hey @ptaoussanis 😄
Re #130 - would you be open to defaulting the dynamic var to the value of a JVM property, say? In Crux it's our users' data that's getting frozen, so we don't know what classes they're using, and our users aren't necessarily using Clojure.
We're adding a property ourselves (xtdb/xtdb#1044) that we're checking on startup but I wonder whether this would be better living in Nippy?
Cheers,
James
The text was updated successfully, but these errors were encountered: