Clojurians Log v2

Perhaps just for convenience?

Or perhaps their architecture requires specific changes to it? Don't know. You could ask on their dedicated slack

yeah, maybe I will do that.

My suggestion is that split is required because of different JIT engines in charge between openjdk and graalvm which may require additional treatment from the observability point of view.

C1, C2 vs Graal (?)

This means that a thread is started on the top level. E.g.:

(def x (future :hello))

yes I tried doing --initialize-at-run-time=... but think I am not selecting the classes properly. When I use the library https://clojars.org/com.github.clj-easy/graal-build-time I get a similar recommendation to initialize the class at runtime.

Error: Classes that should be initialized at run time got initialized during image building:
org.eclipse.jetty.util.Jetty was unintentionally initialized at build time. To see why org.eclipse.jetty.util.Jetty got initialized use --trace-class-initialization=org.eclipse.jetty.util.Jetty

You should add --initialize-at-build-time=org.eclipse.jetty if you are instantiating jetty on the top level

or push that to a delay or function

why do you need jetty btw?

running a server when starting the native executable

tell me what this has to do with datahike? try to use the least amount of dependencies possible when getting started with a native image

if you want a server that works out of the box with graalvm native image, try org.httpkit.server

I thought that jetty was supported with graalvm native image. https://github.com/clj-easy/graalvm-clojure/tree/master/ring-jetty

ok, perhaps, but you need to include the extra build time arguments then. https://app.slack.com/client/T03RZGPFR/threads?cdn_fallback=1#:~:text=jetty.util.Jetty-,5m,-borkdude

the example in the graal-clojure repo uses --initialize-at-build-time which is now deprecated, you need to include the full list of package prefixes

but you can cheat with --initialize-at-build-time=. - don't tell anyone ;)

yea I am trying to get datahike compiling using native image the server I am also interested in / partly also using the server to easily test it.

the graal-build-time makes it default to that again right? so you would then only have to specify the runtime ones?

no, graal-build-time only specifies clojure packages, you still have to specify all Java packages yourself

in general you should try to keep the list as small as possible

oh! and jetty is not

yeah

so I am basically using Jetty because that is the recommended starter by Eric Normand and http-kit was in the give it a pass. https://ericnormand.me/mini-guide/clojure-web-servers

Don't just believe everything people say :)

It depends on your use case

But you can continue with jetty if you want of course, just add those arguments

haha. Yea I have had trouble finding proper comparisons of the different servers.

so this --initialize-at-build-time=org.eclipse.jetty works though I am not sure how to get this to work for superv.async

what is superv.async

the first one is just the org from the ring-jetty-adapter . Doing that with io.replikativ doesn't make much sense because there is also konserve etc. It says to do it with the class name. But when I look in the target directory classes I can't see how the jetty one maps to anything there.

It's a library that provides Erlang style error handling. https://github.com/replikativ/superv.async

Doesn't that lib already have support for graalvm? https://github.com/replikativ/superv.async/blob/5c1ac415b9642e5fb53dccca939dd1a9d44b76ad/src/superv/async.cljc#L113

I'm still getting that build error for some reason.

are you using the correct version?

and what error exactly?

yea I have been working up and down the stack. Even just trying to run superv.async

the one of top level of thread here. The thread error

That indicates that a thread is being started from the top level. Maybe the native image check isn't working anymore

I'm not using it actually.

;; a simple global instance, will probably be removed
(def S
  (try
    ;; We cannot run the simple-supervisor thread in a static context inside
    ;; native image.
    (dummy-supervisor)
    #_#?(:clj (if (native-image-build?)
                (dummy-supervisor)
                (simple-supervisor))
         :cljs (simple-supervisor))
    (catch #?(:clj Exception :cljs js/Error) _
      (dummy-supervisor) #_(simple-supervisor))))

yes, but it will still be used simply because it's a top level thing

I commented it out to just use the dummy-supervisor which was added for the native image support

ah right

maybe you can comment out more until you don't get the error and then detect where the thread is being started

Yea I was just trying to make it work quickly because working on getting it right isn't a major priority atm. So was hoping I could just initialise the class at runtime.

anything used from clojure cannot be initialized at runtime, unless you push it to a function (or delay which is essentially a function)

okay thats a valuable insight. Thanks. Perhaps I will try and pick apart superv.async a bit more and find where this issue is.