Steven Schlansker
Steven Schlansker
I don't think this is intentionally not supported, but neither has it been tested to work. We'd love to add this feature.
Thanks for this. I think to integrate this, we will need to also include test cases, that check that the `reflect-config.json` is correct and does not get out of date.
It looks like it should be possible to setup graalvm in ci: https://github.com/graalvm/setup-graalvm
It looks like the graal native maven plugin does a lot for us. I started https://github.com/jdbi/jdbi/pull/2476 to add graal native-image to ci. I ran into: ``` Produced artifacts: /home/runner/work/jdbi/jdbi/core/target/libawt.so (jdk_library)...
This is a broader change than the scope that you're trying to accomplish here, but I think the performance would be even better if we construct and cache a `MethodHandle`...
One last thought: it would be nice if we could see the improvement in a benchmark in `benchmarks/`. I think the improvement is clear enough from reading that this is...
Merging #2657 . Thanks!
You should be able to run `make install` to build and check everything.
Could they be factored out into a reusable module? It'd be nice to avoid having to duplicate classes.
@SentryMan , I don't feel great about merging it with duplicated classes, but if you keep the duplication in its own commit, then we can make forward progress and get...