From aharmaz at openjdk.org Tue Dec 2 15:48:26 2025 From: aharmaz at openjdk.org (Aymane Harmaz) Date: Tue, 2 Dec 2025 15:48:26 GMT Subject: Integrated: 8335: Duplicate values in tooltip for RSS and RSS (Peak) in memory tab In-Reply-To: References: Message-ID: <_W4e8n4DVZpxJvQt9iUR05c7J2hU7arHe71dbMQ7snI=.67d527e9-9788-42d6-8928-1a4e49917bb4@github.com> On Wed, 12 Nov 2025 09:38:16 GMT, Aymane Harmaz wrote: > This PR makes sure, we do not add attributes twice to the same renderer on the memory page > > The issue : > - The supplier used for merging events was having a side effect (calls to methods adding attributes to the same renderer) > - The supplier was consumed twice leading to the side effect being executed twice and that is mainly what causes the tooltip to contain duplicated attributes on the memory page > > The following screenshots respectively show the tooltips before and after the change > > tooltip-before > > tooltip-fixed This pull request has now been integrated. Changeset: 1e4c82b0 Author: Aymane Harmaz Committer: Marcus Hirt URL: https://git.openjdk.org/jmc/commit/1e4c82b0c3b95b553d5af42052b56fba3c6163f0 Stats: 6 lines in 1 file changed: 4 ins; 1 del; 1 mod 8335: Duplicate values in tooltip for RSS and RSS (Peak) in memory tab Reviewed-by: hdafgard ------------- PR: https://git.openjdk.org/jmc/pull/685 From hirt at openjdk.org Tue Dec 2 15:49:45 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Tue, 2 Dec 2025 15:49:45 GMT Subject: RFR: 6004: Should add tenuring summary to the GC visualization [v2] In-Reply-To: References: Message-ID: On Thu, 23 Oct 2025 14:35:54 GMT, Aymane Harmaz wrote: >> This change adds a new tab to display the tenuring distribution for GC cycles. >> >> There are 3 case scenarios: >> - Single GC cycle selected: A histogram chart is shown representing the tenuring distribution (total size across ages) for the selected cycle. >> - No GC cycle selected: An informational message is displayed to prompt the user to select a cycle. >> - Multiple GC cycles selected: An informational message is displayed indicating that only a single cycle can be viewed at a time. >> This enhancement improves visibility into object tenuring during individual GC cycles. >> >> Screenshot 2025-10-15 at 16 51 19 >> >> Screenshot 2025-10-15 at 16 51 34 >> >> Screenshot 2025-10-15 at 16 51 57 > > Aymane Harmaz has updated the pull request incrementally with one additional commit since the last revision: > > Implemented review comments Marked as reviewed by hirt (Lead). ------------- PR Review: https://git.openjdk.org/jmc/pull/679#pullrequestreview-3530851144 From hirt at openjdk.org Tue Dec 2 15:51:00 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Tue, 2 Dec 2025 15:51:00 GMT Subject: RFR: 8451: Should make the Event Type the first column of event types tree table In-Reply-To: References: Message-ID: On Mon, 27 Oct 2025 12:08:21 GMT, Aymane Harmaz wrote: > This PR made sure to display the event type column as the first column in the table related to event types tree view to make it easier for the user to identify the event type of the rows he will be looking at when multiple event types are selected. Marked as reviewed by hirt (Lead). ------------- PR Review: https://git.openjdk.org/jmc/pull/681#pullrequestreview-3530856397 From aharmaz at openjdk.org Tue Dec 2 15:52:15 2025 From: aharmaz at openjdk.org (Aymane Harmaz) Date: Tue, 2 Dec 2025 15:52:15 GMT Subject: Integrated: 8470: Unresolved placeholders shown in JFR result overview's table mode In-Reply-To: References: Message-ID: On Mon, 10 Nov 2025 08:51:46 GMT, Aymane Harmaz wrote: > This PR populates the summary, explanation, and solution fields of a rule displayed in the result overview table with the actual evaluated values. > > The following screenshots show the state before and after the change. > > unmapped-placeholders > > resolved placeholders This pull request has now been integrated. Changeset: 2c9ad544 Author: Aymane Harmaz Committer: Marcus Hirt URL: https://git.openjdk.org/jmc/commit/2c9ad5444289980c083f00bfe50e147b14edd188 Stats: 4 lines in 1 file changed: 1 ins; 0 del; 3 mod 8470: Unresolved placeholders shown in JFR result overview's table mode Reviewed-by: hdafgard ------------- PR: https://git.openjdk.org/jmc/pull/684 From aharmaz at openjdk.org Tue Dec 2 15:52:16 2025 From: aharmaz at openjdk.org (Aymane Harmaz) Date: Tue, 2 Dec 2025 15:52:16 GMT Subject: Integrated: 8427: Text getting overlapped on graph of Processes screen In-Reply-To: References: Message-ID: On Fri, 10 Oct 2025 16:09:39 GMT, Aymane Harmaz wrote: > When the width of the chart title label exceeds the left margin ( xOffset) reserved for it, the label is truncated without taking into account the width of the y-axis tick labels. This can potentially lead to overlapping between the chart title and the y-axis tick labels. > > Currently, the width of the y-axis tick labels is determined at the formatter level and is not dynamically available during chart label truncation. > > Two potential workarounds are : > > - Increase the xOffset size for the affected chart, allocating enough space to display both the chart title and the tick labels without overlap. > (This impacts only the chart in question.) > - Base Chart title label truncation on a default value for y-axis tick labels, even without access to exact tick widths. > (This impacts all charts.) > > I followed the first strategy. > > Screenshot of processes page post change : > > Screenshot 2025-10-10 at 17 02 40 This pull request has now been integrated. Changeset: a7813947 Author: Aymane Harmaz Committer: Marcus Hirt URL: https://git.openjdk.org/jmc/commit/a7813947a380af6e707a0ddcb5ef9b686e334b99 Stats: 1 line in 1 file changed: 0 ins; 0 del; 1 mod 8427: Text getting overlapped on graph of Processes screen Reviewed-by: hirt ------------- PR: https://git.openjdk.org/jmc/pull/678 From duke at openjdk.org Tue Dec 2 15:57:57 2025 From: duke at openjdk.org (viragpurnam123) Date: Tue, 2 Dec 2025 15:57:57 GMT Subject: RFR: 8474: Eclipse platform upgrade to 2025_09 Message-ID: Upgraded Eclipse Traget Platform to 2025-09. Upgraded Jetty version to 12.1.0 to keep the same jetty version as presnt in eclipse. ------------- Commit messages: - Eclipse platform upgrade to 2025_09 Changes: https://git.openjdk.org/jmc/pull/687/files Webrev: https://webrevs.openjdk.org/?repo=jmc&pr=687&range=00 Issue: https://bugs.openjdk.org/browse/JMC-8474 Stats: 407 lines in 11 files changed: 100 ins; 268 del; 39 mod Patch: https://git.openjdk.org/jmc/pull/687.diff Fetch: git fetch https://git.openjdk.org/jmc.git pull/687/head:pull/687 PR: https://git.openjdk.org/jmc/pull/687 From duke at openjdk.org Tue Dec 2 15:57:57 2025 From: duke at openjdk.org (viragpurnam123) Date: Tue, 2 Dec 2025 15:57:57 GMT Subject: RFR: 8474: Eclipse platform upgrade to 2025_09 In-Reply-To: References: Message-ID: On Mon, 24 Nov 2025 14:28:10 GMT, viragpurnam123 wrote: > Upgraded Eclipse Traget Platform to 2025-09. > Upgraded Jetty version to 12.1.0 to keep the same jetty version as presnt in eclipse. Hi @thegreystone, Could you please create a JIRA issue as I do not have access. Thanks. ------------- PR Comment: https://git.openjdk.org/jmc/pull/687#issuecomment-3571100890 From hirt at openjdk.org Tue Dec 2 15:57:58 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Tue, 2 Dec 2025 15:57:58 GMT Subject: RFR: 8474: Eclipse platform upgrade to 2025_09 In-Reply-To: References: Message-ID: On Mon, 24 Nov 2025 14:28:10 GMT, viragpurnam123 wrote: > Upgraded Eclipse Traget Platform to 2025-09. > Upgraded Jetty version to 12.1.0 to keep the same jetty version as presnt in eclipse. Done! How come you do not have access to Jira? You are a jmc reviewer! You should contact @Suchitainf or someone other at Oracle to see how to re-establish access. ------------- PR Comment: https://git.openjdk.org/jmc/pull/687#issuecomment-3602738335 PR Comment: https://git.openjdk.org/jmc/pull/687#issuecomment-3602746305 From hirt at openjdk.org Tue Dec 2 18:03:04 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Tue, 2 Dec 2025 18:03:04 GMT Subject: RFR: 5561: Support for Crypto Events in JMC In-Reply-To: References: Message-ID: On Mon, 17 Nov 2025 10:44:40 GMT, Suchita Chaturvedi wrote: > This PR enhances the JMC UI for adding new information related to crypto events: specifically X509CertificateEvent. > > This PR adds a new rule with respect to X509CertificateEvent which provides alert related to expired/expiring certificates and weak signature algorithms or weak key length or key type. The rule gives a basic overview of all the certificate ids which needs action or attention, however complete details are provided as part of a new screen - Security. > > Attaching the screenshots here for better reference: > > Rule Page: > > image > > Security Screen Page: > > image Changes requested by hirt (Lead). application/org.openjdk.jmc.flightrecorder.ui/org.openjdk.jmc.flightrecorder.ui_contexts.xml line 151: > 149: > 150: > 151: This seems to link to the exact same page as Using the TLAB Allocations Page. Copy/Paste error? ------------- PR Review: https://git.openjdk.org/jmc/pull/686#pullrequestreview-3531432535 PR Review Comment: https://git.openjdk.org/jmc/pull/686#discussion_r2582248094 From hirt at openjdk.org Tue Dec 2 18:07:15 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Tue, 2 Dec 2025 18:07:15 GMT Subject: RFR: 5561: Support for Crypto Events in JMC In-Reply-To: References: Message-ID: On Tue, 2 Dec 2025 17:53:55 GMT, Marcus Hirt wrote: >> This PR enhances the JMC UI for adding new information related to crypto events: specifically X509CertificateEvent. >> >> This PR adds a new rule with respect to X509CertificateEvent which provides alert related to expired/expiring certificates and weak signature algorithms or weak key length or key type. The rule gives a basic overview of all the certificate ids which needs action or attention, however complete details are provided as part of a new screen - Security. >> >> Attaching the screenshots here for better reference: >> >> Rule Page: >> >> image >> >> Security Screen Page: >> >> image > > application/org.openjdk.jmc.flightrecorder.ui/org.openjdk.jmc.flightrecorder.ui_contexts.xml line 151: > >> 149: >> 150: >> 151: > > This seems to link to the exact same page as Using the TLAB Allocations Page. Copy/Paste error? Should perhaps the messages be more action oriented? * Certificate with id x expired t days ago. It should be replaced. * Certificate with id y is using SHA-1. It should be updated to use SHA-256 or SHA-512. ------------- PR Review Comment: https://git.openjdk.org/jmc/pull/686#discussion_r2582282660 From hirt at openjdk.org Tue Dec 2 18:13:20 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Tue, 2 Dec 2025 18:13:20 GMT Subject: RFR: 8450: Improve usability of selectable action items by supporting keyboard and double-click interactions In-Reply-To: <-Ek8pTmlYneSm3pS00SQtgVdoTe2IrnvEaXP9cWf8jE=.9fc2ebc1-aa75-4654-826d-b54bb76be3d1@github.com> References: <-Ek8pTmlYneSm3pS00SQtgVdoTe2IrnvEaXP9cWf8jE=.9fc2ebc1-aa75-4654-826d-b54bb76be3d1@github.com> Message-ID: On Wed, 29 Oct 2025 14:21:03 GMT, Aymane Harmaz wrote: > This PR adds more interaction capabilities to tree items and checkbox tables: > > - Tree items: Added support for expanding/collapsing via double-click and Enter key press. > - Checkbox table: Added support for enabling/disabling via Enter key press. Nice! ------------- Marked as reviewed by hirt (Lead). PR Review: https://git.openjdk.org/jmc/pull/682#pullrequestreview-3531509228 From hirt at openjdk.org Tue Dec 2 18:19:29 2025 From: hirt at openjdk.org (Marcus Hirt) Date: Tue, 2 Dec 2025 18:19:29 GMT Subject: RFR: 8474: Eclipse platform upgrade to 2025_09 In-Reply-To: References: Message-ID: On Mon, 24 Nov 2025 14:28:10 GMT, viragpurnam123 wrote: > Upgraded Eclipse Traget Platform to 2025-09. > Upgraded Jetty version to 12.1.0 to keep the same jetty version as presnt in eclipse. Marked as reviewed by hirt (Lead). releng/platform-definitions/platform-definition-2025-09/platform-definition-2025-09.target line 4: > 2: