Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
bridgekeeper[bot] created a comment on a pull request on openjdk/jdk
<!-- Welcome message --> :wave: Welcome back jvernee! A progress list of the required criteria for merging this PR into `master` will be added to the body of your pull request. There are additiona...
ascarpino created a comment on a pull request on openjdk/jdk
Is `volatile` a known performance decelerator? I don't see any source control history for that part of the code, so I can only speculate at `previousKey` was to improve benchmarking. Did you try ...
plummercj created a review comment on a pull request on openjdk/jdk
It looks like `sizetType` is initialized further down in the constructor. You could move that up (and the other 6 types also) or move your new code down. And speaking of the new code, it is misplac...
openjdk-notifier[bot] created a branch on openjdk/jdk
pr/21690 - JDK main-line development https://openjdk.org/projects/jdk
mlbridge[bot] created a comment on a pull request on openjdk/jdk
<!-- mlbridge webrev comment --> <!-- mlbridge webrev header --> ### Webrevs <!-- mlbridge webrev list --> * 00: [Full](https://webrevs.openjdk.org/?repo=jdk&pr=21690&range=00) ([9a5953d4](https:/...
ascarpino created a comment on a pull request on openjdk/jdk
Did you see the same performance improvement with specjvm2008::crypto.aes? I believe that should be running through this code as well unless there is something particular with RSA or any other asy...
honkar-jdk created a review comment on a pull request on openjdk/jdk
@seanjmullan Can you please advice on some of these src file javadoc related clean-up review comments. Do they need to be handled in this PR? Some of them seem out-of-scope for jep486 PR.
honkar-jdk created a review comment on a pull request on openjdk/jdk
@aivanov-jdk It was missed when -Djava.security.manager=allow was removed. Out of curiosity: does it affect the performance of CI testing if tests are run in `/othervm` mode when it is not need...
JornVernee created a review on a pull request on openjdk/jdk
Overall I like the idea. I took a look at the `foreign` benchmarks in particular, and don't see any issues with them.
jatin-bhateja created a review comment on a pull request on openjdk/jdk
Lets address it in a follow up PR
ferakocz created a comment on a pull request on openjdk/jdk
Could someone approve my changes so that I can integrate?
openjdk-notifier[bot] commented on a commit on openjdk/jdk
<!-- COMMIT COMMENT NOTIFICATION --> ### Review - [openjdk/jdk/21651](https://git.openjdk.org/jdk/pull/21651) ### Issues - [JDK-8342090](https://bugs.openjdk.org/browse/JDK-8342090) - [JDK-82885...
openjdk[bot] created a comment on a pull request on openjdk/jdk
<!-- Jmerge command reply message (2435913928) --> @TheRealMDoerr <!-- add contributor: 'Richard Reingruber <[email protected]>' --> Contributor `Richard Reingruber <[email protected]>` successfull...
openjdk[bot] created a comment on a pull request on openjdk/jdk
<!-- Jmerge command reply message (2435905904) --> @vicente-romero-oracle Pushed as commit d1540e2a49c7a41eb771fc9896c367187d070dec. :bulb: You may see a message that your pull request was closed ...
openjdk[bot] closed a pull request on openjdk/jdk
8342090: Infer::IncorporationBinaryOp::equals can produce side-effects
Type inference uses a cache to store incorporation operations already done. This way we can avoid redoing operations that once done won't produce any change. This can reduce the compilation time wh...
openjdk[bot] pushed 1 commit to master openjdk/jdk
- 8342090: Infer::IncorporationBinaryOp::equals can produce side-effects 8288590: javac failure: incompatible types: ca... d1540e2
openjdk[bot] created a comment on a pull request on openjdk/jdk
<!-- prepush d1540e2a49c7a41eb771fc9896c367187d070dec --> Going to push as commit d1540e2a49c7a41eb771fc9896c367187d070dec. Since your change was applied there have been 30 commits pushed to the `m...
aivanov-jdk created a review comment on a pull request on openjdk/jdk
Other tests in the set removed `/othervm`. Is it left intentionally? Looks like this question applies to all tests in `XMLEncoder/` directory.
aivanov-jdk created a review comment on a pull request on openjdk/jdk
Other tests in the set removed `/othervm`. Is it left intentionally?