-
-
Notifications
You must be signed in to change notification settings - Fork 239
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
Switch to Java 21 #660
Comments
Hi @JohannesLichtenberger, would like to help out with this issue if its up for grabs |
Thanks, go ahead :) |
Hi there. Is this issue still being worked on? |
@Adaminite Has been assigned to me. Im working on this. |
@adarshjhaa100 and @Adaminite, I guess I couldn't wait and wanted to check locally, especially the generational ZGC. At least I've locally switched to Java 21 and replaced |
@adarshjhaa100 and @Adaminite, it would be more than excellent if you could work on other (more interesting?) issues :) |
@JohannesLichtenberger Sure no problem. I was doing some analysis on new features added to Java 21 and there's some stuff that could be used to enhance the project. The work and time required for this would be significant, and you would be able to do justice to that. Unassigning myself. Would be interested in tracking this tread though and see what you guys could come up with. |
@adarshjhaa100, seems I've still some stuff to do, but have to go to bed now: https://github.com/sirixdb/sirix/tree/update-to-java21 |
So, if you want to work on this... :-) I got to |
Current issue is that Kotlin does not have a Java 21 target |
Yep, official docs suggest support till 20, Iast kotlin release is 1.19.10 that came out last month. Got to track the jetbrains release schedule and roadmap. |
@adarshjhaa100 I wanted to check generational ZGC, but apparently at least with preconfigured buffer sizes maybe it seems to run out of foreign memory, without it's no problem... |
Well, simply seems that direct ByteBuffers are not cleaned up and probably not subject for cleanup somehow. |
Did you try adding both these flags: -XX:+UseZGC -XX:+ZGenerational |
Yes |
One question remains, which objects is it identifying "young" or "old". And what's the time in which "young" objects need to be collected. Btw, is there any reference to ByteBuffer which is preventing it from getting cleaned up. Do you have any profiling data available for this? These flags can help: -Xlog:gc* and -verbose:gc |
@adarshjhaa100 yes, I know the JVM flags, but not at home today. Maybe I can check tomorrow, but it's in the trx commit in |
sirix/bundles/sirix-core/src/main/java/io/sirix/access/trx/page/NodePageTrx.java Line 444 in dca0033
|
@ adarshjhaa100 I've attached the log, but I suppose it's not helping that much.
Also, I'm currently not having much time to work on the issue, so anyone can grab it (maybe using my branch as a base). |
@JohannesLichtenberger Im also working on benchmarking the generational ZGC on my own. Will try to so some additional analysis here once im done with that. Also, if there isn't any hard time constraint on the Java version upgrade, I can reach out to you with some updates on that by next week as I have a busy schedule this week. Will keep posting the results of my ZGC benchmark here though. |
@adarshjhaa100 any news already? :-) |
@JohannesLichtenberger been busy with work. Will share the progress by this weekend |
@adarshjhaa100 I guess you have had no time!? |
@JohannesLichtenberger apologies for the delay. I've been busy with a production issue in my org since last couple of weeks. Hope you understand. |
Sure |
That said, I'm still looking forward to your insights, once you have some time and if you like to spend it on this 😉 |
Sure, I've taken off this Friday will take a look for sure. The issue's with the generational ZGC not working properly in this case, right? |
Yes, resulting in |
You'll need this file: https://www.rakuten-drive.com/transfer/download/96EB5CQP (downloadable until Thursday) |
Guess you haven't had time!? |
Sorry for the delayed response. Due to some health issues. I've been away from any sorts of work since past couple of weeks. Logged in today only. Having said that, i've started working on the analysis with generational ZGC and version upgrade. If it's required I can continue working on this. |
You can check my branch. I think there's only a Gradle issue left... |
Hope you're doing fine again :-) |
Yea better than before :-). It's the onset of winter when pollution suddenly shoots up. |
@adarshjhaa100 I've had some time after work today, so I managed to finish the Java update :) but of course still looking forward to your input and maybe contributions (would be more than awesome) 👍 |
No description provided.
The text was updated successfully, but these errors were encountered: