Replies: 1 comment 2 replies
-
|
@lukaseder You're welcome to update this implementation with a PR. We don't maintain the implementations, we just review that they fall within the test rules and benchmark them. For questions about specific implementations, you'd have to reach out to the person who made the commits. |
Beta Was this translation helpful? Give feedback.
2 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
-
I have a few questions / suggestions about the benchmarks using jOOQ:
https://github.com/TechEmpower/FrameworkBenchmarks/tree/master/frameworks/Java/play2-java/play2-java-jooq-hikaricp
FortuneRecordinApplication.fortunes()is using the API very un-fortune-ately, so to speak: https://github.com/TechEmpower/FrameworkBenchmarks/blob/547370f/frameworks/Java/play2-java/play2-java-jooq-hikaricp/app/controllers/Application.java#L55DSLContextinstance per query execution. Especially when running in a benchmark, I would expect theDSLContextto be injected or otherwise cached. Its underlyingConfigurationcontains a reflection cache, that can't be used in this particular case when callingfetchInto(FortuneRecord.class)(this applies to all methods!)FORTUNEcolumns are being projected inselect(), so why not just useselectFrom(FORTUNE)to get aFortuneRecordwithout the unnecessary reflection?Beta Was this translation helpful? Give feedback.
All reactions